Remove directories in /bin when ensuring portability (#103)

Author: neilmehta24

CHANGELOG.rst

@@ -12,6 +12,11 @@ See the fragment files in the `changelog.d directory`_.
 
 .. scriv-insert-here
 
+Fixed
+-----
+
+- Remove directories from /bin when building layers
+
 .. _changelog-0.2.1:
 
 0.2.1 — 2024-12-05

src/venvstacks/stacks.py

@@ -1388,10 +1388,13 @@ def _create_new_environment(self, *, lock_only: bool = False) -> None:
     def _ensure_portability(self) -> None:
         # Wrapper and activation scripts are not used on deployment targets,
         # so drop them entirely rather than making them portable
-        for executable in self.executables_path.iterdir():
-            if not executable.name.lower().startswith("python"):
-                print(f"    Dropping potentially non-portable file {str(executable)!r}")
-                executable.unlink()
+        for item in self.executables_path.iterdir():
+            if item.is_dir():
+                print(f"    Dropping directory {str(item)!r}")
+                shutil.rmtree(item)
+            elif not item.name.lower().startswith("python"):
+                print(f"    Dropping potentially non-portable file {str(item)!r}")
+                item.unlink()
         # Symlinks within the build folder should be relative
         # Symlinks outside the build folder shouldn't exist
         build_path = self.build_path