[issue941346] AIX shared library fix

[Georg Brandl]

Georg Brandl  added the comment:

Backported to 3.1 in r88560, 2.7 in r88568.

--
status: open -> closed

___
Python tracker 

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

[issue941346] AIX shared library fix

[Antoine Pitrou]

Antoine Pitrou  added the comment:

Georg, the non-abiflags portion seems to need backporting.

--
assignee:  -> georg.brandl
priority: release blocker -> normal
status: closed -> open

___
Python tracker 

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

[issue941346] AIX shared library fix

[Sébastien Sablé]

Sébastien Sablé  added the comment:

Also Michael, I am working on the 3.2 branch, it may be different with Python 
2.7.1 but I haven't tested yet.

--

___
Python tracker 

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

[issue941346] AIX shared library fix

[Sébastien Sablé]

Sébastien Sablé  added the comment:

By the way... thanks Georg!

--

___
Python tracker 

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

[issue941346] AIX shared library fix

[Sébastien Sablé]

Sébastien Sablé  added the comment:

Are you using --enable-shared when compiling python?
That is when you should see the error (it could not find python.exp at some 
stage).

--

___
Python tracker 

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

[issue941346] AIX shared library fix

[Michael Haubenwallner]

Michael Haubenwallner  added the comment:

> You can remove -L\$(srcdir) on this line, but then you need to specify > the 
> full path to Modules/python.exp. See patch below:

Interesting, didn't experience this to be necessary with Python-2.7.1 here... 
Maybe because I do an in-source build, haven't tried an out-of-source build at 
all.

--

___
Python tracker 

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

[issue941346] AIX shared library fix

[Georg Brandl]

Georg Brandl  added the comment:

Committed in r88426.

--
status: open -> closed

___
Python tracker 

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

[issue941346] AIX shared library fix

[Antoine Pitrou]

Antoine Pitrou  added the comment:

Looks good to me.

--

___
Python tracker 

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

[issue941346] AIX shared library fix

[Sébastien Sablé]

Sébastien Sablé  added the comment:

Here is the full patch.
I also modified Makefile.pre.in to clean Modules/python.exp when doing 
distclean.

--
Added file: http://bugs.python.org/file20764/patch_aix_shared_32.diff

___
Python tracker 

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

[issue941346] AIX shared library fix

[Georg Brandl]

Georg Brandl  added the comment:

OK. Sébastien, could you make and attach a complete patch?

--

___
Python tracker 

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

[issue941346] AIX shared library fix

[Antoine Pitrou]

Antoine Pitrou  added the comment:

Agreed.

--

___
Python tracker 

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

[issue941346] AIX shared library fix

[Georg Brandl]

Georg Brandl  added the comment:

This looks like it should (and could) go into 3.2 final.  Agreed?

--
nosy: +barry, benjamin.peterson, georg.brandl
priority: normal -> release blocker
status: closed -> open

___
Python tracker 

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

[issue941346] AIX shared library fix

[Sébastien Sablé]

Sébastien Sablé  added the comment:

OK, Python with shared libraries is broken in trunk since the library was 
renamed to libpython3.2m.
Here is a patch to correct that:

Index: Modules/ld_so_aix.in
===
--- Modules/ld_so_aix.in(revision 88422)
+++ Modules/ld_so_aix.in(working copy)
@@ -131,7 +131,7 @@
   shift
 done
 
-if test "$objfile" = "libpython@VERSION@.so"; then
+if test "$objfile" = "libpython@VERSION@@ABIFLAGS@.so"; then
   ldsocoremode="true" 
 fi

--

___
Python tracker 

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

[issue941346] AIX shared library fix

[Sébastien Sablé]

Sébastien Sablé  added the comment:

Hum, I was incorrect in previous note:
You can remove -L\$(srcdir) on this line, but then you need to specify the full 
path to Modules/python.exp. See patch below:

Index: configure.in
===
--- configure.in(revision 88422)
+++ configure.in(working copy)
@@ -1642,7 +1655,7 @@
 then
case $ac_sys_system/$ac_sys_release in
AIX*)
-   BLDSHARED="\$(srcdir)/Modules/ld_so_aix \$(CC) 
-bI:Modules/python.exp -L\$(srcdir)"
+   BLDSHARED="\$(srcdir)/Modules/ld_so_aix \$(CC) 
-bI:\$(srcdir)/Modules/python.exp"
LDSHARED="\$(BINLIBDEST)/config/ld_so_aix \$(CC) 
-bI:\$(BINLIBDEST)/config/python.exp"
;;
IRIX/5*) LDSHARED="ld -shared";;


