pythongh-106045: Fix venv creation from a python executable symlink

Author: mayeut

Lib/test/test_venv.py

@@ -744,6 +744,28 @@ def test_cli_without_scm_ignore_files(self):
         with self.assertRaises(FileNotFoundError):
             self.get_text_file_contents('.gitignore')
 
+    @requires_subprocess()
+    @unittest.skipUnless(can_symlink(), 'Needs symlinks')
+    def test_executable_symlink(self):
+        """
+        Test creation using a symlink to python executable.
+        """
+        rmtree(self.env_dir)
+
+        with tempfile.TemporaryDirectory() as python_symlink_dir:
+            executable_symlink = os.path.join(
+                python_symlink_dir, os.path.basename(sys.executable))
+            os.symlink(os.path.abspath(sys.executable), executable_symlink)
+            cmd = [executable_symlink, "-m", "venv", "--without-pip",
+                   self.env_dir]
+            subprocess.check_call(cmd)
+        data = self.get_text_file_contents('pyvenv.cfg')
+        executable = sys._base_executable
+        path = os.path.dirname(executable)
+        self.assertIn('home = %s' % path, data)
+        self.assertIn('executable = %s' %
+                      os.path.realpath(sys.executable), data)
+
 @requireVenvCreate
 class EnsurePipTest(BaseTest):
     """Test venv module installation of pip."""

Lib/venv/__init__.py

@@ -137,7 +137,7 @@ def create_if_needed(d):
                              'Python interpreter. Provide an explicit path or '
                              'check that your PATH environment variable is '
                              'correctly set.')
-        dirname, exename = os.path.split(os.path.abspath(executable))
+        dirname, exename = os.path.split(os.path.realpath(executable))
         if sys.platform == 'win32':
             # Always create the simplest name in the venv. It will either be a
             # link back to executable, or a copy of the appropriate launcher

Misc/ACKS

@@ -415,6 +415,7 @@ Eric Daniel
 Scott David Daniels
 Derzsi Dániel
 Lawrence D'Anna
+Matthieu Darbois
 Ben Darnell
 Kushal Das
 Jonathan Dasteel

Misc/NEWS.d/next/Library/2024-02-10-11-41-42.gh-issue-106045.eVZFt2.rst

@@ -0,0 +1,2 @@
+Fix ``venv`` creation from a python executable symlink. Patch by Matthieu
+Darbois.