build.hook.mkVirtualEnv: Set no globbing for make-venv hook

Inserting globs seems to have broken. I thought it was adequetaly escaped but apparently not.

build/checks/default.nix

@@ -62,6 +62,21 @@ let
             touch $out
           '';
 
+      # Test that glob isn't treated strangely by bash and command doesn't explode
+      make-venv-flags =
+        pkgs.runCommand "venv-run-build-flags-test"
+          {
+            nativeBuildInputs = [
+              (testVenv.overrideAttrs (_old: {
+                venvIgnoreCollisions = [ "*" ];
+              }))
+            ];
+          }
+          ''
+            pyproject-build --help > /dev/null
+            touch $out
+          '';
+
       make-venv-unittest =
         pkgs.runCommand "venv-unittest"
           {

build/hooks/pyproject-make-venv-hook.sh

@@ -4,7 +4,9 @@ pyprojectMakeVenv() {
   echo "Executing pyprojectMakeVenv"
   runHook preInstall
 
+  set -f
   @pythonInterpreter@ @makeVenvScript@ --python @python@ "$out" --env "NIX_PYPROJECT_DEPS" $mkVirtualenvFlags
+  set +f
 
   runHook postInstall
   echo "Finished executing pyprojectMakeVenv"

build/packages.nix

@@ -4,7 +4,7 @@
 }:
 let
   inherit (resolvers) resolveCyclic resolveNonCyclic;
-  inherit (lib) makeScope concatStringsSep escapeShellArg;
+  inherit (lib) makeScope concatStringsSep;
 
   mkResolveBuildSystem =
     set:
@@ -58,8 +58,8 @@ let
             NIX_PYPROJECT_DEPS = concatStringsSep ":" (pkgsFinal.resolveVirtualEnv spec);
             dontMoveLib64 = true;
             mkVirtualenvFlags = concatStringsSep " " (
-              map (path: "--skip ${escapeShellArg path}") finalAttrs.venvSkip
-              ++ map (pat: "--ignore-collisions ${escapeShellArg pat}") finalAttrs.venvIgnoreCollisions
+              map (path: "--skip ${path}") finalAttrs.venvSkip
+              ++ map (pat: "--ignore-collisions ${pat}") finalAttrs.venvIgnoreCollisions
             );
           };