Re: [PATCH]: transform_libtool_scripts.py

2011-04-01 Thread Arfrever Frehtes Taifersar Arahesis
2010-12-16 21:36:45 Philip Martin napisaƂ(a): > Stefan Sperling writes: > > > make install on OpenBSD is now broken. > > It's not installing binaries but libtool wrappers: > > > > $ svn status > > /usr/bin/sed: can't load library '' > > /usr/bin/sed: can't load library '' > > /home/stsp/svn/prefi

Re: [PATCH]: transform_libtool_scripts.py

2010-12-16 Thread Philip Martin
Philip Martin writes: > I don't know what the new script is doing differently from the old > script. They both generate a whitespace separated list of libraries. > The new script should have a more accurate list. With some help from Stefan. The new script produces: LD_PRELOAD="" export LD_PRE

Re: [PATCH]: transform_libtool_scripts.py

2010-12-16 Thread Philip Martin
Stefan Sperling writes: > make install on OpenBSD is now broken. > It's not installing binaries but libtool wrappers: > > $ svn status > /usr/bin/sed: can't load library '' > /usr/bin/sed: can't load library '' > /home/stsp/svn/prefix/svn-trunk/bin/svn: error: > `/home/stsp/svn/svn-trunk/.libs/s

Re: [PATCH]: transform_libtool_scripts.py

2010-12-16 Thread Stefan Sperling
On Thu, Dec 16, 2010 at 09:19:45PM +0100, Stefan Sperling wrote: > On Thu, Dec 16, 2010 at 08:02:37PM +, Philip Martin wrote: > > Philip Martin writes: > > > > > Philip Martin writes: > > > > > >> I suppose we fix it by removing libsvn_test from the list of preload > > >> libraries. > > > >

Re: [PATCH]: transform_libtool_scripts.py

2010-12-16 Thread Stefan Sperling
On Thu, Dec 16, 2010 at 08:02:37PM +, Philip Martin wrote: > Philip Martin writes: > > > Philip Martin writes: > > > >> I suppose we fix it by removing libsvn_test from the list of preload > >> libraries. > > > > It's ugly, but I've hardcoded this for now. > > It's failing on the OpenBSD bu

Re: [PATCH]: transform_libtool_scripts.py

2010-12-16 Thread Philip Martin
Philip Martin writes: > Philip Martin writes: > >> Philip Martin writes: >> >>> I suppose we fix it by removing libsvn_test from the list of preload >>> libraries. >> >> It's ugly, but I've hardcoded this for now. > > It's failing on the OpenBSD buildbot: > > http://ci.apache.org/builders/svn-o

Re: [PATCH]: transform_libtool_scripts.py

2010-12-16 Thread Philip Martin
Philip Martin writes: > Philip Martin writes: > >> I suppose we fix it by removing libsvn_test from the list of preload >> libraries. > > It's ugly, but I've hardcoded this for now. It's failing on the OpenBSD buildbot: http://ci.apache.org/builders/svn-openbsd-i386/builds/46 START: auth-te

Re: [PATCH]: transform_libtool_scripts.py

2010-12-16 Thread Philip Martin
Philip Martin writes: > I suppose we fix it by removing libsvn_test from the list of preload > libraries. It's ugly, but I've hardcoded this for now. -- Philip

Re: [PATCH]: transform_libtool_scripts.py

2010-12-16 Thread Philip Martin
Julian Foad writes: > The patch committed in 1049944 has broken the Centos buildbot and my > local tests (in Ubuntu 10.04): all the C tests fail to run. If I try to > run one manually, the result is: > > $ obj-dir/subversion/tests/libsvn_client/client-test --list > /bin/sed: symbol lookup > erro

Re: [PATCH]: transform_libtool_scripts.py

2010-12-16 Thread Julian Foad
On Thu, 2010-12-16 at 11:17 +, Philip Martin wrote: > Martin Furter writes: > > > On Wed, 15 Dec 2010, Philip Martin wrote: > > > >> Martin Furter writes: > >> > >>> Doing some more testing I found another out that non-existing > >>> libraries have to be removed from LD_PRELOAD. > >> > >> Do

Re: [PATCH]: transform_libtool_scripts.py

2010-12-16 Thread Philip Martin
Martin Furter writes: > On Wed, 15 Dec 2010, Philip Martin wrote: > >> Martin Furter writes: >> >>> Doing some more testing I found another out that non-existing >>> libraries have to be removed from LD_PRELOAD. >> >> Does this interact with --enable-runtime-module-search? That's the >> switch

Re: [PATCH]: transform_libtool_scripts.py

2010-12-16 Thread Martin Furter
On Wed, 15 Dec 2010, Philip Martin wrote: Martin Furter writes: Doing some more testing I found another out that non-existing libraries have to be removed from LD_PRELOAD. Does this interact with --enable-runtime-module-search? That's the switch that causes Subversion to load RA/FS modul

