Don't force a pull after fix_params

If there is no difference in fingerprints, there is no need to force a
pull.

Signed-off-by: Konstantin Ryabitsev <konstantin@linuxfoundation.org>

1 files changed, 6 insertions, 6 deletions

diff --git a/grokmirror/pull.py b/grokmirror/pull.py
index 605ba06..dfc9a0a 100755
--- a/grokmirror/pull.py
+++ b/grokmirror/pull.py
@@ -331,19 +331,19 @@ def pull_worker(config, q_pull, q_spa, q_done):
                     logger.info('    purge: %s', gitdir)
                     shutil.rmtree(fullpath)
 
+        if action == 'fix_params':
+            logger.info(' reconfig: %s', gitdir)
+            set_repo_params(fullpath, repoinfo)
+
         if action == 'fix_remotes':
             logger.info(' reorigin: %s', gitdir)
             success = fix_remotes(toplevel, gitdir, site, config)
             if success:
-                action = 'fix_params'
+                set_repo_params(fullpath, repoinfo)
+                action = 'pull'
             else:
                 success = False
 
-        if action == 'fix_params':
-            logger.info(' reconfig: %s', gitdir)
-            set_repo_params(fullpath, repoinfo)
-            action = 'pull'
-
         if action == 'reclone':
             logger.info('  reclone: %s', gitdir)
             try: