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

[Peter Rosin]

Stefano Lattarini skrev 2012-03-02 11:15:
> 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).

Good, also please don't push anything until I have done this, I'm not too
fond of having to redo the merges needed from maint->msvc->master etc.
There's a bit of cruft that needs to be sorted, as it's been a while since
maint was merged into msvc.  Thanks!

Cheers,
Peter