Re: [PATCH]: transform_libtool_scripts.py

2010-12-15 Thread Philip Martin
Martin Furter writes: > Doing some more testing I found another out that non-existing > libraries have to be removed from LD_PRELOAD. Does this interact with --enable-runtime-module-search? That's the switch that causes Subversion to load RA/FS modules at runtime rather than linking them to the

Re: [PATCH]: transform_libtool_scripts.py

2010-12-15 Thread Martin Furter
Doing some more testing I found another out that non-existing libraries have to be removed from LD_PRELOAD. So here's a new patch. - Martin On Wed, 15 Dec 2010, Martin Furter wrote: Here is version 2 of the patch. On Tue, 14 Dec 2010, Philip Martin wrote: Martin Furter writes: The g

Re: [PATCH]: transform_libtool_scripts.py

2010-12-15 Thread Martin Furter
Here is version 2 of the patch. On Tue, 14 Dec 2010, Philip Martin wrote: Martin Furter writes: The generated script then has to be included into the tarball. I believe to remember that there's a script which creates the tarball. Can anyone give me a pointer where I can find it? tools/dis

Re: [PATCH]: transform_libtool_scripts.py

2010-12-14 Thread Philip Martin
Martin Furter writes: > The generated script then has to be included into the tarball. I > believe to remember that there's a script which creates the > tarball. Can anyone give me a pointer where I can find it? tools/dist/dist.sh The tarball is created after autogen.sh has been run; autogen.sh

Re: [PATCH]: transform_libtool_scripts.py

2010-12-14 Thread Martin Furter
On Tue, 14 Dec 2010, Stefan Sperling wrote: On Tue, Dec 14, 2010 at 06:12:54AM -0600, Peter Samuelson wrote: [Martin Furter] Configure has set SHELL=/bin/bash in Makefile. So this would solve one of the problems of that script. r1049057. The other problem is that it does not read build.

Re: [PATCH]: transform_libtool_scripts.py

2010-12-14 Thread Stefan Sperling
On Tue, Dec 14, 2010 at 06:12:54AM -0600, Peter Samuelson wrote: > > [Martin Furter] > > Configure has set SHELL=/bin/bash in Makefile. So this would solve > > one of the problems of that script. > > r1049057. > > > The other problem is that it does not read build.conf so always when > > anyone

Re: [PATCH]: transform_libtool_scripts.py

2010-12-14 Thread Peter Samuelson
[Martin Furter] > Configure has set SHELL=/bin/bash in Makefile. So this would solve > one of the problems of that script. r1049057. > The other problem is that it does not read build.conf so always when > anyone changes build.conf he has to remember that he may have update > transform_libtool_s

Re: [PATCH]: transform_libtool_scripts.py

2010-12-14 Thread Martin Furter
On Tue, 14 Dec 2010, Peter Samuelson wrote: [Martin Furter] I again fell over transform_libtool_scripts.sh on solaris. Would this patch help on Solaris? I believe autoconf goes to some effort to find a reasonable shell, in /usr/xpg4 or whatever. [[[ * Makefile.in (transform-libtool-scri

Re: [PATCH]: transform_libtool_scripts.py

2010-12-14 Thread Martin Furter
On Tue, 14 Dec 2010, Stefan Sperling wrote: On Tue, Dec 14, 2010 at 11:09:34AM +0100, Martin Furter wrote: Replace a disgusting shell script by nice python code. transform_libtool_scripts.py is more portable and also reads build.conf. While python is required to create buidscripts for a sub

Re: [PATCH]: transform_libtool_scripts.py

2010-12-14 Thread Peter Samuelson
[Martin Furter] > I again fell over transform_libtool_scripts.sh on solaris. Would this patch help on Solaris? I believe autoconf goes to some effort to find a reasonable shell, in /usr/xpg4 or whatever. [[[ * Makefile.in (transform-libtool-scripts): Run with $(SHELL) rather than /bin/sh,

Re: [PATCH]: transform_libtool_scripts.py

2010-12-14 Thread Stefan Sperling
On Tue, Dec 14, 2010 at 11:09:34AM +0100, Martin Furter wrote: > Replace a disgusting shell script by nice python code. > transform_libtool_scripts.py is more portable and also reads build.conf. While python is required to create buidscripts for a subversion distribution tarball, we do not require

[PATCH]: transform_libtool_scripts.py

2010-12-14 Thread Martin Furter
Hello I again fell over transform_libtool_scripts.sh on solaris. Here's a version in python which reads all the needed information from build.conf instead of searching the filesystem. I ran "make check" but it stopped at authz_tests: [[[ ... Running tests in authz_tests.py [36/87][SKIPPED] Py