diff options
author | Oswald Buddenhagen <oswald.buddenhagen@gmx.de> | 2023-03-22 10:42:28 +0100 |
---|---|---|
committer | Oswald Buddenhagen <oswald.buddenhagen@gmx.de> | 2023-06-01 07:54:44 +0000 |
commit | 807c012a969af9c3f59c25a36f0b140586cba4d5 (patch) | |
tree | a4b3701b6d37a1f5776e005ba77caf265c6278bb | |
parent | 10ddfaa544ac730e4d6e575a91f041587fd644d5 (diff) |
gpush/gpick: heed --dry-run more carefully
skip saving the state also after branch tracking activity (both local
and remote), to make the operation truly read-only (and thus
reproducible) - the remaining exception are the PatchSet refs gpick
fetches from Gerrit, but this is basically a cache.
Change-Id: I8e3f708945ee0e6d33483177b394de90e833edc2
Reviewed-by: Alexandru Croitor <alexandru.croitor@qt.io>
-rw-r--r-- | bin/git_gpush.pm | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/bin/git_gpush.pm b/bin/git_gpush.pm index 2eb2118..9fc7cf7 100644 --- a/bin/git_gpush.pm +++ b/bin/git_gpush.pm @@ -2104,7 +2104,7 @@ sub _source_map_finish_initial() } source_map_finish(); - save_state() if ($sm_changed); + save_state($dry_run) if ($sm_changed); } # Update the target branches of local Changes according to the data @@ -2159,7 +2159,7 @@ sub _update_target_branches($) # The Changes may be printed again later. delete $$_{annotation} foreach (@changed); } - save_state() if ($need_save); + save_state($dry_run) if ($need_save); } ####################### |