From 48cd4a355c4cdf97b00a5a5104317fbd88f3d841 Mon Sep 17 00:00:00 2001
From: Cesar Douady <cesar.douady@doliam.fr>
Date: Thu, 2 May 2024 09:36:33 +0200
Subject: [PATCH] fix a rerun loop in some cases of manual targets

---
 src/rpc_job.cc | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/rpc_job.cc b/src/rpc_job.cc
index 58e5bd50..e3455d0f 100644
--- a/src/rpc_job.cc
+++ b/src/rpc_job.cc
@@ -40,10 +40,8 @@ ::pair_s<bool/*ok*/> do_file_actions( ::vector_s* unlnks/*out*/ , ::vmap_s<FileA
 	for( auto const& [f,a] : pre_actions ) {                                                                                        // pre_actions are adequately sorted
 		SWEAR(+f) ;                                                                                                                 // acting on root dir is non-sense
 		switch (a.tag) {
-			case FileActionTag::None     :                                                                                          break ;
-			case FileActionTag::Uniquify : if (uniquify(nfs_guard.change(f))) append_to_string(msg,"uniquified ",mk_file(f),'\n') ; break ;
-			case FileActionTag::Mkdir    : mkdir(f,nfs_guard) ;                                                                     break ;
-			case FileActionTag::Unlnk    : {
+			case FileActionTag::None  :
+			case FileActionTag::Unlnk : {
 				FileSig sig { nfs_guard.access(f) } ;
 				if (!sig) break ;                                                                                                    // file does not exist, nothing to do
 				bool done = true/*garbage*/ ;
@@ -58,7 +56,9 @@ ::pair_s<bool/*ok*/> do_file_actions( ::vector_s* unlnks/*out*/ , ::vmap_s<FileA
 				if ( done && unlnks ) unlnks->push_back(f) ;
 				ok &= done ;
 			} break ;
-			case FileActionTag::Rmdir :
+			case FileActionTag::Uniquify : if (uniquify(nfs_guard.change(f))) append_to_string(msg,"uniquified ",mk_file(f),'\n') ; break ;
+			case FileActionTag::Mkdir    : mkdir(f,nfs_guard) ;                                                                     break ;
+			case FileActionTag::Rmdir    :
 				if (!keep_dirs.contains(f))
 					try                     { rmdir(nfs_guard.change(f)) ;                                                        }
 					catch (::string const&) { for( ::string d=f ; +d ; d = dir_name(d) ) if (!keep_dirs.insert(d).second) break ; } // if a dir cannot rmdir'ed, no need to try those uphill