Skip to content

gh-110824 Fix test_sysconfig.test_library on framework builds. #113298

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 11 commits into from
Jan 2, 2024
Merged

gh-110824 Fix test_sysconfig.test_library on framework builds. #113298

Changes from 5 commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
4c77a0c
fix check for LDLIBRARY on framework builds
smontanaro Dec 19, 2023
0f9f2b9
better structure, suggested by Ned Deily
smontanaro Dec 19, 2023
1242405
accommodate --with-framework-name
smontanaro Dec 20, 2023
e051bfa
Merge branch 'python:main' into issue110824
smontanaro Dec 20, 2023
a64e09e
Merge branch 'main' into issue110824
smontanaro Dec 20, 2023
65f0ec0
Merge branch 'python:main' into issue110824
smontanaro Dec 21, 2023
a727550
patch requested by Ronald Oussoren to fix treatment of sys._framework
smontanaro Dec 21, 2023
4f53021
Merge branch 'main' into issue110824
smontanaro Dec 21, 2023
db3fb1f
Merge branch 'python:main' into issue110824
smontanaro Dec 22, 2023
d33b2f4
Merge branch 'main' into issue110824
ned-deily Jan 2, 2024
97a6a97
Temporarily skip test_library for macOS framework builds until the is…
ned-deily Jan 2, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 4 additions & 1 deletion Lib/test/test_sysconfig.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -413,7 +413,10 @@ def test_library(self):
else:
self.assertTrue(library.startswith(f'libpython{major}.{minor}'))
self.assertTrue(library.endswith('.a'))
self.assertTrue(ldlibrary.startswith(f'libpython{major}.{minor}'))
if sys.platform == 'darwin' and sys._framework:
self.assertTrue(ldlibrary.endswith(f'{major}.{minor}/{sys._framework}'))
else:
self.assertTrue(ldlibrary.startswith(f'libpython{major}.{minor}'))

@unittest.skipUnless(sys.platform == "darwin", "test only relevant on MacOSX")
@requires_subprocess()
Expand Down