Apparently, I am too stupid to operate grep, so I overlooked one
instance of oline in libtool.m4 in commit db160ae47c1.
I am applying this obvious followup.
Sorry about that,
Ralf
2009-12-12 Ralf Wildenhues ralf.wildenh...@gmx.de
Replace last __oline__ instance.
* libltdl/m4
[ let's drop automake@ ]
Hi Paolo,
* bonz...@gnu.org wrote on Tue, Dec 22, 2009 at 09:16:59AM CET:
Here is where I was at. After that it was not immediate how to
use a tag-dependent cache variable. Strictly speaking however
using a cache variable is not needed to make the PIC test
Follow-up Comment #1, patch #7055 (project libtool):
The test is written as intended. However, there is code in libtool (from
ltmain.sh) that removes -lc from link command lines for $host matching
*-*-mingw*. So, let's find out why that apparently didn't work in your case.
Please show the
Hi Peter,
* Peter Rosin wrote on Sat, Jan 02, 2010 at 04:02:57AM CET:
Please consider the attached patch.
I'm just about to go skiing for a week or so, so I'll push when I get
back if this patch is blessed (knock wood) after I leave...
Well happy new year and hope you had a fun time in the
Follow-up Comment #3, patch #7055 (project libtool):
If you want cross compilation, enable it with
./configure --host=i586-mingw32msvc
not with
./configure CC=i586-mingw32msvc-gcc
That will fix half of your problems. There is still another
one in your native build: there, -lc should be
Follow-up Comment #6, patch #7055 (project libtool):
OK. You are using an *ancient* version of Autoconf, presumably 2.13, but it
must be from before 2000. Please update to a non-ancient version first;
libtool requires 2.59 or newer, but preferably you should use 2.65.
Hi Eric,
* Eric Blake wrote on Wed, Jan 06, 2010 at 02:22:58PM CET:
According to Ralf Wildenhues on 1/6/2010 12:08 AM:
Pushed to master.
Bump copyright years.
GNU Coding Standards permit bumping the copyright on all files in the
project, not just those that have had significant
Follow-up Comment #8, patch #7055 (project libtool):
Your verbose configure output contains lines like
+ nonopt=NONE
which have been removed from Autoconf in 2000. So, unless they come from
your configure.ac file, this configure script has been generated with an
ancient autoconf version. Try
Follow-up Comment #11, patch #7055 (project libtool):
Thanks for the link. The bug seems to be in your configure.ac:
# This is required to get past a stupid configure bug when making the rpm.
# Basically it is broken to specify the host as a command line argument to
# configure on its own,
[ moving from libtool@ ]
* Bob Friesenhahn wrote on Tue, Jan 05, 2010 at 05:11:56PM CET:
Using 'llvm-gcc' (GCC frontend to llvm compiler) I find that C++
exceptions do not work (are not caught) in the built programs.
[ and suspect libtool as culprit ]
We can only find out better if we have
* Ralf Wildenhues wrote on Tue, Dec 22, 2009 at 10:45:18PM CET:
* bonz...@gnu.org wrote on Tue, Dec 22, 2009 at 09:16:59AM CET:
Here is where I was at. After that it was not immediate how to
use a tag-dependent cache variable. Strictly speaking however
using a cache variable is not needed
Hi Bob,
* Bob Friesenhahn wrote on Sat, Jan 09, 2010 at 08:03:11PM CET:
On Sat, 9 Jan 2010, Ralf Wildenhues wrote:
We can only find out better if we have small examples to look at.
Here is a proposed patch to try exception handling in libraries and
modules. Tested with g++ on GNU/Linux
Hi Bob,
* Bob Friesenhahn wrote on Mon, Jan 11, 2010 at 05:32:15AM CET:
On Sun, 10 Jan 2010, Ralf Wildenhues wrote:
My main comment is that it would be useful if the thrown class is
derived from std:exception (or one of its standard derived classes)
in order to flush out any issues which
Hi Charles,
* Charles Wilson wrote on Tue, Jan 12, 2010 at 12:42:21AM CET:
Ralf Wildenhues wrote:
Yup, indeed. And we need to provide a throw()-qualified destructor as
well.
A word of warning: not all implementations of std::exception use the
'throw' specifier on the virtual what
Hi Bob,
* Bob Friesenhahn wrote on Tue, Jan 12, 2010 at 05:26:27PM CET:
On Tue, 12 Jan 2010, Ralf Wildenhues wrote:
The test is currently skipped if the compiler doesn't like main.cpp
(which already exposes this API), so we should be safe there but not
test on as many systems as we could. I
Hi Peter,
* Peter Rosin wrote on Tue, Jan 12, 2010 at 11:00:17AM CET:
Without the patch, a separate thread of the process could be relying on
the error mode to have some specific flag. Then, during a small window
the loadlibrary loader clears some vital error mode flag since it has
to set a
* Charles Wilson wrote on Mon, Jan 11, 2010 at 03:54:01AM CET:
Ralf Wildenhues wrote:
That's not how these ancient shells work.
[snip long explanation]
Oh, that's just...evil. How could that EVER have been expected to work
properly, except in the most trivial of scripts?
Good question
Hi Peter,
* Peter Rosin wrote on Sun, Jan 17, 2010 at 09:19:27PM CET:
Den 2010-01-17 17:45 skrev Bob Friesenhahn:
On Sun, 17 Jan 2010, Ralf Wildenhues wrote:
I'm not too fond of adding new static storage and manipulation (yet
another reason lt_dlopen may not be called concurrently from
Hi Peter,
* Peter Rosin wrote on Fri, Jan 22, 2010 at 10:12:11AM CET:
As mentioned in another mail, should not lt_dlloader_add
call vtable-dlloader_init? That seems appropriate when
lt_dlloader_remove calls vtable-dlloade_exit...
Like this (completely untested), just showing what I mean...
* Ralf Wildenhues wrote on Sun, Jan 24, 2010 at 10:36:54AM CET:
* Peter Rosin wrote on Fri, Jan 22, 2010 at 10:12:11AM CET:
As mentioned in another mail, should not lt_dlloader_add
call vtable-dlloader_init? That seems appropriate when
lt_dlloader_remove calls vtable-dlloade_exit
Hi Peter,
* Peter Rosin wrote on Fri, Jan 22, 2010 at 09:53:40AM CET:
Recently (when I wrote the testcase for error reporting
in the loadlibrary loader) I found that the documentation
for the dlloader section of the api was a wee bit lacking
in accuracy...
Oh yes it is. Oh brother.
So, I
Hi Peter,
* Peter Rosin wrote on Mon, Jan 25, 2010 at 04:12:47PM CET:
As promised, here's a testsuite addition for the dlloader API.
Thank you!
2010-01-25 Peter Rosin p...@lysator.liu.se
Testsuite exposure for dlloader API.
* tests/dlloader-api.at: New file, new test.
* Peter Rosin wrote on Thu, Jan 28, 2010 at 09:06:10AM CET:
Den 2010-01-28 07:16 skrev Ralf Wildenhues:
There are several more instances that fail when using CC=g++. The patch
below makes it pass in this case for me, please look over it that I
haven't made any mistakes.
Ouch, didn't
the magic word
Applied to master as obvious.
Cheers,
Ralf
Make testsuite code C++ clean again.
* tests/resident.at (resident modules): Fix for C++.
diff --git a/tests/resident.at b/tests/resident.at
index 8667ca7..c16a72a 100644
--- a/tests/resident.at
+++ b/tests/resident.at
@@ -59,7 +59,7 @@
Following the recent recommendation on gnu-prog-discuss, I've applied
this.
Cheers,
Ralf
Use --email with gendocs.sh.
* Makefile.maint (web-manual): Pass bug reporting address to
gendocs.sh.
diff --git a/Makefile.maint b/Makefile.maint
index 02284df..b90b31d 100644
---
* Charles Wilson wrote on Sun, Feb 14, 2010 at 03:36:57PM CET:
Charles Wilson wrote:
Charles Wilson wrote:
Charles Wilson wrote:
This one, I think is OK for pre-2.2.8 -- what do you guys think?
OK to push?
In response to review comments over here:
Re: [PATCH] Enable runtime cwrapper
Hi Charles,
* Charles Wilson wrote on Wed, Feb 17, 2010 at 03:30:47PM CET:
Ralf Wildenhues wrote:
The option parsing in the original patch is overkill -- no need to
re-quote things if all you're going to do is remove a couple of entries
from $@, that can be done with
set x $@
shift
Hi Scott,
* scott mc wrote on Wed, Feb 17, 2010 at 09:14:55PM CET:
Haiku has changed their legacy /boot/beos/system directory to be just
/boot/system, this patch makes that change in libtool.m4
Thanks for the patch. What about users of older systems, where the
change has not been done yet?
[ dropping libtool@ ]
Hi Pierre,
thanks for the report and patch!
* Pierre Ossman wrote on Mon, Feb 15, 2010 at 11:01:40AM CET:
The normal check was fixed some time ago, but the fallback code was
overlooked. This patch fixes that as well.
So that means you've encountered it with some code,
Hi Charles,
* Charles Wilson wrote on Fri, Feb 19, 2010 at 03:48:40AM CET:
Enable runtime cwrapper debugging; add tests
* libltdl/config/ltmain.m4sh (func_emit_cwrapperexe_src):
Update comments. Initialize program_name. Eliminate _LENGTH
variables for string constants.
Hi Charles,
* Charles Wilson wrote on Sun, Feb 21, 2010 at 07:19:53AM CET:
Document wrapper changes.
* NEWS: Indicate new feature and incompatibility.
* doc/libtool.texi [Linking executables]: Mention wrapper
executables, in addition to wrapper scripts. Add menu
Hello,
* Charles Wilson wrote on Sun, Feb 21, 2010 at 07:31:15AM CET:
[cygwin|mingw] Create UAC manifest files.
* libltdl/config/ltmain.m4sh (func_emit_exe_manifest): New
function.
(func_mode_link) [cygwin|mingw]: Create manifest files for
wrapper and target
* Charles Wilson wrote on Sun, Feb 21, 2010 at 06:34:02AM CET:
So, open issues, to be addressed if necessary in additional patches:
1) (func) ... --- '(%s) ..., __func__' ?
2) chmod in cwrapper.at?
Naah, ignore both. Thanks.
strerror(errno) can return NULL for unknown errors on some systems.
Hello Pierre,
* Pierre Ossman wrote on Mon, Feb 15, 2010 at 10:57:23AM CET:
Let's give this another go. I've included a patch against master this
time.
Thanks for your patch and your perseverance.
For a reminder, the problem is that libltdl mangles library names for
the preload table, but
Hi Charles,
* Charles Wilson wrote on Sun, Feb 21, 2010 at 10:09:43PM CET:
Ralf Wildenhues wrote:
If the
latter, then I don't see how the manifest file ever gets to be installed
at `make install' time, nor uninstalled at `make uninstall' time?
It was a deliberate choice -- a shortcut
* Charles Wilson wrote on Sun, Feb 21, 2010 at 09:09:37PM CET:
Ralf Wildenhues wrote:
Don't you need to repeat the menu entry somewhere in the toplevel
@detailmenu?
Didn't know about detailmenu. Fixed.
This has me stumped. The entry was added to the wrong menu, but
makeinfo did
* Pierre Ossman wrote on Mon, Feb 22, 2010 at 11:13:41AM CET:
On Sat, 20 Feb 2010 10:32:27 +0100 Ralf Wildenhues wrote:
* Pierre Ossman wrote on Mon, Feb 15, 2010 at 11:01:40AM CET:
The normal check was fixed some time ago, but the fallback code was
overlooked. This patch fixes
Hello Charles, Yaakov,
thanks for the patch.
* Charles Wilson wrote on Mon, Feb 22, 2010 at 03:39:05PM CET:
Original post went astray somehow...
2009-12-30 Yaakov Selkowitz ...
Pass various runtime library flags to GCC.
* libltdl/config/ltmain.m4sh (func_mode_link): Pass the
* scott mc wrote on Tue, Feb 23, 2010 at 10:09:22AM CET:
On Sat, Feb 20, 2010 at 1:07 AM, Ralf Wildenhues wrote:
* scott mc wrote on Wed, Feb 17, 2010 at 09:14:55PM CET:
Haiku has changed their legacy /boot/beos/system directory to be just
/boot/system, this patch makes that change
libtool.m4 change is included and fail
without. With a cross compiler and wine, it skips, native it should
fail, but I can't test that ATM.
Thanks,
Ralf
2010-02-24 Pierre Ossman oss...@ossman.lkpg.cendio.se (tiny change)
Ralf Wildenhues ralf.wildenh...@gmx.de
Fix deplibs check
* Pierre Ossman wrote on Thu, Feb 25, 2010 at 11:21:54AM CET:
On Wed, 24 Feb 2010 22:31:07 +0100 Ralf Wildenhues wrote:
I managed to send out a broken version of the patch. Please forget that
one, this one has a testcase that should work better. Can you retry
with it? You can use
Hello Werner,
* Werner Koch wrote on Fri, Feb 26, 2010 at 09:54:01AM CET:
I am currently porting GnuPG to Windows CE and would like to see the
attached patch integrated. My company (g10 Code) signed a CA.
Thanks for the report and patch. I've just committed another patch that
should make
This issue is complex enough that it warrants some explanation.
We should help our users on that front.
I'm not the w32 expert in any measure, so here's a chunk of texinfo
text that I've just run down now, without much integration, and by
stealing some of the explanations of others.
I was going
Hello Chris,
thanks for the report and patch.
* Chris Demetriou wrote on Mon, Mar 15, 2010 at 08:12:31PM CET:
A project that I work on wants to make sure our builds are
deterministic, i.e., same input sources - same exact output binaries.
We've solved several problems (e.g., I've added the
* Chris Demetriou wrote on Tue, Mar 16, 2010 at 08:28:12AM CET:
On Mon, Mar 15, 2010 at 23:37, Ralf Wildenhues wrote:
libtool also uses 'ar' in a number of places and cases, most
prominently, but not limited to, the static linking scenario.
Do you have measures in place to use 'D' there too
Hello Peter,
* Peter Rosin wrote on Mon, Mar 15, 2010 at 11:55:16AM CET:
2010-01-21 Peter Rosin p...@lysator.liu.se
Use Get/SetThreadErrorMode if they are available.
* libltdl/loaders/loadlibrary.c (wrap_geterrormode): Replaced...
(wrap_getthreaderrormode): ...by this
Hello Ari,
thanks for the report and patch.
* Ari Entlich wrote on Mon, Mar 08, 2010 at 05:58:59AM CET:
The subject says it all. Comments appreciated.
A very minor nit: mails are easier to grok if the contents don't refer
to the subject. Patches are a lot easier to understand with a
* Peter Rosin wrote on Wed, Mar 17, 2010 at 10:50:40AM CET:
Hmm, my last argument made me think that it's perhaps not wise for
libtool to rely on the last error to not change over a successful
call either. But that potential problem was there before this patch
too, so I'm proposing a separate
* Pete Batard wrote on Sun, Mar 14, 2010 at 11:30:19PM CET:
On 2010.03.14 07:47, Xiaofan Chen wrote:
On Sat, Mar 13, 2010 at 6:31 PM, Ralf Wildenhues wrote:
1) Would it also be sufficient to just
LC_ALL=C; LANGUAGE=C; export LANGUAGE LC_ALL
This is enough to solve the issue.
or do you
* Bob Friesenhahn wrote on Tue, Mar 16, 2010 at 10:12:07PM CET:
On Tue, 16 Mar 2010, Ralf Wildenhues wrote:
Yes, that may be it. However, that also means that, while the patch
fixes things for you, it doesn't really add value to Libtool in the
sense that we cannot guarantee an improvement
[ adding libtool-patches, in case anyone wants to disagree ]
* Karl Berry wrote on Wed, Mar 17, 2010 at 10:50:38PM CET:
I'm okay with changing the m4 and autoconf manuals with
s/@acronym{GNU}/GNU/
s/@acronym{(.*)}/\1/
So the preferred color now is this, I guess:
Hi Dave,
thanks for your quick review! I've incorporated all changes not
addressed below.
* Dave Korn wrote on Tue, Mar 16, 2010 at 11:24:42AM CET:
On 16/03/2010 06:17, Ralf Wildenhues wrote:
and updating of system-wide software. The level of privileges required
for some executable
Hello Simon, Török, Åke,
* Simon Richter wrote on Wed, Mar 24, 2010 at 12:11:13PM CET:
I'm trying to use link time optimization with gcc 4.5, which somewhat
works :), but requires the compiler flags to be passed to the linking
step as well. libtool however removes them from the command line.
* Karl Berry wrote on Fri, Mar 19, 2010 at 10:42:14PM CET:
Is the intention that even the n...@acronym{gnu} cases should be
replaced? Then what purpose is the @acronym keyword for?
I wrote about that earlier. Minor typographic change which is rarely
used in GNU manuals.
Proposed
I'll be pushing this trivial patch.
Thanks,
Ralf
Fix typo in libtool --help output.
* libltdl/config/ltmain.m4sh (func_mode_help): Fix typo in help
output.
diff --git a/libltdl/config/ltmain.m4sh b/libltdl/config/ltmain.m4sh
index 9790579..cdc23a5 100644
---
Working on support for -flto has uncovered an ugly buglet in libltdl.
ltmain declares this symbol as:
extern $lt_dlsym_const lt_dlsymlist
lt_${my_prefix}_LTX_preloaded_symbols[];
but libltdl declares it as:
#define preloaded_symbols LT_CONC3(lt_, LTDLOPEN,
_LTX_preloaded_symbols)
Hello Pierre,
* Pierre Ossman wrote on Mon, Feb 22, 2010 at 04:57:56PM CET:
On Sun, 21 Feb 2010 16:41:31 +0100 Ralf Wildenhues wrote:
* Pierre Ossman wrote on Mon, Feb 15, 2010 at 10:57:23AM CET:
For a reminder, the problem is that libltdl mangles library names for
the preload table
[ dropped libtool@ ]
* Ralf Wildenhues wrote on Sun, Mar 28, 2010 at 05:33:44PM CEST:
Is there any harm in passing them to the linking step?
Yes. Let me answer with a patch to add a FAQ and FAQ entry.
OK to commit?
No more feedback, I've thus pushed this patch ...
Add FAQ, and FAQ
Hello,
I've pushed this patch to fix a fairly obvious glitch.
Thanks,
Ralf
Use linker_flags not compiler_flags with $LD.
* libltdl/m4/libtool.m4 (_LT_LINKER_SHLIBS) [linux]
archive_cmds, archive_expsym_cmds: With xlf and bgxlf
compilers, when linking with $LD, use
* Ralf Wildenhues wrote on Sun, Mar 28, 2010 at 05:33:44PM CEST:
I'm still working on a patch to support -flto in libtool, it needs
changes in several places.
I have a preliminary(!) patch set here for LTO support in libtool.
DO NOT submit this to your favorite distribution, it is not ready
100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,20 @@
2010-04-04 Ralf Wildenhues ralf.wildenh...@gmx.de
+ Support GCC LTO on GNU/Linux.
+ * NEWS: Update.
+ * libltdl/config/ltmain.m4sh (func_mode_link): Allow through
+ flags matching -O*, -flto*, -fwhopr, -fuse-linker
.
(lt_preloaded_symbols, main): Adjust.
* tests/pdemo/longer_file_name_dlmain.c: Likewise.
Signed-off-by: Ralf Wildenhues ralf.wildenh...@gmx.de
---
Hello,
Contrary to popular belief, of these five,
A)
struct lt_symlist {
const char *name;
void *address;
};
B)
struct lt_dlsymlist {
const
(LTDL_SET_PRELOADED_SYMBOLS): Likewise.
Signed-off-by: Ralf Wildenhues ralf.wildenh...@gmx.de
---
Hello,
this patch addresses the const-ness of the symlist variable.
I would like things to be consistent across ltmain, libtool.m4 (where we test
symbol-pipe functionality), and ltdl.h, and since the latter can't
.
[hpux10, hpux11, !hppa*64]: Replace -fPIC with $pic_flag.
[irix5, irix6]: Add $pic_flag if $GXX.
[osf4, osf5]: Likewise.
[solaris]: Add $pic_flag if $GXX and not GCC 2.7.
Signed-off-by: Ralf Wildenhues ralf.wildenh...@gmx.de
---
Hello,
This untested patch cleans up a number of hard-coded -fPIC
* Török Edwin wrote on Sun, Apr 04, 2010 at 06:03:35PM CEST:
On 04/04/2010 11:54 AM, Ralf Wildenhues wrote:
I have a preliminary(!) patch set here for LTO support in libtool.
Thanks a lot for the patches, they worked fine so far!
Thanks for testing.
I'd have one more request: please add
* Ralf Wildenhues wrote on Tue, May 04, 2010 at 07:56:12PM CEST:
I'll apply the patch below soon unless I hear complaints.
Based on the very mixed feedback for this patch, let's drop it.
Thanks everyone for feedback tho,
Ralf
Reword alternate versioning algorithm a bit.
* doc
[ moving from libtool@ ]
Hi Gary,
* Gary V. Vaughan wrote on Fri, May 21, 2010 at 02:22:09AM CEST:
The Libtool Team is pleased to announce release candidate 2.2.7b of GNU
Libtool. If there are no serious deficiencies reported in this release,
it will be renumbered as 2.2.8 and re-released
* Gary V. Vaughan wrote on Sat, May 22, 2010 at 11:21:37AM CEST:
On 22 May 2010, at 16:04, Ralf Wildenhues wrote:
As a veeery minor nit may I note that with git, it is common to separate
the first line of the commit entry from the rest, in order for 'git log
--oneline' to work as expected
Hi Gary,
* Gary V. Vaughan wrote on Sat, Jun 05, 2010 at 11:08:47AM CEST:
Itchy trigger finger. I meant to add:
Okay to push?
Well, the M4SH_GETOPTS part certainly is rather hard to read because of
those long lines and long indents. Other than that, I don't actually
use either of those
This mirrors a similar recent fix to automake.texi.
Any technical reasons against this patch?
The rest of the manual greps ok.
Thanks,
Ralf
Rewrite manual intro to be gender-neutral.
* doc/libtool.texi (Introduction): Use gender-neutral
formulation when addressing developers.
Hi Gary,
* Gary V. Vaughan wrote on Fri, Jun 04, 2010 at 07:50:31PM CEST:
Provide an m4sh reimplementation of announce-gen.
* libltdl/config/getopt.m4sh (M4SH_GETOPTS): New macro that takes
a quoted m4 list of command line options to be parsed, and
generates the shell
Hi Gary, Eric,
* Gary V. Vaughan wrote on Sun, Jun 06, 2010 at 02:13:35PM CEST:
On Jun 6, 2010, at 6:38 PM, Ralf Wildenhues wrote:
I see the point in the factorization of the option parsing, and I have
to admit to not having tested or even looked in detail at these
changes
yet
Hello Bob,
* Bob Friesenhahn wrote on Sun, Jun 06, 2010 at 05:15:11PM CEST:
On Sun, 6 Jun 2010, Ralf Wildenhues wrote:
This mirrors a similar recent fix to automake.texi.
Any technical reasons against this patch?
The rest of the manual greps ok.
Regardless of Gary's affirmation, I don't
Hello Ryan,
* Ryan Hill wrote on Sun, Jun 06, 2010 at 12:24:54PM CEST:
On Sun, 6 Jun 2010 11:44:59 +0200 Ralf Wildenhues wrote:
Thanks for the bug report. To facilitate analysis, please show the
output of:
cd tests/testsuite.dir/056
objdump -p inst/bin/* inst/lib/*.so
Here you
I get this warning from recent Autoconf when bootstrapping Libtool:
$ ./bootstrap
[...]
autom4te --language=m4sh -B libltdl/config libltdl/config/ltmain.m4sh
libltdl/config/ltmain.in
libltdl/config/getopt.m4sh:1: warning: prefer named diversions
The patch below fixes it. OK?
Thanks,
Ralf
* Ralf Wildenhues wrote on Mon, Jun 07, 2010 at 10:42:42PM CEST:
Fix versioning test for LDFLAGS=-Wl,--as-needed.
As a followup, I'm fixing the test for w32 systems with this obvious
patch.
Cheers,
Ralf
Make versioning test stricter for w32, enable shared libs.
* tests
, Inc.
+# Copyright (C) 2005, 2007, 2008, 2009, 2010 Free Software Foundation,
+# Inc.
# Written by Ralf Wildenhues, 2005
#
# This file is part of GNU Libtool.
@@ -227,7 +228,7 @@ AT_CLEANUP
AT_SETUP([Java convenience archives])
-LT_AT_TAG([GCJ])
+LT_AT_EXEC_TAG([GCJ])
AT_KEYWORDS
Hi Peter,
* Peter O'Gorman wrote on Wed, Jun 09, 2010 at 03:40:32AM CEST:
The above is, of course, after tests/demo-nopic.test, and the problem is
the selinux boolean allow_execmod which disallows text relocations.
I'll look into making the test skip in this case.
Ok?
This skips a bit
Software Foundation, Inc.
+# Copyright (C) 2008, 2009, 2010 Free Software Foundation, Inc.
# Written by Ralf Wildenhues, 2008
#
# This file is part of GNU Libtool.
@@ -193,19 +193,19 @@ do
fi
AT_CHECK([$LIBTOOL --mode=execute ./foo abc $arg1 $arg2 xyz], [],
[stdout], [ignore
* Peter Rosin wrote on Wed, Jun 09, 2010 at 09:00:06AM CEST:
Den 2010-06-09 07:14 skrev Ralf Wildenhues:
I would like to see more of the story here. If this is still
reproducible for you, post the output of
make check-local TESTSUITEFLAGS='-v -d -x 77'
It is very reproducible, even
Hi Gary,
* Gary V. Vaughan wrote on Thu, Jun 10, 2010 at 05:18:08PM CEST:
Okay to push?
Some of the $ECHO use ($ECHO $opt) now became plain echo, that might
be a problem with backslashes or leading single hyphens. Otherwise, it
looks to me like with
--ltdl=foo --ltdl=bar
the old version
* Peter Rosin wrote on Thu, Jun 10, 2010 at 12:10:34AM CEST:
Den 2010-06-09 20:30 skrev Ralf Wildenhues:
Sorry for not being clear. I really meant running this in the toplevel
build directory:
make install installcheck TESTSUITEFLAGS='-v -d -x 75'
But since you could reproduce, that's
Hi Peter,
* Peter O'Gorman wrote on Fri, Jun 11, 2010 at 05:04:55AM CEST:
+ Create reloadable object files with non-pic objects too.
+ * libltdl/config/ltmain.m4sh: When not building a shared
+ library, use the non-pic objects to create a reloadable
+ object, because pic
Hi Gary,
* Gary V. Vaughan wrote on Fri, Jun 11, 2010 at 07:22:52PM CEST:
Okay to push?
* libltdl/config/ltmain.m4sh: Replace hand written shell code
with a call to M4SH_GETOPTS. Rename some option variables in
the client code to match the generated option parser settings.
*
Committed to master after testing on MinGW native and GNU/Linux native.
Cheers,
Ralf
2010-06-12 Peter Rosin p...@lysator.liu.se
Ralf Wildenhues ralf.wildenh...@gmx.de
Improve libdir usage from within ltdl on MinGW.
* libltdl/ltdl.c (parse_dotla_file) [__WINDOWS__
* Gary V. Vaughan wrote on Sat, Jun 12, 2010 at 12:34:04PM CEST:
On 12 Jun 2010, at 15:48, Ralf Wildenhues wrote:
The variable preserve_args is not properly initialized any more,
so it could pick up junk from the environment.
Okay. I did that to avoid the initial space, otherwise
Hi Peter,
* Peter O'Gorman wrote on Sun, Jun 13, 2010 at 06:07:44AM CEST:
On 06/11/2010 11:56 PM, Gary V. Vaughan wrote:
1: There might be a better way. I'm thinking...
I'm not sure what testing that modules cannot be unloaded gets you
when you don't have shared libraries.
That may be
OK to commit this patch if this one is accepted into Autoconf?
http://lists.gnu.org/archive/html/autoconf-patches/2010-06/msg00031.html
The move the testsuite.at code above AT_INIT should have no impact
beside reordering things in the PREPARE_TESTS diversion a bit, which
should be harmless (I
Is Autoconf = 2.64 spread widely enough for Libtool development by now?
If not, that could probably be hacked around in configure.ac, but I'd
need to test with old Autoconf then.
Anyway, this updates --help output of libtool, libtoolize, and the
testsuite script to match current GNU Coding
* Ralf Wildenhues wrote on Sun, Jun 13, 2010 at 01:41:47PM CEST:
Print Libtool project URL in program --help output.
* configure.ac (AC_PREREQ): Require Autoconf 2.64 for Libtool.
(AC_INIT): Set URL argument.
* Makefile.am (edit): Substitute PACKAGE_URL.
($(srcdir
Updated patch. This one doesn't require 2.64 any more, it just sets
PACKAGE_URL itself with older Autoconfs, and it changes the PACKAGE_NAME
to GNU Libtool. The GNU prefix lets Autoconf = 2.64 compute the
right gnu.org web page automatically.
This:
* Ralf Wildenhues wrote on Sun, Jun 13, 2010
Hi Peter,
* Peter O'Gorman wrote on Mon, Jun 14, 2010 at 05:18:55AM CEST:
On 06/13/2010 12:07 AM, Ralf Wildenhues wrote:
* Peter O'Gorman wrote on Sun, Jun 13, 2010 at 06:07:44AM CEST:
[ the resident test ]
I'm not sure what testing that modules cannot be unloaded gets you
when you don't have
Hi Peter,
* Peter Rosin wrote on Mon, Jun 14, 2010 at 01:26:04PM CEST:
Here's the next patch on the MSVC branch. I decided to merge in
a couple of things further ahead, no point in reviewing stuff
when an improvement is around the corner, right?
this patch is ok for master, provided that it
* Charles Wilson wrote on Sun, Jun 13, 2010 at 08:51:00PM CEST:
On 6/12/2010 4:58 AM, Ralf Wildenhues wrote:
* Charles Wilson wrote on Fri, Jun 11, 2010 at 02:28:41PM CEST:
In 48, the problem occurs during libtool --clean:
/usr/src/packages/libtool/git/build/libtool: line 1693:
sub3
Ping! I would at least like a comment upon this, why it would be too
dangerous, and all that.
http://thread.gmane.org/gmane.comp.sysutils.automake.general/11229/focus=9626
(No, this patch is unrelated to the -flto patches I just pinged.)
Thank you,
Ralf
* Ralf Wildenhues wrote on Sat, Jan 09
* Bob Friesenhahn wrote on Sun, Jun 13, 2010 at 06:16:15PM CEST:
On Sun, 13 Jun 2010, Ralf Wildenhues wrote:
Tested with git Autoconf and 2.62. OK?
This change looks fine to me.
Thanks for the review. I had applied this, but it turns out it wasn't
complete after all. The bootstrap script
Hello Philip,
thanks for the report and patch.
* Philip Allison wrote on Tue, Jun 15, 2010 at 01:15:14PM CEST:
The test program for whether a program can dlopen itself generates a
compiler warning, which turns into a failure when building with -Wall
-Werror in CFLAGS.
Please note that in
The function func_ltwrapper_executable_p is used right before
func_ltwrapper_scriptname a couple of times in ltmain; the latter again
calls the former. This costs an extra grep. The only time we don't
call func_ltwrapper_executable_p first is on a just-created wrapper
executable.
So this should
Hi Charles,
* Charles Wilson wrote on Wed, Jun 16, 2010 at 05:17:31AM CEST:
On 6/15/2010 3:28 PM, Ralf Wildenhues wrote:
So this should be safe. And the documentation is there already before
the function, too.
Any reasons against committing this?
No reasons. Looks good to me (but I
1301 - 1400 of 1734 matches
Mail list logo
