Page MenuHomePhabricator

unshelve: changes how date is set on interactive mode
ClosedPublic

Authored by navaneeth.suresh on Jul 24 2019, 8:43 AM.

Details

Summary

On an interactive unshelve, the remaining changes are shelved again
for later. This patch modifies the date of remaining shelved change
to the time of interactive shelve.

Diff Detail

Repository
rHG Mercurial
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

pulkit added a subscriber: pulkit.Jul 24 2019, 10:48 AM
pulkit added inline comments.
mercurial/shelve.py
838

IIUC opts.get('date') will be None here always.

navaneeth.suresh marked an inline comment as done.Jul 24 2019, 2:21 PM
pulkit added inline comments.Jul 25 2019, 5:22 AM
mercurial/shelve.py
838

We can just skip passing the date and it will automatically use the current date.

navaneeth.suresh marked an inline comment as done.Jul 25 2019, 8:11 AM
pulkit accepted this revision.Jul 25 2019, 8:48 AM
This revision is now accepted and ready to land.Jul 25 2019, 8:48 AM

the remaining changes are shelved again for later.

the shelve is updated with remaining changes.

(mentioning update is important because we are not creating a new shelve, we are updating that shelve)

This patch modifies the date of remaining shelved change to the time of interactive shelve.

This patch sets the date of updated shelve to the time when unshelve was performed.