* Charles Wilson wrote on Sun, Jun 27, 2010 at 02:51:21PM CEST:
> > In that case, Ralf's suggested libfile_$(transliterated implib name)
> > is used, because we have the .la file available which allows that
> > shortcut. The only time we need `dlltool --identify' is when
> > dlpreopening a non-lib
Hi Gary,
* Gary V. Vaughan wrote on Mon, Jun 28, 2010 at 01:24:39AM CEST:
> Looking through the XSI substitutions, or more correctly the bash/ksh
> func_append usage, there's room here to consistently use func_append
> everywhere to make for easier maintenance... but rather than take the
> additio
Hello Charles,
* Charles Wilson wrote on Mon, Jun 28, 2010 at 12:05:40AM CEST:
> On 6/27/2010 4:43 PM, Ralf Wildenhues wrote:
> > * Charles Wilson wrote on Sun, Jun 27, 2010 at 02:51:21PM CEST:
> >> So...can I get a verdict? Is -dlpreopen not-an-la-file supposed to work
Hi Eric,
thanks for the suggestion. I had considered the idea for a second, but
failed to see the nontrivial half.
* Eric Blake wrote on Mon, Jun 28, 2010 at 02:49:40PM CEST:
> tmp=${1#?}
> patt=
> i=2
> while test $i -lt ${#1}; do
> patt="?$patt"
If the parameter may be expected to be very l
Hello Vincent,
* Ralf Wildenhues wrote on Sat, Jun 19, 2010 at 12:08:29PM CEST:
> I would like to commit the patch quoted below, on the way of support for
> LTO with GCC. For that, I need to know from somebody who is willing to
> help out with the Windows CE port of Libtool whether the
Hi Gary,
* Gary V. Vaughan wrote on Tue, Jun 29, 2010 at 05:09:29PM CEST:
> On 29 Jun 2010, at 21:03, Eric Blake wrote:
> > Ultimately, I'd like to fix m4sh to make it easier to probe/require XSI
> > support, but that will have to wait until after autoconf 2.66.
>
> While that might turn out to b
[ please elide autoconf@ from followups, thanks ]
Hi Peter,
* Peter Breitenlohner wrote on Tue, Jun 29, 2010 at 10:36:32AM CEST:
> Here the macro we are using in TeX Live for such tests (C and C++). Our
> purpose is to test properties of libraries that can be either
> (1) uninstalled libtool lib
* Gary V. Vaughan wrote on Tue, Jun 29, 2010 at 08:30:43PM CEST:
> On 30 Jun 2010, at 01:22, Ralf Wildenhues wrote:
> > I think m4sh can simply use code like
> >
> > if ( eval '$smart_works' ) >/dev/null 2>&1; then
> >func_foo () { smart c
[ moving from -commit to -patches, I think the former was for commits
only; maybe it should have reply-to set? ]
Hi Gary, Peter,
* Gary V. Vaughan wrote on Sat, Jul 03, 2010 at 10:23:12AM CEST:
> On 3 Jul 2010, at 15:15, Peter Rosin wrote:
> >Fix typo in "Add func_append_quoted and do..."
>
* Charles Wilson wrote on Sun, Jul 04, 2010 at 06:15:06AM CEST:
> On 6/26/2010 2:51 PM, Ralf Wildenhues wrote:
> > OK. Here's my take on this: if you fix all nits I noted inline below,
> > post the updated and tested patch (you decide what testing is needed),
> > then
Hi Charles,
* Charles Wilson wrote on Sat, Jul 03, 2010 at 06:10:57PM CEST:
> On 6/28/2010 2:10 AM, Ralf Wildenhues wrote:
> > * Charles Wilson wrote on Mon, Jun 28, 2010 at 12:05:40AM CEST:
> >> It obviously isn't SUPPOSED to be dead -- or it wouldn't be there.
>
Hi Peter,
* Peter Rosin wrote on Wed, Jun 30, 2010 at 09:10:02PM CEST:
> Can you please cherry-pick this commit of yours into master?
> "Fix ccache test for MSVC."
> 563c2406deb4d5ed0ebf5faf5c8b798d52ee6ac2
Done, thanks for the reminder.
> I could do it myself but I don't know what to do about t
Hi Peter,
* Peter Rosin wrote on Thu, Jul 01, 2010 at 10:30:56PM CEST:
> I.e. *.exe get expanded. That doesn't happen with "*.[eE][xX][eE])":
>
[...]
> This all seems awfully fragile. I don't know how to quote
> this expression in libtool.m4 and need help. Someone please?
Ouch. Our expansion co
Hi Peter,
any chance you could post your patches inline? Thanks.
* Peter Rosin wrote on Mon, Jul 05, 2010 at 02:45:28PM CEST:
> http://lists.gnu.org/archive/html/libtool-patches/2009-01/msg00152.html
This is ok with nits below addressed, thanks.
> Make preloading heed libname_spec.
>
e debian/gcc case actually uses the @FILE code
path in the testsuite (should be in the low_cmd_len tests)?
> 2010-07-01 Ralf Wildenhues
> Peter Rosin
>
> Support for response files with $NM.
> * libltdl/m4/libtool.m4 (_LT_CMD_GLOBAL_SYMBOLS)
> : New
Hi Bob,
* Bob Friesenhahn wrote on Tue, Jul 06, 2010 at 06:48:56AM CEST:
> On Mon, 5 Jul 2010, Bob Friesenhahn wrote:
> >Yesterday's libtool was doing quite good with the tests but I am
> >seeing plenty of failures now for all Unixish targets. Even Linux
> >blows failures all over the place. Not
Hi Peter,
* Peter Rosin wrote on Tue, Jul 06, 2010 at 11:27:31AM CEST:
> Den 2010-07-06 08:07 skrev Ralf Wildenhues:
> >Did you confirm that the debian/gcc case actually uses the @FILE code
> >path in the testsuite (should be in the low_cmd_len tests)?
>
> No I didn
Hello Thierry,
thanks for your patch and explanations.
* Thierry Reding wrote on Wed, Jul 07, 2010 at 11:51:58AM CEST:
> the following patch adds support for gcc's --sysroot argument to libtool.
> I'll post the patch first and will reply to it, giving more details where
> needed.
>
> I've been b
Hello Bob,
* Bob Friesenhahn wrote on Thu, Jul 08, 2010 at 06:34:46PM CEST:
> On Tue, 6 Jul 2010, Ralf Wildenhues wrote:
> >>
> >>Perhaps things are not all that bad. While 35 of 110 tests fail
> >>under Debian Linux, only these two extra are failing under
[ moving from -commit ]
Hi Gary,
* Gary V. Vaughan wrote on Thu, Jul 08, 2010 at 03:37:23PM CEST:
> commit 8c4dae1232958c24989f31ab5b5768d00be2ef03
> Author: Gary V. Vaughan
> Date: Thu Jul 8 18:47:59 2010 +0700
>
> Fix a spurious trailing space and a botched merge.
>
> * libltdl
Hi Peter,
* Peter Rosin wrote on Sat, Jul 10, 2010 at 10:15:09AM CEST:
> Next in to MSVC queue...
>
> This is a variation of
> 95bdbc456f7c5d6d669121cf20d62a9c28bec2bf
> "Always dllimport the variable for MSVC in link-order.at"
>
> Patch previously discussed here:
> http://lists.gnu.org/archive/
Hi Peter, Charles,
* Peter Rosin wrote on Thu, Jul 15, 2010 at 10:39:42PM CEST:
> Den 2010-07-12 07:46 skrev Ralf Wildenhues:
> >* Peter Rosin wrote on Sat, Jul 10, 2010 at 10:15:09AM CEST:
> >the patch looks fairly innocent at first sight, but your description
> >makes me
Hi Peter,
* Peter Rosin wrote on Sat, Jul 17, 2010 at 04:33:07AM CEST:
> Den 2010-07-16 05:36 skrev Ralf Wildenhues:
> >>The manual already says that it is not portable to export data items from
> >>a libtool library. At least I think it does.
> >
> >The "Li
Hello Peter,
* Peter Rosin wrote on Sun, Jul 18, 2010 at 06:45:53PM CEST:
> Next in the MSVC queue. It's getting short quickly!
>
> These two patches are commits
> ee8c181987478752a9432a903cf7febbf761d034
> dllimport when the lib is shared and not linking with -static
>
> and
>
> 5ff2e6e4e5a1f0
Hi Peter,
a little while ago:
* Peter Rosin wrote on Wed, Jul 07, 2010 at 10:15:53PM CEST:
> No problem, but I was initially very confused (and still am) by the
> fact that
>
> make check-local TESTSUITEFLAGS='-v -d -k max_cmd_len' \
> INNER_TESTSUITEFLAGS=',export -v -d export' | grep cre
Hello,
* Paolo Bonzini wrote on Wed, Jul 21, 2010 at 06:04:43PM CEST:
> On 07/21/2010 03:58 PM, Charles Wilson wrote:
> >On 7/8/2010 1:02 PM, Paolo Bonzini wrote:
> >
> >>I have another sysroot patch that seems (from a first cursory view)
> >>pretty different. I'll rebase and send.
> >
> >Ping?
>
Hi Paolo,
patch 1/9 is OK right away. Thanks!
* Paolo Bonzini wrote on Thu, Jul 29, 2010 at 01:23:15AM CEST:
> * libltdl/config/ltmain.m4sh (func_mode_link): Recognize --sysroot
> option.
This patch 2/9 is OK but please commit together with the rest.
Thanks,
Ralf
Next one: 3/9. This is a bit of a preliminary review, because I
haven't read all patches and all of the discussion yet.
* Paolo Bonzini wrote on Thu, Jul 29, 2010 at 01:23:16AM CEST:
> * libltdl/m4/libtool.m4 (_LT_HOST_NONCANONICAL, _LT_WITH_SYSROOT): New.
> (LT_SETUP): Require _LT_WITH_SYSROOT.
* Charles Wilson wrote on Sun, Aug 01, 2010 at 02:10:36PM CEST:
> On 8/1/2010 6:46 AM, Ralf Wildenhues wrote:
> > Because that allows it to set run paths to the compiler libs?
> > (I don't know either.)
>
> Hmm. If so, what use case does that capability have? You want t
* Charles Wilson wrote on Mon, Jul 26, 2010 at 01:57:57PM CEST:
> As an aside, WHY is it that libtool tries to figure out what the language
> driver
> does, and use ld directly, rather than just trust the language driver and use
> it
> to link instead? It seems that the current libtool behavior
* Paolo Bonzini wrote on Sun, Aug 01, 2010 at 04:47:26PM CEST:
> This should, and I say "should" :) pass all tests.
Thanks for the repost. Did anything change beside the discussion
attached to the last patch series post?
This passes for me on GNU/Linux with a non sysrooted GCC. How come
there i
* Charles Wilson wrote on Sun, Aug 01, 2010 at 08:03:11PM CEST:
> On 8/1/2010 10:47 AM, Paolo Bonzini wrote:
> >> 112: override pic_flag at configure time FAILED (pic_flag.at:48)
> >>
> >> 112 has been discussed before and should be
> >> skipped on non-ELF.
> >
> > Since I wasn't around, why is
* Paolo Bonzini wrote on Sun, Aug 01, 2010 at 04:47:32PM CEST:
> * libltdl/config/ltmain.m4sh (func_replace_sysroot): New.
> (func_mode_link): Prepend paths in $libdir with the sysroot. Further,
> replace the sysroot with = (using func_replace_sysroot) whenever
> such a path is written in a .la fi
* Paolo Bonzini wrote on Sun, Aug 01, 2010 at 04:47:26PM CEST:
> handle sysroot flags
> add --with-sysroot
> teach libtool -L= and -R=
> handle sysrooted paths when reading dependencies to la files
> process postdeps to include sysrooted paths
> emit sysrooted paths when installing .la
* Charles Wilson wrote on Mon, Aug 02, 2010 at 06:15:29AM CEST:
> > On 8/1/2010 2:43 PM, Ralf Wildenhues wrote:
> >> I can't really give a good review of the rest without a sysrooted
> >> compiler, and I don't have time to build one any more this weekend,
> &
* Charles Wilson wrote on Sun, Aug 01, 2010 at 10:04:16PM CEST:
> support, use --prefix=/mingw + DESTDIR=/the-sysroot.
What about the w32 users that use --prefix=C:/mingw and then cannot use
DESTDIR because that will not concatenate?
Also, in 'ln -sf' in the testsuite addition, the -f is non-port
* Charles Wilson wrote on Mon, Aug 02, 2010 at 08:00:47PM CEST:
> On 8/2/2010 1:55 PM, Ralf Wildenhues wrote:
> >What about the w32 users that use --prefix=C:/mingw and then cannot use
> >DESTDIR because that will not concatenate?
>
> According to the GNU Standards, you can
More issues:
- the part of ltmain starting with
# Collect and forward deplibs of preopened libtool libs
will source .la files but looks like it mistreats = in $dependency_libs
- is it intended that the = remain in the .la files after these are
moved to their final location on the host sy
* Charles Wilson wrote on Mon, Aug 02, 2010 at 05:23:28AM CEST:
> The "unexpected" failure was
> 112: override pic_flag at configure time FAILED (pic_flag.at:48)
> which isn't a regression on cygwin.
I'm pushing this patch to fix two issues with the test case.
Really should've known better than t
* Paolo Bonzini wrote on Mon, Aug 02, 2010 at 10:51:30PM CEST:
> On Mon, Aug 2, 2010 at 22:13, Ralf Wildenhues wrote:
>
> > - is it intended that the = remain in the .la files after these are
> > moved to their final location on the host system? Or should
> > --mode=fi
* Charles Wilson wrote on Tue, Aug 03, 2010 at 07:30:28AM CEST:
> On 8/2/2010 4:51 PM, Paolo Bonzini wrote:
> > On Mon, Aug 2, 2010 at 22:13, Ralf Wildenhues wrote:
> >> - is it intended that the = remain in the .la files after these are
> >> moved to their final locat
* Paolo Bonzini wrote on Tue, Aug 03, 2010 at 09:52:54AM CEST:
> On 08/01/2010 08:31 PM, Ralf Wildenhues wrote:
> >Although it is fairly unlikely, I prefer not relying on :+ support, so
> >please replace with 'if test -z "$lt_sysroot"...'. Two instances.
>
* Paolo Bonzini wrote on Tue, Aug 03, 2010 at 10:04:08AM CEST:
> On 08/01/2010 12:48 PM, Ralf Wildenhues wrote:
>
[ other packages that already use --with-sysroot ]
> >I'm not sure how to address these (not even sure
> >they use Libtool) except for a big sign in NEWS, and
* Peter Rosin wrote on Thu, Jul 01, 2010 at 07:38:45PM CEST:
> Den 2010-06-28 01:24 skrev Gary V. Vaughan:
> >@@ -704,15 +710,14 @@ func_mode_compile ()
> > save_ifs="$IFS"; IFS=','
> > for arg in $args; do
> > IFS="$save_ifs"
> >-func_quote_for_eval "$arg"
> >-l
Interesting shell unportability:
$ bash -c 'f=" val" e=; echo "$e"$f'
val
$ ksh -c 'f=" val" e=; echo "$e"$f'
val
ksh93, dash, zsh all do it like ksh. Is that a bug in bash?
Anyway, the patch below seems to fix it, but I may want to fix the log
entry to pinpoing the culprit correctly.
Thanks,
Hi Charles,
* Charles Wilson wrote on Tue, Aug 03, 2010 at 07:22:46AM CEST:
> For instance, suppose I have a cross compiler for mingw whose sysroot is
> /usr/i686-pc-mingw32/sys-root. Suppose I want to build the gtk2 stack,
> for win32. Well, that means I need to build gettext, and libiconv, and
* Eric Blake wrote on Tue, Aug 03, 2010 at 10:55:36PM CEST:
> On 08/03/2010 02:32 PM, Ralf Wildenhues wrote:
> > Interesting shell unportability:
> >
> > $ bash -c 'f=" val" e=; echo "$e"$f'
> > val
> > $ ksh -c 'f=" val&quo
* Rainer Tammer wrote on Wed, Aug 04, 2010 at 08:41:26AM CEST:
> FAIL: tests/nomode.test
>
> This failure is due to the following error:
>
> $ ./libtool
> ./libtool[1591]: shift: bad number
>
> -> code
> # save first non-option argument
> nonopt="$opt"; shift
>
> But there is nothing in $1,
Hi Charles,
thanks for the elaborate explanations.
* Charles Wilson wrote on Thu, Aug 05, 2010 at 08:10:10AM CEST:
> On 8/4/2010 2:15 PM, Ralf Wildenhues wrote:
> > * Charles Wilson wrote on Tue, Aug 03, 2010 at 07:22:46AM CEST:
[...]
> >> Thanks to a quirk in window'
Hello Gary,
* Gary V. Vaughan wrote on Fri, Aug 06, 2010 at 06:01:37AM CEST:
> I promised to roll a 2.2.12 release this month. But looking at the
> huge progress we've made so far with Windows since 2.2.10, I'm
> wondering if we would be better to roll 2.2.12 sooner rather than
> later?
I am sti
* Charles Wilson wrote on Fri, Aug 06, 2010 at 03:26:30AM CEST:
> But a STEP in that direction is to reduce the current pain experienced
> by cross compiles for $host=mingw even with --prefix=/unixish; pain
> caused by the current lack of sysroot support in libtool (and some other
> utilities).
>
* Charles Wilson wrote on Fri, Aug 06, 2010 at 02:47:59PM CEST:
> On 8/6/2010 1:19 AM, Ralf Wildenhues wrote:
> > Wrt. lib and lib32, lib64, can we find out all files GCC needs by asking
> > it several -print-* questions? Plus maybe optimistically searching for
> > libc and
Hello,
* Gary V. Vaughan wrote on Fri, Aug 06, 2010 at 08:53:15PM CEST:
> Another viable compromise might be to call the next release
> 2.3.0?
Maybe it's early enough to finalize a decision on this when we
know a bit better which patches are in, and which features can
be advertised.
There is als
Pushed as obvious to let 'configure CC=g++' pass the testsuite again.
Cheers,
Ralf
Fix ltdl-libdir test for C++ compilers.
* tests/ltdl-libdir.at (libdir of installed modules): Declare
module function as extern "C".
diff --git a/tests/ltdl-libdir.at b/tests/ltdl-libdir.at
index
> On 02.08.2010 20:40, Ralf Wildenhues wrote:
> > * Rainer Tammer wrote on Mon, Aug 02, 2010 at 05:40:13PM CEST:
> >> I just have tested libtool 2.2.8 on AIX 6.1.
> >> 95: AC_NO_EXECUTABLES FAILED
> >> (no-executables.at:76)
* Rainer Tammer wrote on Tue, Aug 03, 2010 at 09:40:23AM CEST:
> On 02.08.2010 20:40, Ralf Wildenhues wrote:
> > * Rainer Tammer wrote on Mon, Aug 02, 2010 at 05:40:13PM CEST:
> >> 56: versioning FAILED
> >> (versioning.at:169)
&g
Rainer, I'm going through things in some random order, and it will take
me at least a little while to get through them all.
* Rainer Tammer wrote on Thu, Aug 05, 2010 at 09:25:14AM CEST:
> 38. link-order2.at:46: testing Link order of deplibs ...
> libtool: compile: cc_r -qlanglvl=extc89 -g -c a0.
* Rainer Tammer wrote on Thu, Aug 05, 2010 at 09:25:14AM CEST:
> 112. pic_flag.at:24: testing override pic_flag at configure time ...
> ./pic_flag.at:36: $CC $CPPFLAGS $CFLAGS $C_pic_flag -c foo.c || exit 77
> stderr:
> stdout:
> ./pic_flag.at:39: grep fpic stdout stderr && exit 77
> ./pic_flag.at:
I'd like to ensure that $CONFIG_SHELL and $configure_options are used
consistently throughout the testsuite. So I'm fixing the pic_flag test
to also use LT_AT_CONFIGURE, after making that macro accept a second
argument to specify another configure script location.
Hope this is trivial enough to j
There might be no 'print' program available, and some shells (like
FreeBSD sh) redirect that stderr only when you call the program from
a subshell. I'm installing this to beautify configure output.
The second 'print' instance should be fine, and 'printf' can be assumed
to at least be present ever
* Rainer Tammer wrote on Thu, Aug 05, 2010 at 09:25:14AM CEST:
> I just refreshed my source from git and rerun the test suit on AIX 5.3.
> 75: lt_dlopenext error messages FAILED
> (lt_dlopenext.at:202)
> 76: libdir of installed modules FAILED
> (ltdl-libdi
FWIW, I didn't find any more objectionable things in the rest of this
patch series, so whenever you have something that addresses the known
issues, I think a public branch would be fine. If merely additional
testsuite items are missing, that shouldn't hold up creating a branch,
only merging it.
A
Hi Charles,
* Charles Wilson wrote on Sat, Aug 07, 2010 at 06:06:14PM CEST:
> configure:2917: checking whether the C compiler works
> configure:2939: gcc --sysroot=/tmp/sysroot conftest.c >&5
> /usr/bin/ld: this linker was not configured to use sysroots
> collect2: ld returned 1 exit status
>
Hi Gary,
* Ralf Wildenhues wrote on Mon, Jun 28, 2010 at 07:34:35AM CEST:
> * Gary V. Vaughan wrote on Mon, Jun 28, 2010 at 01:24:39AM CEST:
> > Looking through the XSI substitutions, or more correctly the bash/ksh
> > func_append usage, there's room here to consiste
* Charles Wilson wrote on Sun, Aug 08, 2010 at 10:40:56PM CEST:
> So, you suggesting *possibly* a test where:
>
> if not cross-compiling
> if (the native) ld supports --sysroot
> 1. create a fake sysroot (contents TBD?)
> 2. execute sysroot.at contents as a subtest, with
>native
* Charles Wilson wrote on Mon, Aug 09, 2010 at 12:14:37AM CEST:
> On 8/7/2010 2:36 AM, Ralf Wildenhues wrote:
> > FWIW, I didn't find any more objectionable things in the rest of this
> > patch series, so whenever you have something that addresses the known
> > issue
* Charles Wilson wrote on Mon, Aug 09, 2010 at 12:14:37AM CEST:
> I think --mode=finish needs to accept additional arguments that are not
> directories, somehow. Maybe mode=finish needs to iterate through its
> arguments, and split them into "directories" and ".la files", and then
> execute two di
* Gary V. Vaughan wrote on Sun, Aug 08, 2010 at 08:50:12AM CEST:
> On 7 Aug 2010, at 18:15, Ralf Wildenhues wrote:
> > <http://lists.gnu.org/archive/html/libtool-patches/2010-06/msg00190.html>
> I don't see that the +=/func_append substitution is causing any additiona
Hello,
* Peter O'Gorman wrote on Mon, Aug 09, 2010 at 05:38:44PM CEST:
> I don't think having a stable and development branch worked well
> with 1.5.x and 2.x. It added more work for us, and did not serve our
> users well - they had to wait years for the development branch's
> features to make it
* Ralf Wildenhues wrote on Mon, Aug 09, 2010 at 08:42:24PM CEST:
> I do think however that it's time that our testing efforts become a bit
> more coordinated, to make regression tracking easier and more apparent.
> Will reply to this post with more details.
I think Libtool should
* Rainer Tammer wrote on Thu, Aug 05, 2010 at 09:25:14AM CEST:
> The new test suite still yields many failures in comparison to 2.2.8.
I assume most if not all of the regressions are due to a single issue:
> # grep FAILED check.log
> 38: Link order of deplibs FAILED
> (
* Ralf Wildenhues wrote on Mon, Aug 09, 2010 at 10:55:25PM CEST:
> Note the double colon after -blibpath, which is due to a regression in
> v2.2.10-40-g0da7d80 fixed with the patch below. I wish I'd never
> approved that patch, this one took me so long to find ...
I did a re-review
* Gary V. Vaughan wrote on Tue, Aug 10, 2010 at 11:09:33AM CEST:
> On 10 Aug 2010, at 15:19, Peter Rosin wrote:
> > m4_defun([AM_PROG_AR],
> > [AC_CHECK_TOOLS(AR, [ar], false)
> > : ${AR=ar}
> > AC_SUBST([AR])dnl
> > ])# AM_PROG_AR
> > ])
>
> Just like that!
NAK. I don't have time to explain now
* Ralf Wildenhues wrote on Tue, Aug 10, 2010 at 11:15:21AM CEST:
> * Gary V. Vaughan wrote on Tue, Aug 10, 2010 at 11:09:33AM CEST:
> > On 10 Aug 2010, at 15:19, Peter Rosin wrote:
> > > m4_defun([AM_PROG_AR],
> > > [AC_CHECK_TOOLS(AR, [ar], false)
> > >
Hi Peter,
* Peter Rosin wrote on Thu, Aug 12, 2010 at 03:28:57PM CEST:
> This is a patch that makes use of @FILE support in the
> archiver, if the archiver supports it. That makes linking
> succeed for long command lines when using MSVC, as MSVC
> can't do piecewise linking (-r -o) which is the cu
* Paolo Bonzini wrote on Thu, Aug 12, 2010 at 06:12:15AM CEST:
> Hi, I pushed the first 8 sysroot patches to refs/heads/sysroot on
> Savannah.
Thanks. So far, with a merge of sysroot to master, I see one regression
on AIX with runtimelinking (configure LDFLAGS=-Wl,-brtl) and a different
one on So
* Paolo Bonzini wrote on Fri, Aug 13, 2010 at 03:23:03AM CEST:
> * tests/sysroot.at: Link $prefix/include into the sysroot as well, as
> required by newer GCC.
Obviously.
Thanks!
Ralf
* Paolo Bonzini wrote on Fri, Aug 13, 2010 at 03:23:04AM CEST:
> * libltdl/config/ltmain.m4sh: Pass $dependency_libs of preopened libtool
> through func_resolve_sysroot.
This looks ok, but is it covered in the testsuite?
(If not, I can try to do it later, but at least would like to be sure we
make
* Paolo Bonzini wrote on Fri, Aug 13, 2010 at 03:23:05AM CEST:
> * libltdl/config/ltmain.m4sh (func_mode_finish): Split arguments for
> directories and *.la files. Add .la files for all directories.
OK if you fix the log entry. IIUC the last sentence is wrong.
Thanks,
Ralf
* Paolo Bonzini wrote on Fri, Aug 13, 2010 at 03:23:05AM CEST:
> * libltdl/config/ltmain.m4sh (func_mode_finish): Split arguments for
> directories and *.la files. Add .la files for all directories.
Ah, there's another nit.
> --- a/libltdl/config/ltmain.m4sh
> +++ b/libltdl/config/ltmain.m4sh
>
* Paolo Bonzini wrote on Fri, Aug 13, 2010 at 03:23:06AM CEST:
> ---
Erm, you should be mentioning the files you change in your log entries,
and Libtool also doesn't auto-generate its ChangeLog (yet), so you need
that, too, also for all the other patches in the sysroot branch.
Sorry for not notic
* Christopher Hulbert wrote on Fri, Aug 13, 2010 at 01:45:37PM CEST:
> Is there interest (besides mine) in libtool being able to use native
> Windows compilers from cygwin. I just find cygwin easier to
> use/maintain than mingw/msys.
>
> In the branch I had been working on, I defined the libtool v
* Ralf Wildenhues wrote on Thu, Aug 12, 2010 at 11:28:31PM CEST:
> Thanks. So far, with a merge of sysroot to master, I see one regression
> on AIX with runtimelinking (configure LDFLAGS=-Wl,-brtl)
OK so this one really is a regression with the patch series. bisect
converges at the
* Ralf Wildenhues wrote on Thu, Aug 12, 2010 at 11:28:31PM CEST:
> Thanks. So far, with a merge of sysroot to master, I see one regression
> on AIX with runtimelinking (configure LDFLAGS=-Wl,-brtl) and a different
> one on Solaris,
The Solaris issue is shown below and is actual
eneral.m4sh (sed_make_literal_regex): New.
> * libltdl/config/ltmain.m4sh (func_generate_dlsyms,
FWIW, I think GCS style is
* file-name (function-name)
(function-name, ...):
i.e., close and reopen parens at EOL.
> func_cygming_dll_for_implib_fallback_core): Use it. Suggested
> by Ralf Wildenhues.
* Paolo Bonzini wrote on Sun, Aug 15, 2010 at 06:57:18PM CEST:
> Here are the improvements you suggested. I'll send a patch for
> sed_make_literal_regex soon.
>
> Ok?
Yes, with the nit addressed. Thanks!
Ralf
> * libltdl/config/ltmain.m4sh (func_mode_finish): Change sysroot_regex
> and sysroot
* Paolo Bonzini wrote on Sun, Aug 15, 2010 at 03:46:06PM CEST:
> On 08/13/2010 12:59 AM, Ralf Wildenhues wrote:
> >Erm, you should be mentioning the files you change in your log entries,
> >and Libtool also doesn't auto-generate its ChangeLog (yet), so you need
> >that, t
Hi Charles, Paolo,
* Charles Wilson wrote on Mon, Aug 16, 2010 at 05:12:03PM CEST:
> On 8/12/2010 9:23 PM, Paolo Bonzini wrote:
> > These cover bullets 3/5 of Charles' list:
> >
> > 3) issue with '# Collect and forward deplibs of preopened libtool libs'
> >In progress? Anything?
> > 4,5)
> >
* Charles Wilson wrote on Mon, Aug 16, 2010 at 10:27:46PM CEST:
> Taking all of this together, I wonder if a good approach might be the
> following: libltdl replaces the '=' with the value of the environment
> variable LIBLTDL_SYSROOT. So usually -- with this variable empty -- '='
> is just stripp
* Charles Wilson wrote on Mon, Aug 16, 2010 at 10:50:21PM CEST:
> On 8/16/2010 4:42 PM, Ralf Wildenhues wrote:
> >
> > Can you run
> > VERBOSE=yes $srcdir/tests/fcdemo-conf.test; echo $?
> >
> > and post the output of that? VERBOSE=debug is even more info
* Paolo Bonzini wrote on Sun, Aug 01, 2010 at 04:47:29PM CEST:
> * libltdl/config/ltmain.m4sh (func_resolve_sysroot): New.
> (func_mode_link): Always pass result of stripping -L and -R to
> func_resolve_sysroot before using it. Remove absolute path expansion
> when func_resolve_sysroot subsumes it
* Ralf Wildenhues wrote on Thu, Aug 19, 2010 at 06:58:31AM CEST:
> * Paolo Bonzini wrote on Sun, Aug 01, 2010 at 04:47:29PM CEST:
> > +# func_resolve_sysroot PATH
> > +# Change PATH to an absolute path or replace a leading = with a
> > +# sysroot. Store the result into func_r
AM, Ralf Wildenhues wrote:
> >Why does this function absolutize paths that have nothing to do with
> >sysroot, even in-tree deplib paths?
> >Is there a deeper reason in this? If yes, there should probably be a
> >test which is fixed by it.
>
> It's a bug, I think
This moves the ToC to the beginning in the PDF output, and IMHO looks
so much nicer. Will push soon unless somebody complains.
Thanks Bruno!
Ralf
Don't hide the table of contents in the manual.
* doc/libtool.texi: Move the table of contents to the beginning.
Prompted by patch fr
Pushed as obvious: there is nothing that would generate demo-conf.test
and I couldn't find any reason for the test result to depend on the
libtool script in particular.
Cheers,
Ralf
Remove superfluous dependency in Makefile.am.
* Makefile.am (tests/demo-conf.test): Remove dependency
Pushed as obvious.
Bump copyright years in manual.
* doc/libtool.texi: Bump copyright years.
diff --git a/doc/libtool.texi b/doc/libtool.texi
index f508015..d0eaec4 100644
--- a/doc/libtool.texi
+++ b/doc/libtool.texi
@@ -26,7 +26,7 @@
@ifnottex
This file documents GNU Libtool @val
I tried running the Libtool test suites with old Autoconf 2.59 and
Automake 1.9.6 (on GNU/Linux only) after 'autoreconf -fvi'-ing all
but the toplevel directory with those tools. I was surprised to see
that it still works more or less! :-)
This patch fixes the fall-out. OK to push if the use of
* Ralf Wildenhues wrote on Fri, Aug 20, 2010 at 06:34:58AM CEST:
> * libltdl/config/ltmain.m4sh (func_resolve_sysroot): Partly
> revert v2.2.10-83-gc45a288: Do not absolutize path here, only do
> sysroot replacement.
> (func_mode_link): Adjust.
> Fixes AIX testsuite regression.
* Ralf Wildenhues wrote on Sat, Aug 07, 2010 at 12:30:01AM CEST:
> The test tries to ensure that with an incompatible library upgrade,
> programs previously linked against the old library still work. This
> usually works on systems where different major versions of a library
> have di
* Rainer Tammer wrote on Sat, Aug 07, 2010 at 09:35:56AM CEST:
> I have added one additional patch (as I did for the last run):
>
> # diff /daten/source/libtool-2.2.11a/tests/defs.in defs.in
[...]
> < # On AIX, shared libraries remain loaded in memory after use if they
> < # are world-readable, un
1 - 100 of 2537 matches
Mail list logo