[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 x86_64-pc-cygw

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 04:24

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.

On 2016-03-11 22:22, Michael Haubenwallner wrote: > Hi Peter, > > thanks for looking at the patch! > > On 03/10/2016 12:29 PM, Peter Rosin wrote: >> Hi Michael, >> >> I had a look since I wrote a patch for POSIX nm a couple of years ago >> that I never subm

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 POSIX-com

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 value

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 > >

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

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: git commit forces bootstrap

On 2013-12-09 04:37, Gary V. Vaughan wrote: > On Dec 9, 2013, at 11:52 AM, Peter Rosin wrote: >> ok to push? > > Just to reiterate: I found the barrier for contributing to libtool had > become insanely high in years past, to the point where even I found it > hard to motiva

Re: bootstrap breakage starting with fec7d87

On 2013-11-20 00:32, Gary V. Vaughan wrote: > Hi Ozkan, Petor, > > On Nov 19, 2013, at 11:57 PM, Peter Rosin wrote: > >> On 2013-11-19 10:08, Ozkan Sezer wrote: >>>> Starting with fec7d87 ("funclib.sh: simplify version comparison >>>> functions&qu

Re: bootstrap breakage starting with fec7d87

e. I came up with this patch, but I don't know how portable it is, so I would like someone knowledgeable to comment on it before pushing it... Cheers, Peter >From a7462c5563e124e06f4f61ce2a9cea26cf8be390 Mon Sep 17 00:00:00 2001 From: Peter Rosin Date: Tue, 19 Nov 2013 11:54:27 +0100 Sub

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 differe

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

[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 Date: Mon, 11 Nov 2013 10:00:28 +0100 Subject: [PATCH] libt

Re: g++ and -nostdlib

ed some test that tries to throw C++ exceptions over DLL boundaries, if that isn't already done by tests/exceptions.at... Is that it? Cheers, Peter [1] http://gcc.gnu.org/bugzilla/show_bug.cgi?id=25460 >From 56080c9349ae56fcac6e7bf7f6081dfa48a7fc67 Mon Sep 17 00:00:00 2001 From: Peter R

g++ and -nostdlib

incompatible change? 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 Date: Fri, 8 Nov 2013 19:01:2

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

On 2013-08-22 17:48, Alan Modra wrote: > On Thu, Aug 22, 2013 at 09:34:10PM +0700, Gary V. Vaughan wrote: >>> How can it be correct to say "-m elf32lppclinux" (32-bit) when $host is >>> explicitly 64-bit? That seems like utter garbage to me. What am I >>> missing this time? > > As far as I underst

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 wrote: > >> On 2013-08-22 09:40, Gary V. Vaughan wrote: >>>> Can we please get this simple patch pushed? >>> >>> Done. >> >> To me

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 s

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 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 >>&g

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 t

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: [PATCH] Fix conversion warnings in cwrapper

On 2013-05-28 09:07, Peter Rosin wrote: > 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

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-che

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

Re: [PATCH] Fix conversion warnings in cwrapper

On 2013-05-21 00:49, Yaakov (Cygwin/X) wrote: > From: Yaakov Selkowitz > > Signed-off-by: Yaakov Selkowitz > --- > build-aux/ltmain.in | 18 +- > 1 files changed, 9 insertions(+), 9 deletions(-) > > diff --git a/build-aux/ltmain.in b/build-aux/ltmain.in > index 4c56b98..3d1f5

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

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 r

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 actual

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

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

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 is that the code searches for the prefixed nm before the > plain nm (normally fine), but doesn't

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 >&

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 >>>

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 wrote: >> On 2013-01-12 01:26, Peter Rosin wrote: >>> On 2013-01-11 12:34, Martin Doucha wrote: >>>> I'd like to re

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

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

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

above. Signed-off-by: Peter Rosin --- 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 100644 --- a/buil

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 D

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 >

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 la

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

$tarball_version_file and git is not working. (scriptversion): Update. Copyright-paperwork-exempt: yes Signed-off-by: Peter Rosin --- 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-version-gen index

[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 --- top/maint.mk |6 -- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/top/maint.mk b/top/maint.mk index

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

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

[PATCH v3 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 --- top/maint.mk |6 -- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/top/maint.mk b/top/maint.mk index

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

$tarball_version_file and git is not working. (scriptversion): Update. Copyright-paperwork-exempt: yes Signed-off-by: Peter Rosin --- 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-version-gen index

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

$tarball_version_file and git is not working. (scriptversion): Update. Copyright-paperwork-exempt: yes Signed-off-by: Peter Rosin --- 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-version-gen index

[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 --- top/maint.mk |6 -- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/top/maint.mk b/top/maint.mk index

[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

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 opi

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/

Re: git-version-gen w/o git

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! >>> >>

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

dl/loaders/preopen.c (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 --- NEWS |2 ++

[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 --- m4/libtool.m4 | 16 +--- 1 files changed, 13 insertions(+), 3 deletions(-) diff --git a/m4/libtool.m4 b/m4

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

t sed 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 --- m4/libtool.m4 | 14 +++--- 1 files changed, 7 insertions(+), 7 deletions(-) diff --git a/m4/libtool.m4

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/2] git-version-gen: add --fallback option to use if git is not present

$tarball_version_file and git is not working. (scriptversion): Update. Copyright-paperwork-exempt: yes Signed-off-by: Peter Rosin --- 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-version-gen index

[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 --- top/maint.mk |6 -- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/top/maint.mk b/top/maint.mk index

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 wrote: >> With the latest dumpbin work in place, libtool starts to add >> symbols from import libraries to the generated fooS.c files >> in func_genera

[RFC] Fix libltdl preloader issues with MSVC

oid *) &foo}, {"hello", (void *) &hello}, {"nothing", (void *) 0}, {0, (void *) 0} }; /* This works around a problem in FreeBSD linker */ #ifdef FREEBSD_WORKAROUND static const void *lt_preloaded_setup() { return lt__PROGRAM__LTX_preloaded_symbols; } #endif #if

Re: git-version-gen w/o git

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 from a platform that lacks >> git [MSY

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 \ >>r

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 +++ b/Makefil

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 Date: Thu, 18 Oct 2012 14:27:10 +0200 Subject: [PATCH] libtool: fix spelling nit * build-aux/ltmain.in (func_generate_dlsyms): Fix spelling nit. * libl

Re: Windows Line Endings

[I'm going to try to only respond to the technical sides of this] On 2012-10-09 22:51, Roumen Petrov wrote: > Peter Rosin wrote: >> On 2012-10-08 23:29, Roumen Petrov wrote: >>> Peter Rosin wrote: >>>> Hi Roumen, >>>> >>>> On 2012-10-07 1

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

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 som

Re: Windows Line Endings

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. >From 0f31e375104b00a181557d3809e556066b3d98b1 Mon Sep 17 00:00:00 2001 From: Peter Rosin

[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 Date: Sun, 7 Oct 2012 00:57:10 +0200 Subject: [PATCH] tests: use dry runs in both parts of 'ch

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 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 se

Re: Windows Line Endings

On 2012-10-06 10:13, Gary V. Vaughan wrote: > On 6 Oct 2012, at 14:05, "Gary V. Vaughan" 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 test

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-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 (sta

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

S/MSVC I think, but I don't mind if you skip that. Covering MSYS/MinGW would be plenty enough. > On 5 Oct 2012, at 22:21, Peter Rosin wrote: >>> I think you need to use LT_AT_HOST_DATA instead of AT_DATA, where >>> appropriate. >> >> Unfortunately, the below

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 LT

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 st

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 wrote: >> [snipped loads of stuff] > > [snipped a bit more stuff] > >> You're saying that you are about to: >> >> $ git checkout master >> $

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-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 initial

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 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

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

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

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.

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 &

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

hoice in > hindsight :-( It's 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 wrote: >> On 2012-09-19 11:26, Peter Rosin wrote: >>> On 2012-09-19 09:31, Peter

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 lin

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 wrote: > >> * tests/with-pic.at: Windows uses "-DDLL_EXPORT -DPIC" as the pic >> "flag", but never applies it to static libraries. Cater for this

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. >

[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 --- tests/with-pic.at | 11 ++- 1 files changed, 10 ins

[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 --- tests/runpath-in-lalib.at

[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 --- tests/deplibs-mingw.at |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) Ok to push? Or should

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

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 i

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 pa

Re: libtool quoting error

ode changed in the two middle hunks went out after 2.2.6 and are thus gone in 2.2.8 and later, so that no longer applies. I also took the liberty of changing ltmain.m4sh instead of the generated libtool script. So, this is a better attempt for a patch, with Andreas added to THANKS. Ok to push?

Re: restore EXPORT check

PORTS test. We need to look 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

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 calcul

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 re

[PATCH] cwrapper: avoid duplicate strlen calculation.

>From a2e87f5bfed174de7d715dcf5ca5b51da4761e75 Mon Sep 17 00:00:00 2001 From: Peter Rosin 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-by: Pet

  1   2   3   4   5   6   >