[patch #9341] testsuite.at: prefer diff --strip-trailing-cr

Follow-up Comment #1, patch #9341 (project libtool): [copying my msg from the ML to the patch tracker] I think these testsuite problems goes away if you configure with something like ".../configure --build=i686-pc-cygwin --host=mingw32" as indicated in the manual [1]. Or maybe it's

Re: [patch #9341] testsuite.at: prefer diff --strip-trailing-cr

On 2017-05-10 16:24, Michael Haubenwallner wrote: > URL: > > > Summary: testsuite.at: prefer diff --strip-trailing-cr > Project: GNU Libtool > Submitted by: haubi > Submitted on: Wed 10 May 2017

Re: [PATCH 3/4] Use POSIX nm to simplify AIX export_symbols_cmds.

On 2016-03-14 11:01, Michael Haubenwallner wrote: > On 03/12/2016 12:13 AM, Peter Rosin wrote: >> On 2016-03-11 22:22, Michael Haubenwallner wrote: >>> Hi Peter, >>> >>> thanks for looking at the patch! >>> >>> On 03/10/2016 12:29 PM, Pete

Re: [PATCH 3/4] Use POSIX nm to simplify AIX export_symbols_cmds.

Hi Michael, I had a look since I wrote a patch for POSIX nm a couple of years ago that I never submitted (I didn't see any use case) which looked very similar, excepting the AIX-ism in your version. On 2016-03-10 10:01, Michael Haubenwallner wrote: > * m4/libtool.m4 (LT_PATH_NM): Detect

Re: [PATCH 2/2] libtool: set file_list_spec to '@' on OS/2

Hi! On 2016-02-22 12:21, Pavel Raiskup wrote: > KO Myung-Hun, thanks for this patch but I need to see a bit deeper > reasoning for this change as I do not understand the code properly. > > As the $file_list_spec is used in libtool under special circumstances, can > you describe what are the

Re: [PATCH] Support MSYS2

On 2015-08-09 21:26, fabrizio...@tiscali.it wrote: The output of uname -o is indeed the same in MSYS2 and MinGW/MSYS Msys However, that is irrelevant because, config.guess does not use uname -o, it does uname -s UNAME_SYSTEM=`(uname -s) 2/dev/null` || UNAME_SYSTEM=unknown The

Re: [sr #108558] libtool nm test does not really work for W32 versions of nm

On 2014-05-06 15:30, Peter Rosin wrote: Follow-up Comment #8, sr #108558 (project libtool): I have attached a patch that does the safe-safe-safe version. Does it work for you? *snip* http://savannah.gnu.org/support/?108558 To not force everyone to follow the link, here's the patch

Re: [SCM] GNU Libtool branch, master, updated. v2.4.2.444-2-g581d90b

On 2014-01-03 02:01, Gary V. Vaughan wrote: Joking aside, you're quite right, I'll shuffle the order of bootstrap.in to leave that warning at the top, and remove the write bit from the generated file. Thanks for fixing that! (too) Cheers, Peter

Re: [SCM] GNU Libtool branch, master, updated. v2.4.2.444-2-g581d90b

On 2014-01-02 01:32, Gary V. Vaughan wrote: Peter's a7462c5 fix was applied to the generated bootstrap script instead of the funclib.sh source, and had have been overwritten the next time bootstrap was regenerated. Nice catch! As my feeble defense, I claim that there should have

Re: bootstrap breakage starting with fec7d87

pushing it... Cheers, Peter From a7462c5563e124e06f4f61ce2a9cea26cf8be390 Mon Sep 17 00:00:00 2001 From: Peter Rosin p...@lysator.liu.se Date: Tue, 19 Nov 2013 11:54:27 +0100 Subject: [PATCH] bootstrap: fix version sort Reported by Ozkan Sezer who suffered from makeinfo 4.13 being detected as lesser

Re: g++ and -nostdlib

[re-adding libtool@] On 2013-11-11 16:10, Bob Friesenhahn wrote: On Mon, 11 Nov 2013, Peter Rosin wrote: Quite a lot of effort went into making this work the way it currently does. I realize that, but if it really works or not is a different question :-) Yes, of course. It is obviously

Re: g++ and -nostdlib

From 56080c9349ae56fcac6e7bf7f6081dfa48a7fc67 Mon Sep 17 00:00:00 2001 From: Peter Rosin p...@lysator.liu.se Date: Mon, 11 Nov 2013 09:48:23 +0100 Subject: [PATCH] libtool: Do not use -nostdlib when linking with g++. Fixes part of bug#15646 and Debian bug 468555. * m4/libtool.m4

[PATCH] libtool: update cached $GCC value when updating $GXX

Hi! I noticed this while looking at the -nostdlib stuff. Will push in a couple of days unless there are valid objections. Cheers, Peter From 7efe9b28d977fccded55843c8bee3458835d0435 Mon Sep 17 00:00:00 2001 From: Peter Rosin p...@lysator.liu.se Date: Mon, 11 Nov 2013 10:00:28 +0100 Subject

Re: [PATCH] libtool: update cached $GCC value when updating $GXX

On 2013-11-11 10:37, Peter Rosin wrote: Will push in a couple of days unless there are valid objections. Forget it. I am a moron. It would be more valid to simply remove the GXX=no assignment, but I can't classify that as a bug. Sorry for wasting your bandwidth. Cheers, Peter

g++ and -nostdlib

? Or simply a bug fix? BTW, it even appears to work, but please test, as I have only done very light testing... Oh well. Any thoughts? Cheers, Peter From a233b4562d4274053852bc0353e36931beeb4803 Mon Sep 17 00:00:00 2001 From: Peter Rosin p...@lysator.liu.se Date: Fri, 8 Nov 2013 19:01:24 +0100

Re: Two-month patch ping: Re: powerpc*le-linux support

On 2013-08-22 09:40, Gary V. Vaughan wrote: Can we please get this simple patch pushed? Done. To me, it appears as if what you actually pushed was not what was posted? Cheers, Peter

Re: powerpc*le-linux support

Hi! Ok, Gary pushed something while I wrote this, but I'm sending it anyway since what he pushed didn't look quite right to me... On 2013-06-06 07:18, Alan Modra wrote: On Thu, Jun 06, 2013 at 11:31:34AM +0930, Alan Modra wrote: This adds support for little-endian powerpc linux, and tidies the

Re: powerpc*le-linux support

On 2013-08-22 10:19, Gary V. Vaughan wrote: On Aug 22, 2013, at 2:59 PM, Peter Rosin p...@lysator.liu.se wrote: On 2013-06-06 07:18, Alan Modra wrote: *snip* diff --git a/m4/libtool.m4 b/m4/libtool.m4 index d7013c5..501246d 100644 --- a/m4/libtool.m4 +++ b/m4/libtool.m4 @@ -1307,7 +1307,7

Re: powerpc*le-linux support

On 2013-08-22 15:25, Alan Modra wrote: On Thu, Aug 22, 2013 at 01:16:04PM +0200, Peter Rosin wrote: I guess I'm just thoroughly confused, but in my world there ought to be four variations of $host; 64- or 32-bit, and big or little endian. This patch seems to only handle builds going from 64

Re: Two-month patch ping: Re: powerpc*le-linux support

On 2013-08-22 10:20, Gary V. Vaughan wrote: Hi Peter, On Aug 22, 2013, at 2:54 PM, Peter Rosin p...@lysator.liu.se wrote: On 2013-08-22 09:40, Gary V. Vaughan wrote: Can we please get this simple patch pushed? Done. To me, it appears as if what you actually pushed was not what

Re: [PATCH] Fix conversion warnings in cwrapper

On 2013-05-27 21:21, Yaakov (Cygwin/X) wrote: On 2013-05-27 11:28, Peter Rosin wrote: Ok, I took the liberty of writing a ChangeLog and removed the above mentioned lines, as well as changing one unsigned int cast to a size_t cast, when figured I should double-check your email-address

Re: [PATCH] Fix conversion warnings in cwrapper

Hi Yaakov, On 2013-05-21 08:53, Peter Rosin wrote: I have no problem with this patch from a cursory look (haven't tested it yet), but I will wait a couple of days with committing it to see if Chuck (or someone else for that matter) has something to add. Meanwhile, could we please have

Re: [PATCH] Fix conversion warnings in cwrapper

On 2013-05-21 00:49, Yaakov (Cygwin/X) wrote: From: Yaakov Selkowitz yselkow...@users.sourceforge.net Signed-off-by: Yaakov Selkowitz yselkow...@users.sourceforge.net --- build-aux/ltmain.in | 18 +- 1 files changed, 9 insertions(+), 9 deletions(-) diff --git

Re: [PATCH] libtool: LT_PATH_NM: default to ${ac_tool_prefix}nm

On 2013-04-28 09:21, Peter Rosin wrote: On 2013-04-27 22:38, Mike Frysinger wrote: On Saturday 27 April 2013 13:53:28 Peter Rosin wrote: On 2013-04-27 07:58, Mike Frysinger wrote: The current code tries to locate a compatible nm tool. It starts with a prefixed nm tool (great!) and includes

Re: [PATCH] libtool: LT_PATH_NM: default to ${ac_tool_prefix}nm

On 2013-04-29 04:45, Mike Frysinger wrote: On Sunday 28 April 2013 03:21:15 Peter Rosin wrote: And on re-reading, my IFS changes are not very constructive. I removed those. I will push the attached in a couple of days, if there are no objections. i actually thought your IFS changes made

Re: [PATCH] libtool: LT_PATH_NM: default to ${ac_tool_prefix}nm

On 2013-04-29 17:55, Mike Frysinger wrote: On Monday 29 April 2013 02:55:12 Peter Rosin wrote: On 2013-04-29 04:45, Mike Frysinger wrote: On Sunday 28 April 2013 03:21:15 Peter Rosin wrote: And on re-reading, my IFS changes are not very constructive. I removed those. I will push the attached

Re: [PATCH] libtool: LT_PATH_NM: default to ${ac_tool_prefix}nm

On 2013-04-27 22:38, Mike Frysinger wrote: On Saturday 27 April 2013 13:53:28 Peter Rosin wrote: On 2013-04-27 07:58, Mike Frysinger wrote: The current code tries to locate a compatible nm tool. It starts with a prefixed nm tool (great!) and includes a plain nm too (that's fine). The problem

Re: bug#13462: error: file 'build-aux/funclib.sh' not found

On 2013-01-18 10:46, Peter Rosin wrote: On 2013-01-16 18:00, Peter Rosin wrote: Hi! If I, in a dirty git checkout (added an insignificant newline) as of today, do this ./bootstrap -fc cd .. mkdir foo cd foo ../libtool/configure make make check all

Re: bug#13414: Valid DLL def file mangled by libtool

close 13414 thanks Hi! On 2013-01-22 19:45, Erik van Pienbroek wrote: Erik van Pienbroek schreef op zo 20-01-2013 om 15:54 [+0100]: Peter Rosin schreef op zo 20-01-2013 om 08:32 [+0100]: But I will hold off the push a couple of days pending feedback from those actually using .def files

Re: bug#13414: Valid DLL def file mangled by libtool

On 2013-01-19 06:12, Gary V. Vaughan wrote: Hi Peter, Thanks for working on this. On 19 Jan 2013, at 05:55, Peter Rosin p...@lysator.liu.se wrote: On 2013-01-12 01:26, Peter Rosin wrote: On 2013-01-11 12:34, Martin Doucha wrote: I'd like to report a bug in libtool 2.4 (including

Re: bug#13462: error: file 'build-aux/funclib.sh' not found

On 2013-01-16 18:00, Peter Rosin wrote: Hi! If I, in a dirty git checkout (added an insignificant newline) as of today, do this ./bootstrap -fc cd .. mkdir foo cd foo ../libtool/configure make make check all is fine. If I then modify build

Re: bug#13414: Valid DLL def file mangled by libtool

Hi Martin, Erik, On 2013-01-12 01:26, Peter Rosin wrote: Hi Martin, Thanks for the report. On 2013-01-11 12:34, Martin Doucha wrote: Hi, I'd like to report a bug in libtool 2.4 (including the latest git revision) which mangles valid DLL def files under MinGW and makes the linker barf

[PATCH 1/2] libtool: allow tabs in $cmds variables

. Signed-off-by: Peter Rosin p...@lysator.liu.se --- build-aux/ltmain.in |8 ++- m4/libtool.m4 | 170 +- 2 files changed, 91 insertions(+), 87 deletions(-) diff --git a/build-aux/ltmain.in b/build-aux/ltmain.in index c8cdb9c..eb224e3

[PATCH 2/2] libtool: factor out the dll .def file test and improve it

: Update. THANKS: Update. Signed-off-by: Peter Rosin p...@lysator.liu.se # Conflicts: # # m4/libtool.m4 --- Makefile.am |1 + NEWS|3 + THANKS |2 + build-aux/ltmain.in | 19 +++- m4/libtool.m4 | 31 --- tests/export-def.at

Re: [PATCH v4 2/2] git-version-gen: add --fallback option to use if git is not present

Hi Eric, Thanks for the push! On 2012-12-31 23:45, Eric Blake wrote: On 12/28/2012 03:13 PM, Peter Rosin wrote: When building in a git checkout, but from a system lacking git, it is useful to fall back to the version determined when the git checkout was last used from a system sporting git

[no subject]

Sure thing, I also rebased them while at it... ...but forgot the script-version. v4 coming up. Sorry for the noise. Cheers, Peter

[PATCH v4 1/2] maint.mk: handle missing git with more grace

* top/maint.mk (no-submodule-changes, public-submodule-commit): Quietly proceed if git is not present. Copyright-paperwork-exempt: yes Signed-off-by: Peter Rosin p...@lysator.liu.se --- top/maint.mk |6 -- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/top/maint.mk b/top

[PATCH v4 2/2] git-version-gen: add --fallback option to use if git is not present

there is no $tarball_version_file and git is not working. (scriptversion): Update. Copyright-paperwork-exempt: yes Signed-off-by: Peter Rosin p...@lysator.liu.se --- build-aux/git-version-gen |9 +++-- 1 files changed, 7 insertions(+), 2 deletions(-) diff --git a/build-aux/git-version-gen b/build-aux/git

[PATCH v3 2/2] git-version-gen: add --fallback option to use if git is not present

there is no $tarball_version_file and git is not working. (scriptversion): Update. Copyright-paperwork-exempt: yes Signed-off-by: Peter Rosin p...@lysator.liu.se --- build-aux/git-version-gen |9 +++-- 1 files changed, 7 insertions(+), 2 deletions(-) diff --git a/build-aux/git-version-gen b/build-aux/git

Re: git-version-gen w/o git

On 2012-12-28 02:35, Paul Eggert wrote: On 12/27/2012 03:41 PM, Peter Rosin wrote: If it helps I can regenerate with your redirection fix, but I assume whoever commits it can fix that part easily enough. Just let me know. How about if you do that, and we give Eric and/or others a week

Re: git-version-gen w/o git

Sure thing, I also rebased them while at it... ...but forgot the script-version. v4 coming up. Sorry for the noise. But sent the wrong patch anyway and also omitted the subject. *blush* v5 coming up. Cheers, Peter

[PATCH v5 1/2] maint.mk: handle missing git with more grace

* top/maint.mk (no-submodule-changes, public-submodule-commit): Quietly proceed if git is not present. Copyright-paperwork-exempt: yes Signed-off-by: Peter Rosin p...@lysator.liu.se --- top/maint.mk |6 -- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/top/maint.mk b/top

[PATCH v5 2/2] git-version-gen: add --fallback option to use if git is not present

there is no $tarball_version_file and git is not working. (scriptversion): Update. Copyright-paperwork-exempt: yes Signed-off-by: Peter Rosin p...@lysator.liu.se --- build-aux/git-version-gen |9 +++-- 1 files changed, 7 insertions(+), 2 deletions(-) diff --git a/build-aux/git-version-gen b/build-aux/git

Re: git-version-gen w/o git

On 2012-12-24 03:05, Paul Eggert wrote: On 11/29/2012 11:12 PM, Peter Rosin wrote: This seems to be stalled because of a misconception. Can I please get a second opinion? Please? I'm afraid I don't understand the problem well enough to offer an opinion -- maybe it's because I don't

Re: git-version-gen w/o git

On 2012-11-13 15:32, Peter Rosin wrote: On 2012-10-18 15:56, Peter Rosin wrote: Hi Eric! On 2012-10-18 15:02, Eric Blake wrote: [adding-gnulib] I'm not subscribed, please (continue to) keep me in CC. On 10/18/2012 06:50 AM, Peter Rosin wrote: Hi! I used to use a libtool git checkout

Preloading import libraries with MSVC

Hi! As discussed earlier[1], there are problems with preloading import libraries when using Microsoft Visual C/C++. This series is a polished version of the previously posted rough cut. However, the first two patches are largely unrelated to the topic, but they help visualize what the meat in the

[PATCH 2/3] libtool: unify the global symbol transformations

expression. * m4/libtool.m4 (_LT_CMD_GLOBAL_SYMBOLS): Unify the matching expressions in the sed programs that transform the extracted symbol lines. Signed-off-by: Peter Rosin p...@lysator.liu.se --- m4/libtool.m4 | 14 +++--- 1 files changed, 7 insertions(+), 7 deletions(-) diff --git a/m4

[PATCH 1/3] libtool: break up long lines

* m4/libtool.m4 (_LT_CMD_GLOBAL_SYMBOLS): Break up long lines when assigning the sed scripts that transform the extracted symbol lines. Signed-off-by: Peter Rosin p...@lysator.liu.se --- m4/libtool.m4 | 16 +--- 1 files changed, 13 insertions(+), 3 deletions(-) diff --git a/m4

[PATCH 3/3] libtool: add @INIT@ to the preloader, for data imports on Windows

(add_symlist): Look for the virtual @INIT@ symbol (i.e. lt_syminit) and call it. (vm_sym): Step past the @INIT@ symbol, if present. * tests/demo.at (dlmain.c): Call the @INIT@ symbol, if present. * NEWS: Update. Signed-off-by: Peter Rosin p...@lysator.liu.se --- NEWS |2

[PATCH 1/2] maint.mk: handle missing git with more grace

* top/maint.mk (no-submodule-changes, public-submodule-commit): Quietly proceed if git is not present. Copyright-paperwork-exempt: yes Signed-off-by: Peter Rosin p...@lysator.liu.se --- top/maint.mk |6 -- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/top/maint.mk b/top

[PATCH 2/2] git-version-gen: add --fallback option to use if git is not present

there is no $tarball_version_file and git is not working. (scriptversion): Update. Copyright-paperwork-exempt: yes Signed-off-by: Peter Rosin p...@lysator.liu.se --- build-aux/git-version-gen |9 +++-- 1 files changed, 7 insertions(+), 2 deletions(-) diff --git a/build-aux/git-version-gen b/build-aux/git

Re: [RFC] Fix libltdl preloader issues with MSVC

Hi Gary, On 2012-10-20 07:26, Gary V. Vaughan wrote: Hi Peter, On Oct 20, 2012, at 4:06 AM, Peter Rosin p...@lysator.liu.se wrote: With the latest dumpbin work in place, libtool starts to add symbols from import libraries to the generated fooS.c files in func_generate_dlsyms, as needed

[RFC] Fix libltdl preloader issues with MSVC

Cheers, Peter From 3e840b158f7a9949457273ec070f35ab03b04ade Mon Sep 17 00:00:00 2001 From: Peter Rosin p...@lysator.liu.se Date: Fri, 19 Oct 2012 22:49:56 +0200 Subject: [PATCH] libtool: add @INIT@ to the preloader, for data imports on Windows * build-aux/ltmain.in

libtool: fix spelling nit

Hi! I have pushed the below patch. Cheers, Peter From cfcb7afd26a2c41f3dae62766002cac570417c77 Mon Sep 17 00:00:00 2001 From: Peter Rosin p...@lysator.liu.se Date: Thu, 18 Oct 2012 14:27:10 +0200 Subject: [PATCH] libtool: fix spelling nit * build-aux/ltmain.in (func_generate_dlsyms): Fix

git-version-gen w/o git

Hi! I used to use a libtool git checkout from a platform that lacks git [MSYS], but that broke at some point. I would like something like the below to unbreak my work flow. Please? Cheers, Peter diff --git a/Makefile.am b/Makefile.am index 176325c..3bcb419 100644 --- a/Makefile.am +++

Re: git-version-gen w/o git

[dropping-gnulib for this part] On 2012-10-18 15:02, Eric Blake wrote: [adding-gnulib] On 10/18/2012 06:50 AM, Peter Rosin wrote: @@ -128,8 +128,10 @@ $(ltversion_m4): $(ltversion_in) $(dotversion) done; \ if $$rebuild; then \ rm -f '$@'; \ - if test -f '$(srcdir

Re: Windows Line Endings

On 2012-10-08 23:29, Roumen Petrov wrote: Peter Rosin wrote: Hi Roumen, On 2012-10-07 11:37, Roumen Petrov wrote: And now test fail in cross environment : linux for mingw host Thanks for the report! I have pushed this. Let me know if it doesn't help. No comment. Thank you, I'm assuming

Re: [PATCH] tests: skip with-pic test when no real pic flag is used.

On 2012-10-07 11:59, Roumen Petrov wrote: Peter Rosin wrote: On 2012-09-19 23:02, Roumen Petrov wrote: What about to skip test only if DLL_EXPORT is in pic_flag ? Since the patch has already been pushed and it is only a test that may be skipped on some other platforms, I'm going to wait

Re: Windows Line Endings

On 2012-10-06 06:32, Gary V. Vaughan wrote: I'm about to push these changes. Please let me know whether it fixes your fortran test failures -- It'll take me a few more days to find the time to build a Windows test environment. The fortran tests are fine now. Thanks! Cheers, Peter

Re: Windows Line Endings

On 2012-10-06 10:13, Gary V. Vaughan wrote: On 6 Oct 2012, at 14:05, Gary V. Vaughan g...@gnu.org wrote: Scratch that, on closer inspection there are a couple of thinkos and portable quoting errors in the original port of quote.test to Autotest. I'll push if you can confirm that it fixes the

Re: Windows Line Endings

On 2012-10-06 07:58, Gary V. Vaughan wrote: Hi Peter, On 6 Oct 2012, at 06:20, Peter Rosin p...@lysator.liu.se wrote: Over to mdemo.at. It fails on MinGW because I do not have any installed libltdl in MinGW. Temporarily moving the installed Cygwin libltdl out of the lib search path makes

[PATCH] tests: use dry runs in both parts of 'check link mode operation'

Hi! I'm pushing this as obvious as this was how link-2.test used to do it. Cheers, Peter From 82791b3fb7043f81391bb36047f8533f4dd11b7b Mon Sep 17 00:00:00 2001 From: Peter Rosin p...@lysator.liu.se Date: Sun, 7 Oct 2012 00:57:10 +0200 Subject: [PATCH] tests: use dry runs in both parts

Re: [SCM] GNU Libtool branch, master, updated. v2.4.2-273-ge24f183

On 2012-10-05 13:15, Gary V. Vaughan wrote: This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project GNU Libtool. ARRRGH! I assume this is what you referred to when you talked about some stuff

Re: [SCM] GNU Libtool branch, master, updated. v2.4.2-273-ge24f183

The above looks just like a classic Windows failure. I.e. a line ending mismatch between the expout file created by the posixy shell (\n) and the Win32 program (\r\n) and I would guess that this is a problem that caused failures for Chuck last year as well. I think you need to use

Re: Windows Line Endings [WAS Re: [SCM] GNU Libtool branch, master, updated. v2.4.2-273-ge24f183]

/MinGW would be plenty enough. On 5 Oct 2012, at 22:21, Peter Rosin p...@lysator.liu.se wrote: I think you need to use LT_AT_HOST_DATA instead of AT_DATA, where appropriate. Unfortunately, the below is not enough since LT_AT_HOST_DATA eats whitespace ( foo - foo) which affects depdemo.at

Re: Windows Line Endings

On 2012-10-05 22:01, Peter Rosin wrote: The remaining fallout is libtool.at, mdemo.at, f77demo.at and fcdemo.at. I'll see what I can dig out... My guess is that f77demo.at and fcdemo.at fails because of different stdio streams in their mixed mode programs. Tests 159 and 162 (static library

Re: [PATCH] tests: feed -no-undefined when linking libtool libraries

On 2012-10-03 05:45, Gary V. Vaughan wrote: Hi Peter, On Oct 3, 2012, at 12:32 AM, Peter Rosin p...@lysator.liu.se wrote: [snipped loads of stuff] [snipped a bit more stuff] You're saying that you are about to: $ git checkout master $ git merge --strategy=recursive -X theirs gary

Re: [PATCH] tests: feed -no-undefined when linking libtool libraries

[snipped loads of stuff] On 2012-10-02 16:54, Gary V. Vaughan wrote: The new branch gary/reredo-test-operand-order (notice the double re) has everything broken down into digestible chunks. All the differences between that and master now look like improvements upon the original hand rolled

Re: [PATCH] tests: feed -no-undefined when linking libtool libraries

On 2012-09-22 05:31, Gary V. Vaughan wrote: Hi Peter, On 19 Sep 2012, at 21:50, Peter Rosin p...@lysator.liu.se wrote: On 2012-09-19 16:20, Gary V. Vaughan wrote: Hi Peter, My bad, I'm embarrassed to say. I started to write a script to make the appropriate changes, but ended up doing

Re: [PATCH] tests: feed -no-undefined when linking libtool libraries

On 2012-09-26 14:57, Peter Rosin wrote: On 2012-09-22 05:31, Gary V. Vaughan wrote: [Heavily snipped] Why not commit the sc_prohibit_test_const_follows_var improvement last, after fixing all violations? That doesn't make sense, sorry. But the idea would have worked initially, before

[PATCH] tests: don't feed -no-undefined to the linker during, configure.

* tests/deplibs-mingw.at: Restore LDFLAGS for the configure run so that the linker does not see -no-undefined. Makes the test pass instead of skip on MinGW. Signed-off-by: Peter Rosin p...@lysator.liu.se --- tests/deplibs-mingw.at |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) Ok

[PATCH] tests: feed -no-undefined when linking libtool libraries

* tests/runpath-in-lalib.at: Make sure shared libraries are created on Windows by passing -no-undefined. Otherwise libb.la fails to record a dependency on liba.la, and the final link of the program then fails with undefined symbols. Signed-off-by: Peter Rosin p...@lysator.liu.se --- tests

[PATCH] tests: skip with-pic test when no real pic flag is used.

* tests/with-pic.at: Windows uses -DDLL_EXPORT -DPIC as the pic flag, but never applies it to static libraries. Cater for this and skip if no real pic flag is in use. Signed-off-by: Peter Rosin p...@lysator.liu.se --- tests/with-pic.at | 11 ++- 1 files changed, 10 insertions(+), 1

Re: [PATCH] tests: feed -no-undefined when linking libtool libraries

On 2012-09-19 09:31, Peter Rosin wrote: * tests/runpath-in-lalib.at: Make sure shared libraries are created on Windows by passing -no-undefined. Otherwise libb.la fails to record a dependency on liba.la, and the final link of the program then fails with undefined symbols. Signed-off

Re: [PATCH] tests: skip with-pic test when no real pic flag is used.

On 2012-09-19 11:20, Gary V. Vaughan wrote: Hi Peter, On 19 ก.ย. 2012, at 15:56, Peter Rosin p...@lysator.liu.se wrote: * tests/with-pic.at: Windows uses -DDLL_EXPORT -DPIC as the pic flag, but never applies it to static libraries. Cater for this and skip if no real pic flag is in use

Re: [PATCH] tests: feed -no-undefined when linking libtool libraries

On 2012-09-19 11:26, Peter Rosin wrote: On 2012-09-19 09:31, Peter Rosin wrote: * tests/runpath-in-lalib.at: Make sure shared libraries are created on Windows by passing -no-undefined. Otherwise libb.la fails to record a dependency on liba.la, and the final link of the program then fails

Re: [PATCH] tests: feed -no-undefined when linking libtool libraries

easy to be wise after the fact... If you do redo it, may I suggest breaking up the patch in smaller pieces for bisectability? On 19 ก.ย. 2012, at 19:27, Peter Rosin p...@lysator.liu.se wrote: On 2012-09-19 11:26, Peter Rosin wrote: On 2012-09-19 09:31, Peter Rosin wrote: * tests/runpath

Re: [PATCH] tests: don't feed -no-undefined to the linker during, configure.

On 2012-09-19 21:25, Roumen Petrov wrote: Peter Rosin wrote: * tests/deplibs-mingw.at: Restore LDFLAGS for the configure run so that the linker does not see -no-undefined. Makes the test pass instead of skip on MinGW. Signed-off-by: Peter Rosin p...@lysator.liu.se --- tests/deplibs

Re: [PATCH] tests: skip with-pic test when no real pic flag is used.

On 2012-09-19 21:43, Roumen Petrov wrote: Peter Rosin wrote: * tests/with-pic.at: Windows uses -DDLL_EXPORT -DPIC as the pic flag, but never applies it to static libraries. Cater for this and skip if no real pic flag is in use. I'm not sure that this test is suitable for mingw host

Re: [PATCH] tests: feed -no-undefined when linking libtool libraries

On 2012-09-19 21:32, Roumen Petrov wrote: Peter Rosin wrote: On 2012-09-19 09:31, Peter Rosin wrote: * tests/runpath-in-lalib.at: Make sure shared libraries are created on Windows by passing -no-undefined. Otherwise libb.la fails to record a dependency on liba.la, and the final link

Re: [PATCH] tests: skip with-pic test when no real pic flag is used.

On 2012-09-19 23:02, Roumen Petrov wrote: Peter Rosin wrote: On 2012-09-19 21:43, Roumen Petrov wrote: Peter Rosin wrote: * tests/with-pic.at: Windows uses -DDLL_EXPORT -DPIC as the pic flag, but never applies it to static libraries. Cater for this and skip if no real pic flag is in use

Re: libtool quoting error

On 2012-08-19 22:18, Peter Rosin wrote: Oops, forgot a couple of backslashes, trying again. Sorry for the noise. Testsuite running, just in case... The patch does not seem to affect the testsuite, so I'll push in a few days. Unless someone speaks up against it of course. Cheers, Peter

Re: libtool quoting error

On 2012-08-20 16:14, Bob Friesenhahn wrote: On Mon, 20 Aug 2012, Peter Rosin wrote: On 2012-08-19 22:18, Peter Rosin wrote: Oops, forgot a couple of backslashes, trying again. Sorry for the noise. Testsuite running, just in case... The patch does not seem to affect the testsuite, so

Re: libtool quoting error

2001 From: Peter Rosin p...@lysator.liu.se Date: Sun, 19 Aug 2012 22:06:06 +0200 Subject: [PATCH] libtool: quote progpath properly Attempt to handle spaces in paths better. * build-aux/ltmain.m4sh (func_mode_install, func_mode_link): Quote $progpath. * THANKS: Update. --- THANKS

Re: libtool quoting error

On 2012-08-19 22:09, Peter Rosin wrote: [Cygwinners: Taking this to the Libtool lists] [Libtoolers: Following up on a post on the cygwin mailing list] On 2012-08-19 19:03, Andreas Schiffler wrote: The libtool distributed with cygwin has a bug that prevents use in paths containing spaces

Re: restore EXPORT check

at the symbols when it's _not_ already a .def file (in which case we trust the user input blindly). Copyright-paperwork-exempt: Yes Signed-off-by: Peter Rosin p...@lysator.liu.se

[PATCH] cwrapper: avoid duplicate strlen calculation.

a2e87f5bfed174de7d715dcf5ca5b51da4761e75 Mon Sep 17 00:00:00 2001 From: Peter Rosin p...@lysator.liu.se Date: Mon, 30 Jan 2012 15:26:13 +0100 Subject: [PATCH] cwrapper: avoid duplicate strlen calculation. * build-aux/ltmain.m4sh (func_emit_cwrapperexe_src:lt_update_exe_path): Remove duplicate strlen calculation. Signed-off

Re: [PATCH] cwrapper: avoid duplicate strlen calculation.

[Sorry for replying to myself] Peter Rosin skrev 2012-01-30 15:32: Hi! This is very close to obvious and I very nearly just pushed it out, but what's the rush? However, I will push under the 72h rule if noone speaks up. It fixes half the issues reported by the clang static analyzer

Re: [PATCH] cwrapper: avoid duplicate strlen calculation.

Bob Friesenhahn skrev 2012-01-30 18:11: On Mon, 30 Jan 2012, Peter Rosin wrote: [Sorry for replying to myself] I do that often. * build-aux/ltmain.m4sh (func_emit_cwrapperexe_src:lt_update_exe_path): Remove duplicate strlen calculation. Hmmm, I like the following better, so I'm going

Re: [PATCH 3/4] maint: pick XSI funcs at runtime, not configure time.

Gary V. Vaughan skrev 2011-11-25 09:57: Determine, on a function by function basis, what XSI features are available in the shell that is actually running the script, rather than the one that was picked at configure time by the re-execution engine. Doesn't this mean that the libtool script

Re: [PATCH 3/4] maint: pick XSI funcs at runtime, not configure time.

Gary V. Vaughan skrev 2011-11-28 10:20: Hi Peter, On 28 Nov 2011, at 15:48, Peter Rosin wrote: Gary V. Vaughan skrev 2011-11-25 09:57: Determine, on a function by function basis, what XSI features are available in the shell that is actually running the script, rather than the one

Re: FYI: [PATCH] ltmain.sh: append relative path trailing slashes explicitly.

Gary V. Vaughan skrev 2011-11-14 11:44: A small refactoring necessary to enable upcoming changesets. Applied as obvious. In addition to being more idiomatic, and hence minimising minimizing, I think US English is preferred. suprises, seeing the slash written explicity when appending to

Re: [PATCH 1/4] libtoolize: use only space delimited file lists.

Gary V. Vaughan skrev 2011-11-05 18:04: This next series of patches simplifies both Makefile.am and libtoolize.m4sh enough that manipulations of the file lists by libtoolize at install time become managable enough to do a nice rewrite of the config-build-aux rename patch that I'll get to next.

Re: [SCM] GNU Libtool branch, master, updated. v2.4-58-g789817d

Hi Gary, Extreme nits follows, you have been warned... Cheers, Peter Gary V. Vaughan skrev 2011-10-17 07:42: This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project GNU Libtool. The

Re: func_convert_file_cygwin_to_w32 woes

Den 2011-01-06 21:29 skrev Ralf Wildenhues: [ dropping libtool@ ] Hi Peter, thanks for working on this! * Peter Rosin wrote on Tue, Jan 04, 2011 at 05:44:58PM CET: Subject: [PATCH] Convert ranlib argument to toolchain format. --- a/libltdl/config/ltmain.m4sh +++ b/libltdl/config

Re: func_convert_file_cygwin_to_w32 woes

Den 2011-01-07 09:02 skrev Peter Rosin: Den 2011-01-06 21:29 skrev Ralf Wildenhues: [ dropping libtool@ ] Hi Peter, thanks for working on this! * Peter Rosin wrote on Tue, Jan 04, 2011 at 05:44:58PM CET: Subject: [PATCH] Convert ranlib argument to toolchain format. --- a/libltdl/config

Re: func_convert_file_cygwin_to_w32 woes

Den 2011-01-07 11:52 skrev Peter Rosin: Subject: [PATCH] Convert file name to toolchain format when blessing archives. * libltdl/config/ltmain.m4sh (func_mode_install): When executing old_postinstall_cmds and old_archive_cmds, convert $oldlib to a format appropriate for the tool and provide

Re: func_convert_file_cygwin_to_w32 woes

Den 2011-01-07 17:55 skrev Charles Wilson: On 1/7/2011 3:02 AM, Peter Rosin wrote: Den 2011-01-06 21:29 skrev Ralf Wildenhues: * Peter Rosin wrote on Tue, Jan 04, 2011 at 05:44:58PM CET: Before I tie up the lose ends with this patch, I wonder if Ralf (or someone else) could tell me if I

Re: func_convert_file_cygwin_to_w32 woes

Den 2011-01-05 05:30 skrev Dan McMahill: On 1/4/2011 11:44 AM, Peter Rosin wrote: Den 2011-01-04 05:39 skrev Dan McMahill: On 1/2/2011 12:37 AM, Ralf Wildenhues wrote: Hi Dan, * Dan McMahill wrote on Sat, Jan 01, 2011 at 04:53:25AM CET: I am trying to build a program under cygwin but using

Re: func_convert_file_cygwin_to_w32 woes

check if this patch helps? (It still needs a ChangeLog etc...) Cheers, Peter From 589ec246bb679612441a849d852b153cf4820466 Mon Sep 17 00:00:00 2001 From: Peter Rosin p...@lysator.liu.se Date: Tue, 4 Jan 2011 17:39:06 +0100 Subject: [PATCH] Convert ranlib argument to toolchain format. --- libltdl

  1   2   3   4   5   >