[PATCH] color: add support for the 'export' command

Dan Villiom Podlaski Christiansen danchr at gmail.com
Sun Dec 6 09:26:09 CST 2009


On 6 Dec 2009, at 16:02, Gilles Moris wrote:

> On Saturday 05 December 2009 09:07:19 am Dan Villiom Podlaski Christiansen 
> wrote:
>> color: add support for the 'export' command.
> 
> I am not sure I like the idea: export is for ... export
> The possibility to have mangled escape sequences in exports is not something 
> desirable for me, even if it's in the tty case only. We already have log -p 
> and diff colorized. Isn't that sufficient ? What's the use case ?

Personally, I use it for reviewing patches prior to sending them to the list. I find that colorising ‘export’ makes its output much more readable, and metadata and message stand out much more than ‘log -p’. There's also comfort; there just might be something in the patch that ‘log -p’ doesn't show…

--

Dan Villiom Podlaski Christiansen
danchr at gmail.com

-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 1943 bytes
Desc: not available
URL: <http://selenic.com/pipermail/mercurial-devel/attachments/20091206/8e911fea/attachment.bin>


More information about the Mercurial-devel mailing list