[PATCH] new command debugignore

Benoit Boissinot bboissin at gmail.com
Sun Jul 4 09:55:06 CDT 2010


On Sun, Jul 4, 2010 at 4:37 PM, Martin Geisler <mg at lazybytes.net> wrote:
> Jason Harris <jason at jasonfharris.com> writes:
>> Ahh... I guess that would be useful for some people as well. However
>> for my purposes I need something which implements what I added. But I
>> can see that it might not be so useful to others... Would you want
>> this current command with a different name or maybe you guys wouldn't
>> want it in core Mercurial at all, and I will just leave it in my
>> patched version which ships with MacHg (once I implement your
>> suggestions...)
>
> I think it should be an extension -- printing the big, ugly regexp seems
> of little use as a core feature. You can then ship and enable the
> extension by default in MacHg.

I disagree on that. It doesn't look complex, should be easy to
maintain, it's a debug command and it can be useful.
On the other hand we do not promise backward compat for debug commands, I think.

Furthermore it needs refactor in core to have it as an extension. Once
this refactor is done, the extension is trivial.

cheers,

Benoit


More information about the Mercurial-devel mailing list