translation of command line arguments (was: New auxiliary archive script)

2010-08-13 Thread Ralf Wildenhues
* Peter Rosin wrote on Fri, Aug 13, 2010 at 11:38:43AM CEST: > Den 2010-08-12 19:55 skrev Ralf Wildenhues: > > Same with 'compile' by the way. If -O3 is spelt differently for cl, > > then I think translating that is a good idea. > I think we should keep such translation to a bare minimum. The cha

Re: New auxiliary archive script

2010-08-13 Thread Peter Rosin
Den 2010-08-12 19:55 skrev Ralf Wildenhues: > * Peter Rosin wrote on Thu, Aug 12, 2010 at 01:05:09PM CEST: >> Or always pass on any '-lib' option to PROGRAM w/o trying to interpret >> it further. That might be a good addition, since there might be other >> interesting options to pass through, and t

Re: New auxiliary archive script

2010-08-12 Thread Ralf Wildenhues
* Peter Rosin wrote on Thu, Aug 12, 2010 at 01:05:09PM CEST: > Den 2010-08-12 05:29 skrev Ralf Wildenhues: > > * Peter Rosin wrote on Wed, Aug 11, 2010 at 04:12:10PM CEST: > Oh... oh! That means that AM_PROG_CC_WRAPPER and AM_PROG_CC_C_O should > trigger 'compile' for completely different reasons,

Re: New auxiliary archive script

2010-08-12 Thread Peter Rosin
Den 2010-08-12 05:29 skrev Ralf Wildenhues: > * Peter Rosin wrote on Wed, Aug 11, 2010 at 04:12:10PM CEST: >> One thing to consider is that the compile script is triggered "by chance" >> for MSVC. It is just a lucky coincidence that MSVC doesn't support -c -o, >> so that the compile script can add

Re: New auxiliary archive script

2010-08-11 Thread Ralf Wildenhues
* Peter Rosin wrote on Wed, Aug 11, 2010 at 04:12:10PM CEST: > One thing to consider is that the compile script is triggered "by chance" > for MSVC. It is just a lucky coincidence that MSVC doesn't support -c -o, > so that the compile script can add that support and then tag on the other > needed c

Re: New auxiliary archive script

2010-08-11 Thread Christopher Hulbert
On Wed, Aug 11, 2010 at 10:12 AM, Peter Rosin wrote: > Den 2010-08-10 22:18 skrev Ralf Wildenhues: Wrt. the AM_PROG_AR implementation, __oline__ is a real nono, as it causes spurious diffs in generated configure scripts.  I'll propose an updated version. >>> >>> Oh, didn't know that

Re: New auxiliary archive script

2010-08-11 Thread Peter Rosin
Den 2010-08-10 22:18 skrev Ralf Wildenhues: >>> Wrt. the AM_PROG_AR implementation, __oline__ is a real nono, as it >>> causes spurious diffs in generated configure scripts. I'll propose >>> an updated version. >> >> Oh, didn't know that. I look forward to it, since I reused some of the >> code fo

Re: New auxiliary archive script

2010-08-10 Thread Ralf Wildenhues
* Peter Rosin wrote on Tue, Aug 10, 2010 at 09:01:29AM CEST: > 'wrappar' did cross my mind, but that just looks like a funny accent :-) Yes, that one is funny indeed, and another almost-palindrome. > Den 2010-08-09 21:39 skrev Ralf Wildenhues: > >Further I'm fixing the archive and the compile scr

Re: New auxiliary archive script

2010-08-10 Thread Eric Blake
On 08/10/2010 01:01 AM, Peter Rosin wrote: >> Further I'm fixing the archive and the compile script to use ! not ^ for >> shell pattern negation, and using $me also in the --help output. > > I tested with $me in the --help output but didn't get it to work so I > removed it. But your version works.

Re: New auxiliary archive script

2010-08-10 Thread Peter Rosin
Den 2010-08-10 09:01 skrev Peter Rosin: BTW, why is ! better than ^ here? Just curious. I found the answer myself. Quoting the portable shell chapter in the autoconf manual: | 11.5 Shell Pattern Matching | | Nowadays portable patterns can use negated character classes | like ‘[!-aeiou]’. The o

Re: New auxiliary archive script

