GNU bug report logs - #20062
[PATCH] diff: add support for --color

Previous Next

Package: diffutils;

Reported by: Giuseppe Scrivano <gscrivan <at> redhat.com>

Date: Sun, 8 Mar 2015 21:57:02 UTC

Severity: normal

Tags: patch

Done: Jim Meyering <jim <at> meyering.net>

Bug is archived. No further changes may be made.

Full log

View this message in rfc822 format

From: Paul Eggert <eggert <at> cs.ucla.edu>
To: Giuseppe Scrivano <gscrivan <at> redhat.com>
Cc: Eric Blake <eblake <at> redhat.com>, 20062 <at> debbugs.gnu.org
Subject: bug#20062: [bug-diffutils] bug#20062: bug#20062: bug#20062: bug#20062: [PATCH] diff: add support for --color
Date: Sat, 12 Sep 2015 02:37:46 -0700

Giuseppe Scrivano wrote:
>   Would block signals between a set_*_color_context and
> reset_color_context be enough or do we need more granularity (there are
> many places where printf is used in the code)?

No, for reasons Eric described: output is buffered.

I suggest looking at the source code of GNU ls and seeing how it does things. 
In normal execution it never blocks signals at all.  (It temporarily blocks 
signals only when processing a signal.)
This bug report was last modified 8 years and 79 days ago.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.