[issue10112] Use -Wl, --dynamic-list=x.list, not -Xlinker -export-dynamic

[STINNER Victor]

Change by STINNER Victor :


--
nosy: +vstinner

___
Python tracker 

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

[issue10112] Use -Wl, --dynamic-list=x.list, not -Xlinker -export-dynamic

[Jan Kratochvil]

Jan Kratochvil added the comment:

It even crashes applications due to pollution of dynamic symbols namespace by 
application symbols as seen in:
  https://bugzilla.redhat.com/show_bug.cgi?id=1198158

--

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

[issue10112] Use -Wl, --dynamic-list=x.list, not -Xlinker -export-dynamic

[Tom Tromey]

Changes by Tom Tromey tro...@redhat.com:


--
nosy: +tromey

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

[issue10112] Use -Wl, --dynamic-list=x.list, not -Xlinker -export-dynamic

[Antoine Pitrou]

Changes by Antoine Pitrou pit...@free.fr:


--
nosy: +loewis

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue10112
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue10112] Use -Wl, --dynamic-list=x.list, not -Xlinker -export-dynamic

[Martin v . Löwis]

Martin v. Löwis mar...@v.loewis.de added the comment:

The patch is fairly incomplete: it assumes that the compiler supports 
__attribute__((visibility())), but it really needs to check for that.

Also, there is no chance that this can go into Python 2.7; retargetting for 3.3.

--
versions: +Python 3.3 -Python 2.7

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue10112
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue10112] Use -Wl, --dynamic-list=x.list, not -Xlinker -export-dynamic

[Arfrever Frehtes Taifersar Arahesis]

Changes by Arfrever Frehtes Taifersar Arahesis arfrever@gmail.com:


--
nosy: +Arfrever

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue10112
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue10112] Use -Wl, --dynamic-list=x.list, not -Xlinker -export-dynamic

[Jan Kratochvil]

Jan Kratochvil jan.kratoch...@redhat.com added the comment:

There was a bug for non-shared build of Python, thanks.

This patch is an improvement on its but it is not yet complete.
One should also replace -Xlinker -export-dynamic by -Wl,--dynamic-list so 
that linking application with libpython is no longer intrusive.

But I found out GDB does not need to be affected by this Bug.  When I use 
-fvisibility=hidden for GDB .c files then -Xlinker -export-dynamic has no 
longer any negative effect (GDB was using only -Wl,--dynamic-list now which 
works without libpython but not with libpython). Therefore I no longer mind 
much about this Bug.

Still at least:
1771024 libpython2.7.so.1.0
-
1757776 libpython2.7.so.1.0

Even with clean tree the upstream build generates for me:
Python build finished, but the necessary bits to build these modules were not 
found:
_bsddb _ssl   bsddb185
dl imageopsunaudiodev

(so sunaudiodev.c was changed without any testing of it)

--
Added file: http://bugs.python.org/file24575/hidden4.patch

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue10112
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue10112] Use -Wl, --dynamic-list=x.list, not -Xlinker -export-dynamic

[Jan Kratochvil]

Changes by Jan Kratochvil jan.kratoch...@redhat.com:


Removed file: http://bugs.python.org/file24221/hidden.patch

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue10112
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue10112] Use -Wl, --dynamic-list=x.list, not -Xlinker -export-dynamic

[Jan Kratochvil]

Jan Kratochvil jan.kratoch...@redhat.com added the comment:

What more can be done to get it fixed?  I do not mind Python but GDB linking is 
broken due to it.  And while I can workaround it in GDB I am not used for 
workarounding one Free package in another Free package.  Free software has the 
advantage problems can be fixed at the right place.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue10112
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue10112] Use -Wl, --dynamic-list=x.list, not -Xlinker -export-dynamic

[Meador Inge]

Meador Inge mead...@gmail.com added the comment:

When I try to build with the attached patch on the 2.7 branch many (if not all) 
of the modules fail to build:

Failed to build these modules:
_bisect_bsddb _codecs_cn  
_codecs_hk _codecs_iso2022_codecs_jp  
_codecs_kr _codecs_tw _collections
_csv   _ctypes_ctypes_test
_curses_curses_panel  _elementtree
_functools _hashlib   _heapq  
_hotshot   _io_json   
_locale_lsprof_multibytecodec 
_multiprocessing   _random_socket 
_sqlite3   _ssl   _struct 
_testcapi  _tkinter   array   
audioopbinascii   bz2 
cmath  cPicklecrypt   
cStringIO  datetime   dbm 
fcntl  future_builtinsgdbm
grpitertools  linuxaudiodev   
math   mmap   nis 
operator   ossaudiodevparser  
pyexpatreadline   resource
select spwd   strop   
syslog termiostime
unicodedatazlib   

I see several warnings in the build output:

*** WARNING: renaming _codecs_iso2022 since importing it failed: build/lib.lin
ux-x86_64-2.7/_codecs_iso2022.so: undefined symbol: PyExc_ValueError
..
*** WARNING: renaming linuxaudiodev since importing it failed: build/lib.linux
-x86_64-2.7/linuxaudiodev.so: undefined symbol: Py_Py3kWarningFlag
...
*** WARNING: renaming _tkinter since importing it failed: 
build/lib.linux-x86_64-2.7/_tkinter.so: undefined symbol: _Py_ZeroStruct