2010-08-10 Thread Peter Rosin
27;m picking the script addition part from your commit, adding it to the msvc branch and merging to master for now: Cool, thanks! +2010-08-06 Peter Rosin + + Add new auxiliary 'archive' script, wrapping Microsoft lib. + * lib/archive: New auxiliary script.

Re: New auxiliary archive script

2010-08-09 Thread Karl Berry
could you please adjust your scripts to also autoupdate the 'ar-lib' script from Automake to Gnulib, Sure, done. Tx.

Re: New auxiliary archive script

2010-08-09 Thread Ralf Wildenhues
Hello Karl, * Peter Rosin wrote on Fri, Aug 06, 2010 at 01:43:03PM CEST: > Add new auxiliary 'ar-lib' script, wrapping Microsoft lib. > > * lib/ar-lib: New auxiliary script. [...] could you please adjust your scripts to also autoupdate the 'ar-lib' script from Automake to Gnulib, just li

Re: New auxiliary archive script

2010-08-09 Thread Ralf Wildenhues
light preference of that over arwrap, simply because I cannot pronounce the latter. And we can re-interpret ar-lib from "archive wrapper for LIB" to "archiver for libraries" if that helps making it more general. ;-) So, I'm picking the script addition part from your

Re: New auxiliary archive script

2010-08-08 Thread Ralf Wildenhues
* Ralf Wildenhues wrote on Sat, Aug 07, 2010 at 08:03:56AM CEST: > Darn, we need another name: 'archive' is likely too common when automake > is going to put all files with that name in the distribution of each > package using Automake. No need to send another patch, but any other > good suggestio

Re: New auxiliary archive script

2010-08-08 Thread Gary V. Vaughan
Hallo Ralf, On 7 Aug 2010, at 13:03, Ralf Wildenhues wrote: > Darn, we need another name: 'archive' is likely too common when automake > is going to put all files with that name in the distribution of each > package using Automake. No need to send another patch, but any other > good suggestions t

Re: New auxiliary archive script

2010-08-08 Thread Peter Rosin
Den 2010-08-07 08:03 skrev Ralf Wildenhues: Darn, we need another name: 'archive' is likely too common when automake is going to put all files with that name in the distribution of each package using Automake. No need to send another patch, but any other good suggestions that fit in with the exi

Re: New auxiliary archive script

2010-08-08 Thread Andrew W. Nosenko
On Sat, Aug 7, 2010 at 09:03, Ralf Wildenhues wrote: > Darn, we need another name: 'archive' is likely too common when automake > is going to put all files with that name in the distribution of each > package using Automake.  No need to send another patch, but any other > good suggestions that fit

Re: New auxiliary archive script

2010-08-08 Thread Peter Rosin
Den 2010-08-07 17:53 skrev Ralf Wildenhues: * Ralf Wildenhues wrote on Sat, Aug 07, 2010 at 08:03:56AM CEST: Darn, we need another name: 'archive' is likely too common when automake is going to put all files with that name in the distribution of each package using Automake. No need to send anot

Re: New auxiliary archive script

2010-08-06 Thread Ralf Wildenhues
Darn, we need another name: 'archive' is likely too common when automake is going to put all files with that name in the distribution of each package using Automake. No need to send another patch, but any other good suggestions that fit in with the existing scripts? I don't find arlib so bad, exc

Re: New auxiliary archive script

2010-08-06 Thread Peter Rosin
Den 2010-08-06 13:43 skrev Peter Rosin: commit 3d3229ff86bd8194974bcf56948f5c6b0b54d99a Author: Peter Rosin Date: Fri Aug 6 13:35:26 2010 +0200 Add new auxiliary 'archive' script, wrapping Microsoft lib. * lib/archive: New auxiliary script. * lib/Makefile.am: Add above. * m4/archi

cond5.test spurious failure (was: New auxiliary archive script)

2010-08-06 Thread Stefano Lattarini
Hello Peter. At Friday 06 August 2010, Peter Rosin wrote: > [CUT] > I did get a spurious failure in cond5.test, but it went away when > I ran it separately. > Did you by chance run many tests in parallel? If yes, you might have hit a known issue: "The test cond5.test can fail spuriously due to t

Re: New auxiliary archive script

2010-08-06 Thread Peter Rosin
If I'm touching too many files, feel free to zap whatever hunk you don't like... Cheers, Peter commit 3d3229ff86bd8194974bcf56948f5c6b0b54d99a Author: Peter Rosin Date: Fri Aug 6 13:35:26 2010 +0200 Add new auxiliary 'archive' script, wrapping Microsoft lib. * lib/archive: New

