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

Previous Next

Package: diffutils;

Reported by: Gisle Vanem <gvanem <at> yahoo.no>

Date: Tue, 1 Dec 2015 09:58:02 UTC

Severity: normal

Tags: patch

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

Bug is archived. No further changes may be made.

Full log


Message #67 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Ondřej Svoboda <ondrej <at> svobodasoft.cz>
To: bug-diffutils <at> gnu.org
Subject: Re: [bug-diffutils] bug#22067: bug#22067: bug#22067: bug#20062:
 bug#20062: [PATCH] diff: add support for --color
Date: Sat, 12 Mar 2016 12:57:51 +0100
Giuseppe, Jim, guys,

I am the author of "compare major, minor numbers of block/character 
special files" (19508) and "treat fifos as identical" (19509). I needed 
them to compare backups of my Maemo 5 system (Nokia N900) -- there are 
static device files in /dev and some fifos too.

The first patch hasn't received a code review yet -- may I ask you or 
the list to give it a look? I think the patch is in a better shape than 
"treat fifos as identical" which isn't finished although it received a 
suggestion by Eric Blake (on 12 Feb last year) that fstat could be used 
to compare the fifos.

I rebased the patches in my working copy and I will at least send new 
versions to the list.

I am finishing my studies now so I can dedicate only limited time to 
finishing the patches. Let me know if you would like the functionality 
in the next release and I'll see what I can do.

Cheers,
Ondra

On 10.3.2016 22:02, Giuseppe Scrivano wrote:
> Hi Jim,
>
> Jim Meyering <jim <at> meyering.net> writes:
>
>> Yes. It's in the queue, but first I'll make releases for grep and gzip.
>> sed is also overdue. Any help with triaging things in the bug
>> lists would be most welcome:
>>
>>    http://debbugs.gnu.org/cgi/pkgreport.cgi?package=diffutils
> I went through the open bugs (I skipped the ones already on wishlist)
> and this is what I've found:
>
> * 18641 [diffutils] [PATCH] diff: Add --only=REGEX option to parse only files that match REGEX
> RFC
>
> * 19508 [diffutils] [PATCH] diff: compare major, minor numbers of block/character special files
> * 19509 [diffutils] [PATCH] diff: treat fifos as identical
> not blocking issue, patch needs review
>
> * 17185  diff --help doesn't mention short forms of some options
> minor issue, not blocking
>
> * 18655 Regression in 3.3 documentation versus 3.2
> answered, bug must be closed
>
> * 18973 diffutils-3.3 test-mbrtowc3.sh fails on Solaris without European localization
> not triaged, no access to solaris to try it
>
> * 19489 FYI: I've updated copyright dates -- and gnulib
> not a bug, must be closed
>
> * 19825 unified output format, number of consecutive unaffected lines, and POSIX
> answered, can be closed
>
> * 19835 RFC: diff: skip initial columns before comparing
> RFC with patch not reviewed
>
> * 19984 [diffutils] Diff and symlinks
> commented, can be closed
>
> * 20250 [diffutils] diff by size
> RFC for size only comparison (I would close as NOTFIX)
>
> * 20279 [diffutils] With -r, option to avoid duplicate differences due to relative symbolic links
> RFC
>
> * 20794 [diffutils] ignore subdirectory with diff
> Went unanswered, looks like a RFC for a weird use case
>
> * 20929 [diffutils] problem with diff -B and incomplete lines
> Fixed by:
>
> commit d2fd9d4683ef60c259a3b426f71cef1b89ff383d
> Author: Paul Eggert <eggert <at> cs.ucla.edu>
> Date:   Wed Sep 3 15:58:03 2014 -0700
>
>      diff: fix bug with diff -B and incomplete lines
>
>
> * 21034 [diffutils] [bug-diffutils] bug#21023: bug#21023: REQUEST: --no-dereference missing from manpage + please make a short option for it
> RFC, should probably be closed and pointed out that --no-d will work as well
>
> * 21579 [diffutils] diffutils 3.3 man page may need update
>
> Fixed by:
>
> commit 2cd4ff3a5ff52d89b6b992d158f389b757f4faf4
> Author: Jim Meyering <meyering <at> fb.com>
> Date:   Mon Aug 31 23:12:43 2015 -0700
>
>      build: correct man-page generation rule
>
>
> * 21665  [diffutils] Use of mmap for large files
> already answered as WONTFIX, must be closed
>
> * 21674 [diffutils] Fix for context-header
> plain diff patch proposed, changes the format of the header
>
> * 21715 [diffutils] feat req: an option to skip directory inode comparison
> RFC
>
> * 22108 [diffutils] diff wrapper script for very large files, low memory
> duplicate of 21665
>
> * 22245 [diffutils] [lamby <at> debian.org: Bug* 809007: diffutils: FTBFS: FAIL: test-update-copyright.sh]
> I had that issue some time ago but it was fixed upstream.  I have not bisected
>
> * 22507 [diffutils] diffutils releases
> not a bug, working for the release :)
>
> * 22529 [diffutils] cmp man page too sophisticated
> not a bug
>
> * 22535 [diffutils] REQ: regex dialect switching (for -F and -I)
> RFC, I would close it as a WONTFIX
>
> * 22816 [diffutils] cmp --verbose EOF message could be more verbose
> RFC
>
> * 16467 [diffutils] Diffutils 3.3 v. VMS (et al.)
> could be fixed in gnulib, not a regression anyway
>
> Regards,
> Giuseppe
>




This bug report was last modified 9 years and 68 days ago.

Previous Next


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