--
components: +Build -Library (Lib)
stage:  - patch review

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue10112
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue10112] Use -Wl, --dynamic-list=x.list, not -Xlinker -export-dynamic

[Jan Kratochvil]

Jan Kratochvil jan.kratoch...@redhat.com added the comment:

Here is the implementation.

Python/getargs.c was modified for:
ImportError: /usr/lib64/python2.7/lib-dynload/fcntlmodule.so: undefined symbol: 
_PyArg_ParseTuple_SizeT
but I guess that patch part should be different.  There is no need to do #ifdef 
HAVE_DECLSPEC_DLL as PyAPI_FUNC is properly defined even without it.  So the 
declaration+export block should be unified there.

The Modules/ files were modified for:
*** WARNING: renaming _ctypes_test since importing it failed: dynamic module 
does not define init function (init_ctypes_test)
*** WARNING: renaming _elementtree since importing it failed: dynamic module 
does not define init function (init_elementtree)
*** WARNING: renaming _hotshot since importing it failed: dynamic module does 
not define init function (init_hotshot)
*** WARNING: renaming _json since importing it failed: dynamic module does 
not define init function (init_json)
*** WARNING: renaming ossaudiodev since importing it failed: dynamic module 
does not define init function (initossaudiodev)

without this patch:
-rw-r--r-- 1 jkratoch jkratoch 5775829 Jan 12 17:42 
python-libs-2.7.2-18.fc17.x86_64.rpm
-r-xr-xr-x 1 jkratoch jkratoch 1738264 Jan 12 17:44 
./clean/python-libs-2.7.2-18.fc17.x86_64/usr/lib64/libpython2.7.so.1.0*
1226 libpython2.7.so.1.0 .dynsym exports
+
-rw-r--r-- 1 jkratoch jkratoch 1986161 Jan 12 17:48 
gdb-7.4.50.20120103-8.fc17.x86_64.rpm
-rwxr-xr-x 1 jkratoch jkratoch 5018800 Jan 12 19:13 
clean/gdb-7.4.50.20120103-8.fc17.x86_64/usr/bin/gdb*

with this patch:
-rw-r--r-- 1 jkratoch jkratoch 5762537 Jan 12 19:04 
python-libs-2.7.2-18hidden3.fc17.x86_64.rpm
-r-xr-xr-x 1 jkratoch jkratoch 1720920 Jan 12 19:06 
./hidden3/python-libs-2.7.2-18hidden3.fc17.x86_64/usr/lib64/libpython2.7.so.1.0*
1046 libpython2.7.so.1.0 .dynsym exports
+
-rw-r--r-- 1 jkratoch jkratoch 1886781 Jan 12 19:11 
gdb-7.4.50.20120103-8.fc17.x86_64.rpm
-rwxr-xr-x 1 jkratoch jkratoch 4732080 Jan 12 19:13 
hidden3/gdb-7.4.50.20120103-8.fc17.x86_64/usr/bin/gdb*

--
keywords: +patch
versions: +Python 2.7 -Python 3.2
Added file: http://bugs.python.org/file24221/hidden.patch

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue10112
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue10112] Use -Wl, --dynamic-list=x.list, not -Xlinker -export-dynamic

[Meador Inge]

Changes by Meador Inge mead...@gmail.com:


--
nosy: +meador.inge

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue10112
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue10112] Use -Wl, --dynamic-list=x.list, not -Xlinker -export-dynamic

[Jan Kratochvil]

New submission from Jan Kratochvil jan.kratoch...@redhat.com:

FSF GDB (and future Fedora GDBs) became 250KB smaller than before.  Python 
recently disabled this GDB build optimization so GDB is 250KB larger again.

-rwxr-xr-x 1 4524488 gdb-7.1-19.fc13.x86_64/usr/bin/gdb
-rwxr-xr-x 1 4266728 gdb-7.1-19dynamiclist.fc13.x86_64/usr/bin/gdb
-rw-r--r-- 1 2364656 gdb-7.1-19.fc13.x86_64.rpm
-rw-r--r-- 1 2274300 gdb-7.1-19dynamiclist.fc13.x86_64.rpm

Some Python binaries/libraries could probably also benefit from smaller 
pageable code image.

GDB regressed due to /usr/lib*/python*/config/Makefile contains:
  LINKFORSHARED=-Xlinker -export-dynamic

and GDB thus has to use it even for its own link, effectively disabling its:
  -Wl,--dynamic-list=./proc-service.list

AFAIK Python already contains the required exports list but it is used only on 
MS-Windows.  It should be utilized even for GNU/Linux builds.

--
components: Library (Lib)
messages: 118756
nosy: dmalcolm, jankratochvil
priority: normal
severity: normal
status: open
title: Use -Wl,--dynamic-list=x.list, not -Xlinker -export-dynamic
type: resource usage
versions: Python 3.2

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue10112
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue10112] Use -Wl, --dynamic-list=x.list, not -Xlinker -export-dynamic

[Éric Araujo]

Changes by Éric Araujo mer...@netwok.org:


--
nosy: +eric.araujo, pitrou

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue10112
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com