ScriptMaker without enquoting executable path makes wrong scripts

[linw1995]

• I have searched the issue tracker and believe that this is not a duplicate.

I discover this problem by using PDM with pdm-venv on MacOS. The default virtualenvs directory on MacOS is ~/Library/Application Support/pdm/venvs. And this reveals the bug under the installer code of PDM.

pdm/pdm/installers/installers.py

Lines 49 to 54 in a2098b4

	def install_wheel(self, wheel: Wheel) -> None:
	paths = self.environment.get_paths()
	maker = distlib.scripts.ScriptMaker(None, None)
	maker.variants = set(("",))
	maker.executable = self.environment.interpreter.executable
	wheel.install(paths, maker)

The ScriptMaker._get_shebang assumes the executable path is taken care of by the caller. So the path is not enquoted.

https://github.com/vsajip/distlib/blob/4f54a3117db58f6bb00c69ab9a892170b91acd3b/distlib/scripts.py#L160-L162

I will create a pull request for resolving this issue via passing the enquoted executable path to ScriptMaker.

Steps to reproduce

1. install pdm and pdm-venv
2. configure the path with whitespace as venvs dir
3. use pdm to install a package with scripts. like pytest
4. run it

Actual behavior

fail to run due to the corrupted script generated by the un-enquoted exetuable path.

Expected behavior

run success

Environment Information

# Paste the output of `pdm info && pdm info --env` below:
PDM version:        1.5.0b1
Python Interpreter: /Users/linw1995/Library/Application Support/pdm/venvs/pdm-Tbf7Oqsf-3.9/bin/python (3.9)
Project Root:       /private/tmp/pdm
Project Packages:   None
{
  "implementation_name": "cpython",
  "implementation_version": "3.9.2",
  "os_name": "posix",
  "platform_machine": "x86_64",
  "platform_release": "20.3.0",
  "platform_system": "Darwin",
  "platform_version": "Darwin Kernel Version 20.3.0: Thu Jan 21 00:07:06 PST 2021; root:xnu-7195.81.3~1/RELEASE_X86_64",
  "python_full_version": "3.9.2",
  "platform_python_implementation": "CPython",
  "python_version": "3.9",
  "sys_platform": "darwin"
}