Also, I think there are other issues with building Python 3.2 with shared 
libraries. I am currently investigating.

--

___
Python tracker 

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

[issue941346] AIX shared library fix

[Sébastien Sablé]

Sébastien Sablé  added the comment:

Hi Micheal,

I don't remember why "-L\$(srcdir)" was added since it has been quite a long 
time.

I tried to recompile everything after removing it and everything compiled just 
fine, so I guess it can be removed.

regards

--

___
Python tracker 

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

[issue941346] AIX shared library fix

[Michael Haubenwallner]

Michael Haubenwallner  added the comment:

Sébastien, why did you need to add '-L$(srcdir)' to BLDSHARED in these patches?

As LDLIBRARY points to the immediate file 'libpython$(VERSION).so' instead of 
'-lpython$(VERSION)', I don't see the need for '-L$(srcdir)' at all.

The problem is that '-L$(srcdir)' adds '$(srcdir)' to the runpath too (as there 
is no '-blibpath' argument), opening a security hole for libpythonX.Y.so as 
well as the modules.so.

--
nosy: +haubi

___
Python tracker 

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

[issue941346] AIX shared library fix

[Sébastien Sablé]

Sébastien Sablé  added the comment:

Great! Thanks Antoine.

I checked quickly and there is a small correction to do on the 2.7 branch: this 
branch is different because there has been a change between python 2.x and 3.x 
in pep-3121 concerning the name of the entry function in a module 
(init to PyInit_)

So you need to change this line in Modules/ld_so_aix.in from:
  entry=PyInit_`echo $filename | sed "s/module.*//"`
to
  entry=init`echo $filename | sed "s/module.*//"`

Except for that it looks fine.

I will continue to try to make all unitary tests pass on AIX. And I would like 
to setup a buildbot that will compile and test the trunk every day and post the 
log on a public web site, in order to help keep this platform well supported.

--

___
Python tracker 

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

[issue941346] AIX shared library fix

[Antoine Pitrou]

Antoine Pitrou  added the comment:

I've committed the latest patch in r84680 (3.x), r84682 (3.1) and r84683 (2.7). 
Perhaps you want to check it's ok. Thank you anyway!

--
resolution:  -> fixed
stage: patch review -> committed/rejected
status: open -> closed

___
Python tracker 

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

[issue941346] AIX shared library fix

[Sébastien Sablé]

Changes by Sébastien Sablé :


Added file: 
http://bugs.python.org/file18832/py3k_aix61_shared_xlc_pathshared.txt.gz

___
Python tracker 

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

[issue941346] AIX shared library fix

[Sébastien Sablé]

Changes by Sébastien Sablé :


Removed file: 
http://bugs.python.org/file18820/py3k_aix61_shared_xlc_pathshared.txt.gz

___
Python tracker 

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

[issue941346] AIX shared library fix

[Sébastien Sablé]

Changes by Sébastien Sablé :


Added file: 
http://bugs.python.org/file18822/py3k_aix61_shared_gcc_patchshared.txt.gz

___
Python tracker 

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

[issue941346] AIX shared library fix

[Sébastien Sablé]

Changes by Sébastien Sablé :


Added file: 
http://bugs.python.org/file18821/py3k_aix61_static_gcc_patchshared.txt.gz

___
Python tracker 

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

[issue941346] AIX shared library fix

[Sébastien Sablé]

Changes by Sébastien Sablé :


Added file: 
http://bugs.python.org/file18820/py3k_aix61_shared_xlc_pathshared.txt.gz

___
Python tracker 

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

[issue941346] AIX shared library fix

[Sébastien Sablé]

Changes by Sébastien Sablé :


Added file: 
http://bugs.python.org/file18819/py3k_aix61_static_xlc_patchshared.txt.gz

___
Python tracker 

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

[issue941346] AIX shared library fix

[Sébastien Sablé]

Sébastien Sablé  added the comment:

Antoine, I tested this patch on py3k with both gcc and xlc in static and shared 
mode and I did not notice any issue.

I attach the build and test logs.

I think you can safely commit it.

--
Added file: 
http://bugs.python.org/file18818/py3k_aix61_static_xlc_nopatch.txt.gz

___
Python tracker 

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

[issue941346] AIX shared library fix

[Sébastien Sablé]

Sébastien Sablé  added the comment:

