Re: [PATCH] scripts: recognize the "q", "s" and "S" actions/modifiers in ar-lib

[Peter Rosin]

Stefano Lattarini skrev 2012-03-01 20:28:
> On 03/01/2012 08:57 AM, Peter Rosin wrote:
>> * lib/ar-lib: Implement the "q" (quick) action as a synonym
>> for "r" (replace).  Ignore "s" (symbol index) and "S" (no symbol
>> index) when used as modifiers and "s" when used as a command,
>> there is simply no way for Microsoft lib to not update the
>> symbol table index in the archive.
>> * tests/ar-lib.test: Check the added behavior.  Also add checks
>> for the recently added "u" (update) and "v" (verbose) modifiers.
>> ---
>>  lib/ar-lib        |   11 ++++++++---
>>  tests/ar-lib.test |   26 ++++++++++++++++++++++++++
>>  2 files changed, 34 insertions(+), 3 deletions(-)
>>
>> Hi!
>>
>> I tried building GNU MP with AR="/path/to/ar-lib lib" and found
>> that ar-lib needed a few more bits.
>>
>> Ok for the msvc branch (and merges to master and branch-1.11)?
>>
> ACK from me, of course (I assume without asking you've already done
> all the proper testing).

I did test it, works like a charm for me, no FAILs etc.  Patch pushed
and thanks for looking!

Cheers,
Peter