◐ Shell
reader mode source ↗
Skip to content

bpo-34032: Add platlibdir as an option for distinguishing /usr/lib and /usr/lib64#8068

Closed
mcepl wants to merge 2 commits into
python:masterfrom
mcepl:34032-multilib
Closed

bpo-34032: Add platlibdir as an option for distinguishing /usr/lib and /usr/lib64#8068
mcepl wants to merge 2 commits into
python:masterfrom
mcepl:34032-multilib

Conversation

@mcepl

@mcepl mcepl commented Jul 3, 2018 •
edited
Loading

Copy link
Copy Markdown
Contributor

Build system has been modified to allow /usr/lib64 as well as /usr/lib into install path as well as to the search path for the Linux distributions which distinguish between 32bit and 64bit libraries.

https://bugs.python.org/issue34032

@vstinner , @doko42 , who else?

vstinner
vstinner requested changes Jul 4, 2018
View reviewed changes

@vstinner vstinner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hide comment

The new "platlibdir" variable should be documented in Doc/library/sysconfig.rst:
https://docs.python.org/dev/library/sysconfig.html#installation-paths

You have to add it to the list, and also add a ".. versionchanged:: 3.8 (...)" markup to document the change.

IMHO it's worth it to also add a second NEWS entry for the sysconfig change (new "platlibdir" variable).

@bedevere-bot

bedevere-bot commented Jul 4, 2018

Copy link
Copy Markdown

A Python core developer has requested some changes be made to your pull request before we can consider merging it. If you could please address their requests along with any other requests in other reviews from core developers that would be appreciated.

Once you have made the requested changes, please leave a comment on this pull request containing the phrase I have made the requested changes; please review again. I will then notify any core developers who have left a review that you're ready for them to take another look at this pull request.

vstinner
vstinner reviewed Jul 4, 2018
View reviewed changes

@vstinner vstinner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hide comment

The OpenSUSE patch only changes the "unix_prefix" scheme, but the Fedora patch also changes the 'unix_home' scheme. To be honest, I don't understand well this change.

@mcepl

mcepl commented Jul 4, 2018 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Hmm, this pull request fails on AppVeyor with:

======================================================================
ERROR: test_user_site (distutils.tests.test_install.InstallTestCase)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "C:\projects\cpython\lib\distutils\tests\test_install.py", line 114, in test_user_site
    cmd.ensure_finalized()
  File "C:\projects\cpython\lib\distutils\cmd.py", line 107, in ensure_finalized
    self.finalize_options()
  File "C:\projects\cpython\lib\distutils\command\install.py", line 330, in finalize_options
    self.create_home_path()
  File "C:\projects\cpython\lib\distutils\command\install.py", line 536, in create_home_path
    if path.startswith(home) and not os.path.isdir(path):
AttributeError: 'NoneType' object has no attribute 'startswith'
----------------------------------------------------------------------
Ran 246 tests in 32.131s
FAILED (errors=1, skipped=24)
test test_distutils failed

and with

test_user_similar (test.test_sysconfig.TestSysConfig) ... test test_sysconfig failed
ERROR
======================================================================
ERROR: test_get_path (test.test_sysconfig.TestSysConfig)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "C:\projects\cpython\lib\test\test_sysconfig.py", line 100, in test_get_path
    res = get_path(name, scheme)
  File "C:\projects\cpython\lib\sysconfig.py", line 506, in get_path
    return get_paths(scheme, vars, expand)[name]
  File "C:\projects\cpython\lib\sysconfig.py", line 496, in get_paths
    return _expand_vars(scheme, vars)
  File "C:\projects\cpython\lib\sysconfig.py", line 170, in _expand_vars
    res[key] = os.path.normpath(_subst_vars(value, vars))
  File "C:\projects\cpython\lib\sysconfig.py", line 151, in _subst_vars
    raise AttributeError('{%s}' % var) from None
AttributeError: {'installed_base'}
======================================================================
ERROR: test_user_similar (test.test_sysconfig.TestSysConfig)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "C:\projects\cpython\lib\test\test_sysconfig.py", line 267, in test_user_similar
    global_path = get_path(name, 'posix_prefix')
  File "C:\projects\cpython\lib\sysconfig.py", line 506, in get_path
    return get_paths(scheme, vars, expand)[name]
  File "C:\projects\cpython\lib\sysconfig.py", line 496, in get_paths
    return _expand_vars(scheme, vars)
  File "C:\projects\cpython\lib\sysconfig.py", line 170, in _expand_vars
    res[key] = os.path.normpath(_subst_vars(value, vars))
  File "C:\projects\cpython\lib\sysconfig.py", line 151, in _subst_vars
    raise AttributeError('{%s}' % var) from None
AttributeError: {'installed_base'}
----------------------------------------------------------------------
Ran 21 tests in 0.084s

Hmm, I guess this whole thing should be limited just to Unix (non-Windows, non-Mac) systems, right?

@mcepl
add /usr/lib64 into install path as well as to the search path. …
ad144a3 
Originally created for https://bugzilla.opensuse.org/637176
@mcepl
bpo-34032: Add platlibdir as an option for distinguishing /usr/lib an… …
2d6ed3b 
…d /usr/lib64

Build system has been modified to allow /usr/lib64 as well as /usr/lib into
install path as well as to the search path for the Linux distributions which
distinguish between 32bit and 64bit libraries.
@Fak3 Fak3 mentioned this pull request Jul 27, 2019
Closed
@Fak3

Fak3 commented Jul 27, 2019

Copy link
Copy Markdown

Was this superseded by #11755 ?

@vstinner vstinner mentioned this pull request Feb 6, 2020
Merged
@vstinner

vstinner commented Mar 10, 2020

Copy link
Copy Markdown
Member

I merged PR #18381 instead which is very similar. Thanks @mcepl for helping on PR #18381.

@vstinner vstinner closed this Mar 10, 2020
@mcepl mcepl deleted the 34032-multilib branch March 10, 2020 10:40
@manisandro manisandro mentioned this pull request Jun 3, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vstinner vstinner vstinner requested changes

Assignees

No one assigned

Labels

awaiting changes

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@mcepl @bedevere-bot @Fak3 @vstinner @the-knights-who-say-ni