FIXUP: Implement review comments

dangerzone/__init__.py

@@ -4,7 +4,9 @@
 try:
     from . import vendor  # type: ignore [attr-defined]
 
-    sys.path.insert(0, vendor.__path__[0])
+    vendor_path =  vendor.__path__[0]
+    print(f"Using vendored PyMuPDF libraries from '{vendor_path}'")
+    sys.path.insert(0, vendor_path)
 except ImportError:
     pass
 

debian/rules

@@ -1,9 +1,9 @@
 #!/usr/bin/make -f
 export PYBUILD_NAME=dangerzone
 export DEB_BUILD_OPTIONS=nocheck
-#export PYBUILD_INSTALL_ARGS=--install-lib=/usr/lib/python3/dist-packages
-#export PYTHONDONTWRITEBYTECODE=1
-#export DH_VERBOSE=1
+export PYBUILD_INSTALL_ARGS=--install-lib=/usr/lib/python3/dist-packages
+export PYTHONDONTWRITEBYTECODE=1
+export DH_VERBOSE=1
 
 %:
 	dh $@ --with python3 --buildsystem=pybuild

install/linux/vendor-pymupdf.py

@@ -22,16 +22,19 @@ def main():
     container_requirements_txt = subprocess.check_output(cmd)
 
     print(f">>> Vendoring PyMuPDF under '{args.dest}'", file=sys.stderr)
+    # We prefer to call the CLI version of `pip`, instead of importing it directly, as
+    # instructed here:
+    # https://pip.pypa.io/en/latest/user_guide/#using-pip-from-your-program
     cmd = [
-        "python3",
+        sys.executable,
         "-m",
         "pip",
         "install",
         "--no-cache-dir",
         "--no-compile",
-        "-t",
+        "--target",
         args.dest,
-        "-r",
+        "--requirements",
         "/proc/self/fd/0",  # XXX: pip does not read requirements.txt from stdin
     ]
     subprocess.check_output(cmd, input=container_requirements_txt)