revert: fix the inconsistency of status msgs in --interactive mode
Before this patch we were priting every action msg before actually performing that action and that was resulting in inconsistencies; like in --interactive session if user decided to not revert any changes in a file foo, still there will be a msg on console saying "reverting foo". To fix this, I have made some changes to print status msg just before the action it is going to perform, no matter if --interactive or not. Changes made in test-revert-interactive.t reflect the changed behavior. There are also some changes in test-revert.t because of change in the order of messages. Differential Revision: https://phab.mercurial-scm.org/D4380
Showing
- mercurial/cmdutil.py 32 additions, 10 deletionsmercurial/cmdutil.py
- tests/test-backout.t 1 addition, 1 deletiontests/test-backout.t
- tests/test-confused-revert.t 2 additions, 2 deletionstests/test-confused-revert.t
- tests/test-fileset-generated.t 2 additions, 2 deletionstests/test-fileset-generated.t
- tests/test-import-git.t 3 additions, 3 deletionstests/test-import-git.t
- tests/test-import.t 1 addition, 1 deletiontests/test-import.t
- tests/test-issue660.t 1 addition, 1 deletiontests/test-issue660.t
- tests/test-largefiles-misc.t 2 additions, 2 deletionstests/test-largefiles-misc.t
- tests/test-largefiles.t 1 addition, 1 deletiontests/test-largefiles.t
- tests/test-lfs-test-server.t 4 additions, 4 deletionstests/test-lfs-test-server.t
- tests/test-merge-remove.t 1 addition, 1 deletiontests/test-merge-remove.t
- tests/test-revert-interactive.t 8 additions, 14 deletionstests/test-revert-interactive.t
- tests/test-revert.t 42 additions, 42 deletionstests/test-revert.t
Loading
Please register or sign in to comment