Move skipping exclusion before inserting remote_list

src/cli/sync.rs

@@ -344,6 +344,17 @@ fn download_handler(
         "expected_path: {:?}, relative: {:?}",
         expected_path, relative_filepath
     );
+
+    // We should put relative filepath into exclusion manager here
+    if task_context.exclusion_manager.match_str(&relative_filepath)
+        == regex_process::Comparison::Stop
+    {
+        // This should be run before inserting remote_list.
+        // Otherwise newly excluded files will not be deleted later.
+        info!("Skipping excluded {:?}", &relative_filepath);
+        return;
+    }
+
     {
         if !thr_context
             .remote_list
@@ -353,20 +364,12 @@ fn download_handler(
         {
             // It is possible that multiple tasks might download the same file
             // (generated by apt/yum parser, etc.)
-            // skip when we find that some thread has already downloaded it
+            // skip when we find that some threads has already downloaded it
             info!("Skipping already handled {:?}", &expected_path);
             return;
         }
     }
 
-    // We should put relative filepath into exclusion manager here
-    if task_context.exclusion_manager.match_str(&relative_filepath)
-        == regex_process::Comparison::Stop
-    {
-        info!("Skipping excluded {:?}", &relative_filepath);
-        return;
-    }
-
     let mut should_download = true;
     let mut skip_if_exists = false;
     for i in &args.skip_if_exists {