[PATCH 7 of 7] mq: add --empty option to qdelete to remove empty patches

Matt Mackall mpm at selenic.com
Wed May 25 14:06:35 CDT 2011


On Wed, 2011-05-25 at 15:08 +0000, Faheem Mitha wrote:
> On Wed, 25 May 2011 00:23:03 +0300, Idan Kamara <idankk86 at gmail.com> wrote:
> > --===============1828215266==
> > Content-Type: multipart/alternative; boundary=20cf300faf53818fda04a40c310a
> >
> > --20cf300faf53818fda04a40c310a
> > Content-Type: text/plain; charset=ISO-8859-1
> >
> > On Tue, May 24, 2011 at 7:26 PM, Idan Kamara <idankk86 at gmail.com> wrote:
> >
> >> # HG changeset patch
> >> # User Idan Kamara <idankk86 at gmail.com>
> >> # Date 1306254023 -10800
> >> # Node ID 025fd4d81b4ee30ecddf6d3fd04a3d556d03c1c2
> >> # Parent  c21a4febed3f70f2b84b2c8b8cf88ec8269b37ef
> >> mq: add --empty option to qdelete to remove empty patches
> >
> >
> > While this isn't super useful, it's nice to have for some situations.
> > People on IRC aren't convinced so if anyone other than me likes it, say so
> > before it's forgotten...
> 
> Looks good to me. What are these people's objections? Would they care
> to post them here?

The counterargument is that it seems to be not quite useful enough to
justify itself. In other words, it's feature creep.

-- 
Mathematics is the supreme nostalgia of our time.




More information about the Mercurial-devel mailing list