Antoine, I wanted to test this improvement (and others) on the branch py3k (I 
was using Python 2.7 and Python 3.1.2 for the moment). But I have some issues 
compiling this branch, even without any patch (see issue 9799).

So you may want to wait for the other issue to be resolved before to commit 
this patch on branch py3k.

--

___
Python tracker 

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

[issue941346] AIX shared library fix

[Sébastien Sablé]

Changes by Sébastien Sablé :


Added file: 
http://bugs.python.org/file18797/python312_AIX61_shared_gcc_patchshared.txt.gz

___
Python tracker 

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

[issue941346] AIX shared library fix

[Sébastien Sablé]

Sébastien Sablé  added the comment:

Here are the build logs for Python 3.1.2 modified with this patch and compiled 
with gcc.
The extensions are correctly built.

I also compiled an extension not directly provided in Python source (cx_Oracle) 
and it worked fine also.

--
Added file: 
http://bugs.python.org/file18796/python312_AIX61_static_gcc_patchshared.txt.gz

___
Python tracker 

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

[issue941346] AIX shared library fix

[Antoine Pitrou]

Antoine Pitrou  added the comment:

> Anyway, I will run the test with gcc on Wednesday and know for sure if
> that works.

Thank you! If it works, it's good for commit.

--

___
Python tracker 

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

[issue941346] AIX shared library fix

[Sébastien Sablé]

Sébastien Sablé  added the comment:

Hum, forget my previous note;I checked ld_so_aix and it actually calls $CC to 
handle linking not ld. I suppose gcc will call the native ld.

Anyway, I will run the test with gcc on Wednesday and know for sure if that 
works.

--

___
Python tracker 

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

[issue941346] AIX shared library fix

[Sébastien Sablé]

Sébastien Sablé  added the comment:

Yes Antoine, gcc only handles compilation; the linker is explicitly called 
through the ld_so_aix script which handles calling the native ld with the right 
flags to import symbols.

I will check with gcc and attach the log on Wednesday (not at work tomorrow) 
but there should be no problem.

--

___
Python tracker 

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

[issue941346] AIX shared library fix

[Antoine Pitrou]

Antoine Pitrou  added the comment:

> * there are no more warnings for 'Duplicate symbol': I changed
> build_ext.py to not link extensions with libpython since the symbols
> are already defined by python.exp

Could this affect building extensions with gcc under AIX? Or does gcc delegate 
to the AIX linker?

--

___
Python tracker 

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

[issue941346] AIX shared library fix

[Sébastien Sablé]

Sébastien Sablé  added the comment:

Here is the patch for Python 2.7.

The only difference compared to Python 3.1.2 is that we don't rename 
init to PyInit_.

--
versions: +Python 2.6 -Python 3.2
Added file: http://bugs.python.org/file18768/Python-2.7_shared_AIX.diff

___
Python tracker 

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

[issue941346] AIX shared library fix

[Antoine Pitrou]

Antoine Pitrou  added the comment:

Thank you. For the record, here is the patch adapted for 3.2 development 
branch. It looks ok to me.

--
stage: unit test needed -> patch review
versions: +Python 3.2 -Python 2.6
Added file: http://bugs.python.org/file18767/aix32.diff

___
Python tracker 

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

[issue941346] AIX shared library fix

[Sébastien Sablé]

Changes by Sébastien Sablé :


--
versions: +Python 2.6, Python 3.1

___
Python tracker 

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

[issue941346] AIX shared library fix

[Sébastien Sablé]

Changes by Sébastien Sablé :


Added file: 
http://bugs.python.org/file18765/python312_AIX61_shared_patch_shared.txt.gz

___
Python tracker 

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

[issue941346] AIX shared library fix

[Sébastien Sablé]

Changes by Sébastien Sablé :


Added file: 
http://bugs.python.org/file18764/python312_AIX61_static_patch_shared.txt.gz

___
Python tracker 

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

[issue941346] AIX shared library fix

[Sébastien Sablé]

Changes by Sébastien Sablé :


Added file: 
http://bugs.python.org/file18763/python312_AIX61_static_nopatch.txt.gz

___
Python tracker 

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

[issue941346] AIX shared library fix

[Sébastien Sablé]

Sébastien Sablé  added the comment:

Hi,

I have updated the patch and adapted it for Python 3.1.2:

* this time ld_so_aix is moved to ld_so_aix.in which is converted to ld_so_aix 
by 'configure' so that it has the right version number for the python library

* there are no more warnings for 'Duplicate symbol': I changed build_ext.py to 
not link extensions with libpython since the symbols are already defined by 
python.exp

