Fix copying of directory deps

Fixes #13

appimage/private/tool/tool.py

@@ -45,7 +45,10 @@ def make_appimage(
             src = Path(file["src"]).resolve()
             dst = (appdir / file["dst"]).resolve()
             dst.parent.mkdir(parents=True, exist_ok=True)
-            shutil.copy(src=src, dst=dst, follow_symlinks=True)
+            try:
+                shutil.copy(src=src, dst=dst, follow_symlinks=True)
+            except IsADirectoryError:
+                shutil.copytree(src=src, dst=dst, symlinks=True)
         for link in manifest_data["symlinks"]:
             linkfile = (appdir / Path(link["linkname"])).resolve()
             linkfile.parent.mkdir(parents=True, exist_ok=True)

tests/BUILD

@@ -15,7 +15,10 @@ appimage_test(
 py_binary(
     name = "test_py",
     srcs = ["test.py"],
-    data = ["data.txt"],
+    data = [
+        "data.txt",
+        "dir",  # this is a relative directory, not a target label
+    ],
     main = "test.py",
     deps = [requirement("click")],
 )

tests/dir/BUILD

@@ -0,0 +1,5 @@
+filegroup(
+    name = "dir",
+    srcs = ["file_in_dir.txt"],
+    visibility = ["//tests:__subpackages__"],
+)

tests/test.py

@@ -3,11 +3,13 @@
 import click
 
 DATA_DEP = Path("tests/data.txt")
+DIR_DEP = Path("tests/dir/file_in_dir.txt")
 
 
 def test_datadep() -> None:
     assert DATA_DEP.is_file(), f"{DATA_DEP} does not exist"
     assert (s := DATA_DEP.stat().st_size) == 591, f"{DATA_DEP} has wrong size {s}"
+    assert DIR_DEP.is_file(), f"{DIR_DEP} does not exist"
 
 
 @click.command()