From 863a09ef653efb2d99dae8d44e903ae3c98c0900 Mon Sep 17 00:00:00 2001
From: daimond113 <72147841+daimond113@users.noreply.github.com>
Date: Sat, 28 Sep 2024 23:46:09 +0200
Subject: [PATCH] refactor: always use target names in packages folders

---
 src/manifest/target.rs | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/src/manifest/target.rs b/src/manifest/target.rs
index b7e2679..9d238fe 100644
--- a/src/manifest/target.rs
+++ b/src/manifest/target.rs
@@ -75,9 +75,12 @@ impl TargetKind {
     /// The folder to store packages in for this target
     /// self is the project's target, dependency is the target of the dependency
     pub fn packages_folder(&self, dependency: &Self) -> String {
-        if self == dependency {
-            return "packages".to_string();
-        }
+        // the code below might seem better, but it's just going to create issues with users trying
+        // to use a build script, since imports would break between targets
+
+        // if self == dependency {
+        //     return "packages".to_string();
+        // }
 
         format!("{dependency}_packages")
     }