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

[Stefano Lattarini]

On 03/02/2012 10:37 AM, Peter Rosin wrote:
> Stefano Lattarini skrev 2012-03-02 10:32:
>> On 03/01/2012 08:36 PM, 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.
>>>
>>> I did test it, works like a charm for me, no FAILs etc.  Patch pushed
>>> and thanks for looking!
>>>
>> Oh, I knew I was forgetting something: could you please add a NEWS
>> entry for this, and merge it in all the relevant branches?
> 
> Oh crap, I now see that pushed it directly on master.  So much for all
> my whining about using the msvc branch.  I'm planning on cherry-picking
> the patch to the msvc branch, if I'm not hearing anything else.
>
Go for it; given a recent vala-related bug report, we'll probably need
a 1.11.4 release anyway (as you had predicted would likely happen).

> I'll be back with the NEWS wording for review before I'm pushing the
> cherry-pick.
> 

Thanks,
  Stefano