[issue1294959] Add sys.platlibdir to use /usr/lib64 on Fedora and SuSE

[STINNER Victor]

STINNER Victor  added the comment:

> Can we clarify the wording to clearly indicate it's to be used only for 
> Python modules/extensions and not system dynamic libs?

Sure. Can you please propose a different wording? English is not my first 
language.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1294959] Add sys.platlibdir to use /usr/lib64 on Fedora and SuSE

[Michał Górny]

Michał Górny  added the comment:

Can we clarify the wording to clearly indicate it's to be used only for Python 
modules/extensions and not system dynamic libs?

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1294959] Add sys.platlibdir to use /usr/lib64 on Fedora and SuSE

[STINNER Victor]

STINNER Victor  added the comment:

If you don't use the new --with-platlibdir configuration option, Python 3.9 
behaves exactly as Python 3.8: there is zero change.

Python 3.9 only builds a different sys.path if sys.platlibdir is different than 
"lib".

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1294959] Add sys.platlibdir to use /usr/lib64 on Fedora and SuSE

[Michał Górny]

Michał Górny  added the comment:

> Not at all. This means that it is possible to do so. It remains optional.

...but then sys.platlibdir is going to incorrectly list 'lib', isn't it?  
According to https://docs.python.org/3.9/library/sys.html#sys.platlibdir it's 
used 'to build the path of platform-specific dynamic libraries and the path of 
the standard library'.  This means that people relying on it for the former 
will get 32-bit libraries instead.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1294959] Add sys.platlibdir to use /usr/lib64 on Fedora and SuSE

[Miro Hrončok]

Miro Hrončok  added the comment:

> Does this mean that platforms using /usr/lib64 for shared libraries are now 
> forced to install Python into /usr/lib64/python*?

Not at all. This means that it is possible to do so. It remains optional.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1294959] Add sys.platlibdir to use /usr/lib64 on Fedora and SuSE

[Michał Górny]

Michał Górny  added the comment:

Does this mean that platforms using /usr/lib64 for shared libraries are now 
forced to install Python into /usr/lib64/python*?

--
nosy: +mgorny

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1294959] Add sys.platlibdir to use /usr/lib64 on Fedora and SuSE

[STINNER Victor]

STINNER Victor  added the comment:

I close this very old issue (15 years old!). Thank you *very much* to everybody 
who was involved in helping to get it fixed!

If I forgot someone, comment this issue or open a new one.

There is one last minor issue, but there is already a dedicated issue: 
bpo-24871 "freeze.py doesn't work on x86_64 Linux out of the box".

--
resolution:  -> fixed
stage: patch review -> resolved
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1294959] Add sys.platlibdir to use /usr/lib64 on Fedora and SuSE

[STINNER Victor]

STINNER Victor  added the comment:

Miro Hrončok ran rpmdiff to compare the Fedora with the old downstream patches 
and with the new upstream commit 8510f430781118d9b603c3a2f06945d6ebc5fe42: 
compare the Python package with and without setup.py changes.

https://src.fedoraproject.org/rpms/python39/pull-request/31#comment-39392

In short, there is no difference. Moreover, Miro Hrončok, Matej Cepl and me 
don't know the rationale of these setup.py changes.

I propose to drop them and wait until someone complains. If someone complains, 
we would know the rationale and so have a good reason to have these setup.py 
changes.

