GNU bug report logs - #17156
[PATCH 0/5] Patches to apply

Previous Next

Package: grep;

Reported by: Paolo Bonzini <bonzini <at> gnu.org>

Date: Tue, 1 Apr 2014 09:19:01 UTC

Severity: normal

Tags: patch

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

Bug is archived. No further changes may be made.

Full log


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

From: Paolo Bonzini <bonzini <at> gnu.org>
To: Paul Eggert <eggert <at> cs.ucla.edu>, 17156 <at> debbugs.gnu.org
Cc: meyering <at> fb.com
Subject: Re: bug#17156: [PATCH 2/5] Revert conversion to shell scripts
Date: Tue, 01 Apr 2014 19:02:32 +0200
Il 01/04/2014 18:07, Paul Eggert ha scritto:
> On 04/01/2014 08:08 AM, Paolo Bonzini wrote:
>> Unless I misunderstand, this is clearly not possible since "grep" does
>> have to deal with -E and -F.
>
> Yes, sorry I was not clear.  Of course 'grep' must implement -E and -F.
> But this does not require acollection of C code in which fgrep.c,
> egrep.c, and grep.c collide with another batch of C code in subtle
> ways.  Instead, all we should need (if we want egrep.c and fgrep.c to be
> in C) is to have egrep.c exec 'grep', and likewise for fgrep.c.
>
> There may be other ways to do it, but the point is to keep it reaallly
> simple, and separated from grep.c, so that people who maintain grep
> don't have to worry about egrep and fgrep unless they're actually
> *working* on egrep and fgrep (which should be never....).  This is the
> separation of concerns that I referred to.

Yeah, understood now.  I'm not sure how this is not satisfied by the 
current code though. :)  It's already a big step forward from the 
manually-listed objects of grep 2.5.x.

Paolo




This bug report was last modified 11 years and 48 days ago.

Previous Next


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