From 0ce39682c5d7eb6bf40f43a4e378d712bbc247eb Mon Sep 17 00:00:00 2001
From: Wildan M <willnode@wellosoft.net>
Date: Fri, 8 May 2026 21:18:15 +0700
Subject: [PATCH] Fix push command not respecting ignore rule

---
 mk/repo.mk      |  2 +-
 src/bin/repo.rs | 46 +++++++++++++++++++---------------------------
 2 files changed, 20 insertions(+), 28 deletions(-)

diff --git a/mk/repo.mk b/mk/repo.mk
index e2e4ae44..7a2a9522 100644
--- a/mk/repo.mk
+++ b/mk/repo.mk
@@ -124,7 +124,7 @@ endif
 ifeq ($(PODMAN_BUILD),1)
 	$(PODMAN_RUN) make $@ ALLOW_FSTOOLS=$(FSTOOLS_IN_PODMAN)
 else
-	$(REPO_BIN) push $(foreach f,$(subst $(comma), ,$*),$(f)) "--sysroot=$(MOUNT_DIR)"
+	$(REPO_BIN) push $(foreach f,$(subst $(comma), ,$*),$(f)) "--sysroot=$(MOUNT_DIR)" $(COOKBOOK_OPTS)
 endif
 ifeq ($(ALLOW_FSTOOLS),1)
 	@if [ -f $(MOUNTED_TAG) ]; then \
diff --git a/src/bin/repo.rs b/src/bin/repo.rs
index 7983ef01..e8a85a94 100644
--- a/src/bin/repo.rs
+++ b/src/bin/repo.rs
@@ -567,7 +567,7 @@ fn parse_args(args: Vec<String>) -> Result<(CliConfig, CliCommand, Vec<CookRecip
 
     let lock = &get_config().recipe_lock;
 
-    let mut recipes = if let Some(conf) = config.filesystem.as_ref() {
+    let mut recipes = {
         let repo_binary = override_filesystem_repo_binary;
 
         // Expand deps for "source" + "local" and "binary"
@@ -585,24 +585,26 @@ fn parse_args(args: Vec<String>) -> Result<(CliConfig, CliCommand, Vec<CookRecip
             special_rules.insert(recipe_name.clone(), rule.to_string());
             // lock rules does not recurse as it's done already in the file
         }
-        for (recipe_name_str, recipe_config) in conf.packages.iter() {
-            let Ok(recipe_name) = PackageName::new(recipe_name_str) else {
-                continue;
-            };
+        if let Some(conf) = config.filesystem.as_ref() {
+            for (recipe_name_str, recipe_config) in conf.packages.iter() {
+                let Ok(recipe_name) = PackageName::new(recipe_name_str) else {
+                    continue;
+                };
 
-            let rule = if let Some(rule) = special_rules.get(&recipe_name) {
-                rule.as_str()
-            } else if let PackageConfig::Build(rule) = recipe_config {
-                special_rules.insert(recipe_name.clone(), rule.to_string());
-                rule
-            } else {
-                default_rule
-            };
+                let rule = if let Some(rule) = special_rules.get(&recipe_name) {
+                    rule.as_str()
+                } else if let PackageConfig::Build(rule) = recipe_config {
+                    special_rules.insert(recipe_name.clone(), rule.to_string());
+                    rule
+                } else {
+                    default_rule
+                };
 
-            if rule == "source" || rule == "local" {
-                source_names.push(recipe_name);
-            } else if rule == "binary" {
-                binary_names.push(recipe_name);
+                if rule == "source" || rule == "local" {
+                    source_names.push(recipe_name);
+                } else if rule == "binary" {
+                    binary_names.push(recipe_name);
+                }
             }
         }
         source_names = CookRecipe::get_all_deps_names_recursive(&source_names, true)?;
@@ -706,16 +708,6 @@ fn parse_args(args: Vec<String>) -> Result<(CliConfig, CliCommand, Vec<CookRecip
         }
 
         recipes
-    } else {
-        if config.with_package_deps || command.is_pushing() {
-            recipe_names = CookRecipe::get_package_deps_recursive(&recipe_names, true)?;
-        }
-        if command.is_building() || command.is_pushing() {
-            let include_dev = command.is_building();
-            CookRecipe::get_build_deps_recursive(&recipe_names, include_dev)?
-        } else {
-            CookRecipe::from_list(recipe_names.clone())?
-        }
     };
 
     if !get_config().recipe_lock.is_empty() {