Keep intermediate git state if --allow-noop was passed

Author: Kobzol

src/bin/rustc_josh_sync.rs

@@ -42,8 +42,10 @@ enum Command {
         #[clap(long)]
         upstream_commit: Option<String>,
 
-        /// By default, the `pull` command will exit with status code 2 if there is nothing to pull.
-        /// If you instead want to exit successfully in that case, pass this flag.
+        /// By default, the `pull` command will exit with status code 2 if there is nothing to pull,
+        /// and reset git to the original state.
+        /// If you instead want to exit successfully and keep the intermediate changes
+        /// in that case, pass this flag.
         #[clap(long)]
         allow_noop: bool,
 
@@ -110,7 +112,7 @@ fn main() -> anyhow::Result<()> {
             let ctx = load_context(&config_path, &rust_version_path)?;
             let josh = get_josh_proxy(verbose)?;
             let sync = GitSync::new(ctx.clone(), josh, verbose);
-            match sync.rustc_pull(upstream_repo, upstream_commit) {
+            match sync.rustc_pull(upstream_repo, upstream_commit, allow_noop) {
                 Ok(result) => {
                     if !maybe_create_gh_pr(
                         &ctx.config.full_repo_name(),

src/sync.rs

@@ -45,6 +45,7 @@ impl GitSync {
         &self,
         upstream_repo: String,
         upstream_commit: Option<String>,
+        allow_noop: bool,
     ) -> Result<PullResult, RustcPullError> {
         // The upstream commit that we want to pull
         let upstream_sha = if let Some(sha) = upstream_commit {
@@ -215,17 +216,29 @@ After you fix the conflicts, `git add` the changes and run `git merge --continue
 
         // This is the easy case, no merge was performed, so we bail
         if current_sha == sha_pre_merge {
-            eprintln!(
-                "No merge was performed, no changes to pull were found. Rolling back the preparation commit."
-            );
+            if allow_noop {
+                git_reset.disarm();
+                eprintln!(
+                    "No merge was performed, no changes to pull were found. Keeping the intermediate state."
+                );
+            } else {
+                eprintln!(
+                    "No merge was performed, no changes to pull were found. Rolling back the preparation commit."
+                );
+            }
             return Err(RustcPullError::NothingToPull);
         }
 
         // But it can be more tricky - we can have only empty merge/rollup merge commits from
         // rustc, so a merge was created, but the in-tree diff can still be empty.
         // In that case we also bail.
         if self.has_empty_diff(&sha_pre_merge) {
-            eprintln!("Only empty changes were pulled. Rolling back the preparation commit.");
+            if allow_noop {
+                git_reset.disarm();
+                eprintln!("Only empty changes were pulled. Keeping the intermediate state.");
+            } else {
+                eprintln!("Only empty changes were pulled. Rolling back the preparation commit.");
+            }
             return Err(RustcPullError::NothingToPull);
         }