[PATCH] mq: add -d/--delete option to qpop that deletes after popping

Mads Kiilerich mads at kiilerich.com
Thu Jun 2 17:14:19 CDT 2011


Matt Mackall wrote, On 06/02/2011 09:26 PM:
> On Thu, 2011-06-02 at 14:07 -0500, Kevin Bullock wrote:
>> On Jun 2, 2011, at 11:17 AM, Idan Kamara wrote:
>>
>>> # HG changeset patch
>>> # User Idan Kamara<idankk86 at gmail.com>
>>> # Date 1307031405 -10800
>>> # Node ID 970d8d9b69004d550ec5e33f529de118033047ab
>>> # Parent  30506b89435921bf2e7622613044cca2acab4196
>>> mq: add -d/--delete option to qpop that deletes after popping
>> I'm -1 on this one on the grounds of the Command-Line Option Non-Proliferation Treaty.
> Agreed.

I have been using a patch that does something similar but as qrm -f - 
based on http://markmail.org/message/3vilvesb6wn433ui .

The force option is already used on a lot of commands, and I think the 
obvious behaviour for qrm with force on an applied patch is to pop it 
and remove it. FWIW I think that is consistency, not proliferation.

The slightly less obvious next step is if qrm -f without any patch name 
should pop the top-most patch (like qpop does) and remove it. I think it 
should.

/Mads



More information about the Mercurial-devel mailing list