Re: [PATCH] tests: explicitly state that our lexers do not require unistd.h

2012-03-05 Thread Peter Rosin
Peter Rosin skrev 2012-03-06 08:38: > Stefano Lattarini skrev 2012-03-05 21:05: >> Hi Peter. >> >> On 03/05/2012 03:10 PM, Peter Rosin wrote: >>> By default, GNU flex generates code that is dependent on unistd.h, >>> which is not portable to non-ANSI systems causing spurious test >>> failures. >>>

Re: [PATCH] tests: explicitly state that our lexers do not require unistd.h

2012-03-05 Thread Peter Rosin
Stefano Lattarini skrev 2012-03-05 21:05: > Hi Peter. > > On 03/05/2012 03:10 PM, Peter Rosin wrote: >> By default, GNU flex generates code that is dependent on unistd.h, >> which is not portable to non-ANSI systems causing spurious test >> failures. >> >> * tests/cond35.test (tscan.l): Define YY_

Re: [PATCH] tests: explicitly state that our lexers do not require unistd.h

2012-03-05 Thread Stefano Lattarini
Hi Peter. On 03/05/2012 03:10 PM, Peter Rosin wrote: > By default, GNU flex generates code that is dependent on unistd.h, > which is not portable to non-ANSI systems causing spurious test > failures. > > * tests/cond35.test (tscan.l): Define YY_NO_UNISTD_H. > * tests/lex-clean.test (lexer.l): Lik

Re: [PATCH] scripts: support -I -L and -l for cl in compile

2012-03-05 Thread Stefano Lattarini
On 03/05/2012 02:25 PM, Peter Rosin wrote: > Hi! > > I noticed that some expect the compiler to accept a space between > the option letter and the option argument. Looking at POSIX, this > seems like an acceptable expectation, if not even preferred. So, > here's an update for the compile script w

Re: [PATCH 2/2] maint: drop 'Automake::Struct' module

2012-03-05 Thread Stefano Lattarini
On 03/05/2012 08:29 PM, Eric Blake wrote: > On 03/05/2012 12:20 PM, Stefano Lattarini wrote: >> On 03/05/2012 02:18 PM, Eric Blake wrote: >>> On 03/04/2012 08:14 AM, Stefano Lattarini wrote: Now that we require Perl 5.6.2, we can drop the 'Automake::Struct' module, which was basically jus

Re: [PATCH 2/2] maint: drop 'Automake::Struct' module

2012-03-05 Thread Eric Blake
On 03/05/2012 12:20 PM, Stefano Lattarini wrote: > On 03/05/2012 02:18 PM, Eric Blake wrote: >> On 03/04/2012 08:14 AM, Stefano Lattarini wrote: >>> Now that we require Perl 5.6.2, we can drop the 'Automake::Struct' >>> module, which was basically just a backport of Perl 5.6's >>> 'Class::Struct' t

Re: [PATCH 2/2] maint: drop 'Automake::Struct' module

2012-03-05 Thread Stefano Lattarini
On 03/05/2012 02:18 PM, Eric Blake wrote: > On 03/04/2012 08:14 AM, Stefano Lattarini wrote: >> Now that we require Perl 5.6.2, we can drop the 'Automake::Struct' >> module, which was basically just a backport of Perl 5.6's >> 'Class::Struct' to Perl 5.5. >> >> * automake.in: Use 'Class::Struct', n

Re: [PATCH] add xlc support to depcomp

2012-03-05 Thread Stefano Lattarini
On 03/05/2012 05:52 PM, Michael wrote: > Hi Stefano, > > Am 03.03.2012 19:53, schrieb Stefano Lattarini: >> The patch looks good, is unobtrusive, and is small enough to be applied >> without requiring you to sign any copyright paper (yay!). Before applying >> it, though, I'd like to know if you h

[PATCH] tests: explicitly state that our lexers do not require unistd.h

2012-03-05 Thread Peter Rosin
By default, GNU flex generates code that is dependent on unistd.h, which is not portable to non-ANSI systems causing spurious test failures. * tests/cond35.test (tscan.l): Define YY_NO_UNISTD_H. * tests/lex-clean.test (lexer.l): Likewise. * tests/lex-depend-cxx.test (joe.ll): Likewise. * tests/lex

[PATCH] scripts: support -I -L and -l for cl in compile

2012-03-05 Thread Peter Rosin
POSIX mandates that the compiler accepts a space between the -I, -l and -L options and their respective arguments. * lib/compile (func_cl_dashl): New function with factored out code for implementing the -l option for the cl wrapper. (func_cl_dashL): New function with factored out code implementing

Re: [PATCH 2/2] maint: drop 'Automake::Struct' module

2012-03-05 Thread Eric Blake
On 03/04/2012 08:14 AM, Stefano Lattarini wrote: > Now that we require Perl 5.6.2, we can drop the 'Automake::Struct' > module, which was basically just a backport of Perl 5.6's > 'Class::Struct' to Perl 5.5. > > * automake.in: Use 'Class::Struct', not 'Automake::Struct'. > * lib/Automake/Struct.p