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

2012-03-06 Thread Stefano Lattarini
On 03/06/2012 09:44 AM, Peter Rosin wrote: > > I made the changes and pushed, but it feels a bit hard to read -l${sp}foo > compared to -lfoo, so I'm not too happy about it. Oh well... > Thanks. Regards, Stefano

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

2012-03-06 Thread Peter Rosin
Stefano Lattarini skrev 2012-03-05 21:00: > 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.

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

[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