* I made a modification to ld_so_aix specifically for Python 3 as according to 
pep-3121 the entry function has been renamed from init to 
PyInit_

I will provide another patch for Python 2.7 (nearly the same, but without the 
modification needed by pep-3121).
I also attach the build log for Python 3.1.2 without and with this patch.

regards

--
Added file: http://bugs.python.org/file18762/Python-3.1.2_shared_AIX.diff

___
Python tracker 

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

[issue941346] AIX shared library fix

[Sébastien Sablé]

Sébastien Sablé  added the comment:

Le 01/09/2010 00:16, Antoine Pitrou a écrit :
> In dynamic builds, there seem to be lots of messages such as:
>
> ld: 0711-224 WARNING: Duplicate symbol: .PyLong_FromString
> ld: 0711-224 WARNING: Duplicate symbol: PyLong_FromString
>
> Are you sure this is normal?

This is not normal, but I always had this warning and this does not 
prevent the libraries from working OK.
I will try to see if I can clean that.

> In any case, if you find a way to cleanup ld_so_aix, this should probably be 
> checked in anyway. I doubt anybody else tests under AIX, so it's reasonable 
> to trust you on this.
>
> Even better if you manage to diagnose and/or fix some of the test failures, 
> by the way. (if not, you can still open separate issues for each of them)

Great! this is the kind of answer I was looking for. I will cleanup my 
patch to make it compatible with Python 2.7 and Python 3.1.2 and test it 
with those 2 versions.

I will also correct as many unitary tests as possible with this platform.
I already have a patch for issue 4026.

--

___
Python tracker 

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

[issue941346] AIX shared library fix

[Antoine Pitrou]

Antoine Pitrou  added the comment:

In dynamic builds, there seem to be lots of messages such as:

ld: 0711-224 WARNING: Duplicate symbol: .PyLong_FromString
ld: 0711-224 WARNING: Duplicate symbol: PyLong_FromString

Are you sure this is normal?

In any case, if you find a way to cleanup ld_so_aix, this should probably be 
checked in anyway. I doubt anybody else tests under AIX, so it's reasonable to 
trust you on this.

Even better if you manage to diagnose and/or fix some of the test failures, by 
the way. (if not, you can still open separate issues for each of them)

--

___
Python tracker 

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

[issue941346] AIX shared library fix

[Sébastien Sablé]

Changes by Sébastien Sablé :


Added file: 
http://bugs.python.org/file18695/python266_AIX61_shared_xlc_patchshared.txt.gz

___
Python tracker 

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

[issue941346] AIX shared library fix

[Sébastien Sablé]

Changes by Sébastien Sablé :


Added file: 
http://bugs.python.org/file18694/python266_AIX61_shared_xlc_nopatch.txt.gz

___
Python tracker 

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

[issue941346] AIX shared library fix

[Sébastien Sablé]

Changes by Sébastien Sablé :


Added file: 
http://bugs.python.org/file18693/python266_AIX61_shared_gcc_patchshared.txt.gz

___
Python tracker 

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

[issue941346] AIX shared library fix

[Sébastien Sablé]

Changes by Sébastien Sablé :


Added file: 
http://bugs.python.org/file18692/python266_AIX61_shared_gcc_nopatch.txt.gz

___
Python tracker 

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

[issue941346] AIX shared library fix

[Sébastien Sablé]

Changes by Sébastien Sablé :


Added file: 
http://bugs.python.org/file18691/python266_AIX61_static_xlc_patchshared.txt.gz

___
Python tracker 

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

[issue941346] AIX shared library fix

[Sébastien Sablé]

Changes by Sébastien Sablé :


Added file: 
http://bugs.python.org/file18690/python266_AIX61_static_xlc_nopatch.txt.gz

___
Python tracker 

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

[issue941346] AIX shared library fix

[Sébastien Sablé]

Changes by Sébastien Sablé :


Added file: 
http://bugs.python.org/file18689/python266_AIX61_static_gcc_patchshared.txt.gz

___
Python tracker 

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

[issue941346] AIX shared library fix

[Sébastien Sablé]

Changes by Sébastien Sablé :


Added file: 
http://bugs.python.org/file18688/python266_AIX61_static_gcc_nopatch.txt.gz

___
Python tracker 

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

[issue941346] AIX shared library fix

[Sébastien Sablé]

Sébastien Sablé  added the comment:

Hi,

As reported in this issue and issue 1756343 and issue 1542544, Python does not 
produce a shared python library on AIX even with the --enable-shared flag.

