Re: [PATCH] Cwrapper should not eat -- arguments

2008-05-28 Thread Olly Betts
On 2008-05-25, Charles Wilson [EMAIL PROTECTED] wrote: +#define LTWRAPPER_OPTION_PREFIX --lt- +#define LTWRAPPER_OPTION_PREFIX_LENGTH 5 If the idea is that the user can change LTWRAPPER_OPTION_PREFIX, then hard-coding the length risks a mismatch if they fail to update

Re: arz_add_sep missing

2008-05-28 Thread Ralf Wildenhues
* Ralf Wildenhues wrote on Wed, May 28, 2008 at 09:52:30PM CEST: Ahem, wasn't that necessary for argz_add and argz_count already, which were added to Libtool in 2008-02-26? OK like this? The test would have exposed the issue on, say, Solaris. The test will skip on systems where argz.o is not

Re: [PATCH] Cwrapper should not eat -- arguments

2008-05-28 Thread Charles Wilson
Olly Betts wrote: On 2008-05-25, Charles Wilson [EMAIL PROTECTED] wrote: +#define LTWRAPPER_OPTION_PREFIX --lt- +#define LTWRAPPER_OPTION_PREFIX_LENGTH 5 If the idea is that the user can change LTWRAPPER_OPTION_PREFIX, then hard-coding the length risks a mismatch if they fail to