For readline: Fedora libreadline is already linked to libtinfo and so setup.py 
doesn't go up to `self.compiler.find_library_file(self.lib_dirs + 
['/usr/lib/termcap'], 'termcap')` code path. This code is deadcode on Fedora.

For /usr/local/lib vs /usr/local/lib64: I'm not sure why /usr/local/lib is used 
in the first place. Fedora installs libraries in /usr, not in /usr/local. Maybe 
it's there for users who install dependencies without Fedora: by installed them 
manually in /usr/local. Again, since it's unclear how /usr/local is supposed to 
be used, I prefer to leave it unchanged.

I closed the PR 18917.

Morevoer, if it becomes an issue in Fedora, we can start with a downstream 
patch again, to see how it goes, before going directly upstream.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1294959] Add sys.platlibdir to use /usr/lib64 on Fedora and SuSE

[miss-islington]

miss-islington  added the comment:


New changeset 3c29675d8736cb1860f342d872ae41e342c6d383 by Raúl Cumplido in 
branch 'master':
bpo-1294959: Fix typo for new attribute platlibdir. (GH-18960)
https://github.com/python/cpython/commit/3c29675d8736cb1860f342d872ae41e342c6d383


--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1294959] Add sys.platlibdir to use /usr/lib64 on Fedora and SuSE

[Roundup Robot]

Change by Roundup Robot :


--
nosy: +python-dev
nosy_count: 24.0 -> 25.0
pull_requests: +18310
pull_request: https://github.com/python/cpython/pull/18960

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1294959] Add sys.platlibdir to use /usr/lib64 on Fedora and SuSE

[Christian Heimes]

Change by Christian Heimes :


--
nosy:  -christian.heimes

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1294959] Add sys.platlibdir to use /usr/lib64 on Fedora and SuSE

[STINNER Victor]

Change by STINNER Victor :


--
pull_requests: +18268
pull_request: https://github.com/python/cpython/pull/18917

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1294959] Add sys.platlibdir to use /usr/lib64 on Fedora and SuSE

[STINNER Victor]

STINNER Victor  added the comment:

> Another related issue is bpo-34058: "Default Python 3.7 install broken on 
> openSUSE Leap 42.3: $PYTHONHOME/lib64/python3.7/lib-dynload/ not linked to 
> $PYTHONHOME/lib/python3.7/lib-dynload/".

I marked bpo-34058 as a duplicate of this issue.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1294959] Add sys.platlibdir to use /usr/lib64 on Fedora and SuSE

[STINNER Victor]

STINNER Victor  added the comment:

Another related issue is bpo-34058: "Default Python 3.7 install broken on 
openSUSE Leap 42.3: $PYTHONHOME/lib64/python3.7/lib-dynload/ not linked to 
$PYTHONHOME/lib/python3.7/lib-dynload/".

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1294959] Add sys.platlibdir to use /usr/lib64 on Fedora and SuSE

[STINNER Victor]

STINNER Victor  added the comment:

I chose to exclude setup.py changes from my commit 
8510f430781118d9b603c3a2f06945d6ebc5fe42, whereas the Fedora downstream patch 
has a few changes:
https://src.fedoraproject.org/rpms/python39/blob/master/f/00102-lib64.patch

See also bpo-14791: "setup.py only adds /prefix/lib, not /prefix/lib64".

diff --git a/setup.py b/setup.py
index 51e67fe4a5..bafa0bf99a 100644
--- a/setup.py
+++ b/setup.py
@@ -649,7 +649,7 @@ class PyBuildExt(build_ext):
 # directories (i.e. '.' and 'Include') must be first.  See issue
 # 10520.
 if not CROSS_COMPILING:
-add_dir_to_list(self.compiler.library_dirs, '/usr/local/lib')
+add_dir_to_list(self.compiler.library_dirs, '/usr/local/lib64')
 add_dir_to_list(self.compiler.include_dirs, '/usr/local/include')
 # only change this for cross builds for 3.3, issues on Mageia
 if CROSS_COMPILING:
@@ -955,11 +955,11 @@ class PyBuildExt(build_ext):
 elif curses_library:
 readline_libs.append(curses_library)
 elif self.compiler.find_library_file(self.lib_dirs +
- ['/usr/lib/termcap'],
+ ['/usr/lib64/termcap'],
  'termcap'):
 readline_libs.append('termcap')
 self.add(Extension('readline', ['readline.c'],
-   library_dirs=['/usr/lib/termcap'],
+   library_dirs=['/usr/lib64/termcap'],
extra_link_args=readline_extra_link_args,
libraries=readline_libs))
 else:

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1294959] Add sys.platlibdir to use /usr/lib64 on Fedora and SuSE

[STINNER Victor]

STINNER Victor  added the comment:


New changeset 8510f430781118d9b603c3a2f06945d6ebc5fe42 by Victor Stinner in 
branch 'master':
bpo-1294959: Add sys.platlibdir attribute (GH-18381)
https://github.com/python/cpython/commit/8510f430781118d9b603c3a2f06945d6ebc5fe42


--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1294959] Add sys.platlibdir to use /usr/lib64 on Fedora and SuSE

[STINNER Victor]

STINNER Victor  added the comment:

lib64_tests-3.py: Updated script to handle different ABI flags (release vs 
debug mode).

--
Added file: https://bugs.python.org/file48966/lib64_tests-3.py

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1294959] Add sys.platlibdir to use /usr/lib64 on Fedora and SuSE

[STINNER Victor]

Change by STINNER Victor :


--
versions:  -Python 3.7, Python 3.8

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1294959] Add sys.platlibdir to use /usr/lib64 on Fedora and SuSE

[STINNER Victor]

Change by STINNER Victor :


--
title: Problems with /usr/lib64 builds. -> Add sys.platlibdir to use /usr/lib64 
on Fedora and SuSE

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com