Re: New auxiliary archive script

2010-08-05 Thread Ralf Wildenhues
* Peter Rosin wrote on Fri, Aug 06, 2010 at 12:00:53AM CEST: > Den 2010-08-01 20:06 skrev Ralf Wildenhues: > >For actual applicability, it is sufficient if you provide a working > >script, post testsuite results with it in place (you might want to > > MAKE='make AR=...' > > When I do > > MAKE='

Re: New auxiliary archive script

2010-08-05 Thread Peter Rosin
Hi Ralf, Den 2010-08-01 20:06 skrev Ralf Wildenhues: For actual applicability, it is sufficient if you provide a working script, post testsuite results with it in place (you might want to MAKE='make AR=...' When I do MAKE='make "AR=/path/to/archive lib"' make "AR=/path/to/archive lib" chec

Re: New auxiliary archive script

2010-08-05 Thread Ralf Wildenhues
* Peter Rosin wrote on Thu, Aug 05, 2010 at 11:10:38AM CEST: > Den 2010-08-04 22:18 skrev Ralf Wildenhues: > >Now if you insist, then maybe we can just find another compromise name. > >Please set $me to the name and use that throughout, so at least a change > >is easily done. ;-) > > I'm perfectl

Re: New auxiliary archive script

2010-08-05 Thread Peter Rosin
Hi Ralf, and thank you right back for _your_ work on this! :-) Den 2010-08-04 22:18 skrev Ralf Wildenhues: thanks for your work on this! * Peter Rosin wrote on Wed, Aug 04, 2010 at 12:14:28PM CEST: Den 2010-08-01 20:06 skrev Ralf Wildenhues: I wonder whether we can and should call it 'archive

Re: New auxiliary archive script

2010-08-04 Thread Ralf Wildenhues
* Eric Blake wrote on Wed, Aug 04, 2010 at 11:04:29PM CEST: > On 08/04/2010 02:18 PM, Ralf Wildenhues wrote: > >> # strip leading dash in $action > >> case $action in > >> -*) action="${action#-}" ;; > >> esac > > > > The case statement is unnecessary, you can use just > > action=${action#-} >

Re: New auxiliary archive script

2010-08-04 Thread Eric Blake
On 08/04/2010 02:18 PM, Ralf Wildenhues wrote: >> # strip leading dash in $action >> case $action in >> -*) action="${action#-}" ;; >> esac > > The case statement is unnecessary, you can use just > action=${action#-} Either way, this assumes an XSI shell. Are we guaranteed that this script w

Re: New auxiliary archive script

2010-08-04 Thread Ralf Wildenhues
Hi Peter, thanks for your work on this! * Peter Rosin wrote on Wed, Aug 04, 2010 at 12:14:28PM CEST: > Den 2010-08-01 20:06 skrev Ralf Wildenhues: > >Yes, in principle: when them bugs are fixed. Would you be willing to > >rewrite the script so it > > > >- has the blurb header similar to 'compile

Re: New auxiliary archive script

2010-08-04 Thread Peter Rosin
Hi Ralf, Den 2010-08-01 20:06 skrev Ralf Wildenhues: * Peter Rosin wrote on Sun, Aug 01, 2010 at 06:54:57PM CEST: The script still needs @-support, in order to make it possible for libtool to avoid falling back to "-r -o" piecewise linking when the command line gets long, so that's a TODO, and

Re: New auxiliary archive script

2010-08-01 Thread Ralf Wildenhues
Hi Peter, * Peter Rosin wrote on Sun, Aug 01, 2010 at 06:54:57PM CEST: > Now we need a something similar for the MS archiver (lib.exe) Right. > As it happens, binutils ar does not work as well as lib, MSVC > sometimes has problems with the archives produced by ar. I > know that I have stated oth

New auxiliary archive script (was: Re: pr-msvc-support merge)

2010-08-01 Thread Peter Rosin
Den 2010-08-01 08:59 skrev Ralf Wildenhues: Now that assignment has gone through a couple of weeks ago, I went ahead and committed your version 7 of this patch into a new 'msvc' branch off of maint and merged that into master. I intend to merge also into branch-1.11/maint before the 1.11.2 relea