I had provided a patch to correct that, but it was breaking static compilation 
of Python on AIX.

We are upgrading our AIX build environment to AIX 6.1 and Python 2.6.6, so I 
took the time to review this patch again and to clean it so that it works well 
when compiling Python statically and dynamically, with either gcc and xlc_r.

I attach the resulting patch, and the build and unit test logs. You can notice 
that not only this patch allows to compile Python dynamically on AIX, but also 
that it improves the unit tests results when compiling Python statically.

For example with the xlc_r compiler and Python compiled statically, the results 
without the patch are:
  274 tests OK.
  22 tests failed
  70 tests skipped
The results with the patch are:
  286 tests OK.
  21 tests failed
  59 tests skipped

A minor issue with this patch is that "libpython2.6.so" is hardcoded in the 
ld_so_aix file. That should be modified by moving ld_so_aix to ld_so_aix.in and 
using $LDLIBRARY, or by using an explicit flag when calling ld_so_aix to 
compile libpythonx.y.so.

If you agree to the principle of this patch, I can make the extra work to clean 
ld_so_aix and test this patch on Python 2.7 and 3.1.

regards

--
Added file: http://bugs.python.org/file18687/Python-2.6.6_shared_AIX.diff

___
Python tracker 

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

[issue941346] AIX shared library fix

[Loris Bennett]

Loris Bennett  added the comment:

Addition to last message:

More importantly, without "-Wl", the flags are not passed to the linker
and the tcl/tk libs will not be found.

--

___
Python tracker 

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

[issue941346] AIX shared library fix

[Loris Bennett]

Loris Bennett  added the comment:

Shouldn't line 170 be

   CCOPT="$CCOPT -Wl,-bM:SRE -Wl,-T512 -Wl,-H512 -Wl,-brtl
-Wl,-bnortllib -lm -o $objfile"

instead of

   CCOPT="$CCOPT -Wl,-bM:SRE -Wl,-T512 -Wl,-H512 -brtl -bnortllib -lm -o
$objfile"

Otherwise the -brtl and -bnortllib flags are interpreted as compiler
flags and errors like the following will occur:

   gcc: unrecognized option '-brtl'

--
nosy: +loris

___
Python tracker 

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

[issue941346] AIX shared library fix

[Daniel Diniz]

Changes by Daniel Diniz :


--
components: +Build, Extension Modules -None
stage:  -> test needed
type:  -> feature request
versions: +Python 2.7

___
Python tracker 

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

[issue941346] AIX shared library fix

[Sébastien Sablé]

Sébastien Sablé <[EMAIL PROTECTED]> added the comment:

Hum, I tried to recompile Python without the --enable-shared option on
AIX and it crashed; so this patch may not be compatible with a static
version of Python (though I have other patches in my version - cf issue
3526 - which may be incompatible).

I will do a bit of cleanup in my environment and try to make a version
of the patch that works also when compiling a static Python.

___
Python tracker <[EMAIL PROTECTED]>

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

[issue941346] AIX shared library fix

[Antoine Pitrou]

Antoine Pitrou <[EMAIL PROTECTED]> added the comment:

Sébastien, what does your patch change exactly? Does it build a shared
lib for Python by default? If that's the case, I'm not sure it's a good
idea due to AIX's particular way of treating shared libraries; it might
break e.g. when trying to embed Python in another program.
(I say this while being an AIX novice, but I did have to compile Python
under AIX for embedded use in another third-party software...
unfortunately right now I don't have access to an AIX machine with a
compiler)

--
nosy: +pitrou

___
Python tracker <[EMAIL PROTECTED]>

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

[issue941346] AIX shared library fix

[Sébastien Sablé]

Sébastien Sablé <[EMAIL PROTECTED]> added the comment:

Hi,

I have modified a bit this patch to make it work with Python 2.5.2 (see
attached patched).

It is a great enhancement for Python users on AIX to be able to use a
shared library instead of a static one, so I would appreciate if this
feature could be integrated in trunk.

We have various AIX 5.2 and 5.3 servers in the Sungard company where I
work, and we could run some tests according to your instructions. It may
also be possible to provide some limited access to one of those servers
if needed but it would require some time and paper work.

What kind of access would you need in order to validate those patchs?

thanks in advance

--
keywords: +patch
nosy: +sable
Added file: http://bugs.python.org/file11081/patch_AIX_shared.diff

___
Python tracker <[EMAIL PROTECTED]>

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