mirror of
https://github.com/python/cpython.git
synced 2025-09-26 18:29:57 +00:00
GH-128469: Revert "warn when libpython was loaded from outside the build directory (#128645)" (#129506)
This commit is contained in:
parent
674befbd7b
commit
31c82c28f9
2 changed files with 0 additions and 50 deletions
|
@ -1,13 +1,7 @@
|
||||||
import copy
|
import copy
|
||||||
import ntpath
|
import ntpath
|
||||||
import os
|
|
||||||
import pathlib
|
import pathlib
|
||||||
import posixpath
|
import posixpath
|
||||||
import shutil
|
|
||||||
import subprocess
|
|
||||||
import sys
|
|
||||||
import sysconfig
|
|
||||||
import tempfile
|
|
||||||
import unittest
|
import unittest
|
||||||
|
|
||||||
from test.support import verbose
|
from test.support import verbose
|
||||||
|
@ -870,37 +864,6 @@ class MockGetPathTests(unittest.TestCase):
|
||||||
actual = getpath(ns, expected)
|
actual = getpath(ns, expected)
|
||||||
self.assertEqual(expected, actual)
|
self.assertEqual(expected, actual)
|
||||||
|
|
||||||
|
|
||||||
class RealGetPathTests(unittest.TestCase):
|
|
||||||
@unittest.skipUnless(
|
|
||||||
sysconfig.is_python_build(),
|
|
||||||
'Test only available when running from the buildir',
|
|
||||||
)
|
|
||||||
@unittest.skipUnless(
|
|
||||||
any(sys.platform.startswith(p) for p in ('linux', 'freebsd', 'centos')),
|
|
||||||
'Test only support on Linux-like OS-es (support LD_LIBRARY_PATH)',
|
|
||||||
)
|
|
||||||
@unittest.skipUnless(
|
|
||||||
sysconfig.get_config_var('LDLIBRARY') != sysconfig.get_config_var('LIBRARY'),
|
|
||||||
'Test only available when using a dynamic libpython',
|
|
||||||
)
|
|
||||||
def test_builddir_wrong_library_warning(self):
|
|
||||||
library_name = sysconfig.get_config_var('INSTSONAME')
|
|
||||||
with tempfile.TemporaryDirectory() as tmpdir:
|
|
||||||
shutil.copy2(
|
|
||||||
os.path.join(sysconfig.get_config_var('srcdir'), library_name),
|
|
||||||
os.path.join(tmpdir, library_name)
|
|
||||||
)
|
|
||||||
env = os.environ.copy()
|
|
||||||
env['LD_LIBRARY_PATH'] = tmpdir
|
|
||||||
process = subprocess.run(
|
|
||||||
[sys.executable, '-c', ''],
|
|
||||||
env=env, check=True, capture_output=True, text=True,
|
|
||||||
)
|
|
||||||
error_msg = 'The runtime library has been loaded from outside the build directory'
|
|
||||||
self.assertTrue(process.stderr.startswith(error_msg), process.stderr)
|
|
||||||
|
|
||||||
|
|
||||||
# ******************************************************************************
|
# ******************************************************************************
|
||||||
|
|
||||||
DEFAULT_NAMESPACE = dict(
|
DEFAULT_NAMESPACE = dict(
|
||||||
|
|
|
@ -785,19 +785,6 @@ if os_name != 'nt' and build_prefix:
|
||||||
base_exec_prefix = config.get('base_exec_prefix') or EXEC_PREFIX or base_prefix
|
base_exec_prefix = config.get('base_exec_prefix') or EXEC_PREFIX or base_prefix
|
||||||
|
|
||||||
|
|
||||||
# ******************************************************************************
|
|
||||||
# MISC. RUNTIME WARNINGS
|
|
||||||
# ******************************************************************************
|
|
||||||
|
|
||||||
# When running Python from the build directory, if libpython is dynamically
|
|
||||||
# linked, the wrong library might be loaded.
|
|
||||||
if build_prefix and library and not dirname(abspath(library)).startswith(build_prefix):
|
|
||||||
msg = f'The runtime library has been loaded from outside the build directory ({library})!'
|
|
||||||
if os_name == 'posix':
|
|
||||||
msg += ' Consider setting LD_LIBRARY_PATH=. to force it to be loaded from the build directory.'
|
|
||||||
warn(msg)
|
|
||||||
|
|
||||||
|
|
||||||
# ******************************************************************************
|
# ******************************************************************************
|
||||||
# SET pythonpath FROM _PTH FILE
|
# SET pythonpath FROM _PTH FILE
|
||||||
# ******************************************************************************
|
# ******************************************************************************
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue