GNU bug report logs - #18266
grep -P and invalid exits with error

Previous Next

Package: grep;

Reported by: Santiago <santiago <at> debian.org>

Date: Thu, 14 Aug 2014 15:43:02 UTC

Severity: wishlist

Merged with 18455

Done: Paul Eggert <eggert <at> cs.ucla.edu>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Jim Meyering <jim <at> meyering.net>
To: 18266 <at> debbugs.gnu.org, Paul Eggert <eggert <at> cs.ucla.edu>,  Santiago <santiago <at> debian.org>
Cc: 758105 <at> bugs.debian.org, 18266-done <at> debbugs.gnu.org, Vincent Lefevre <vincent <at> vinc17.net>, 761157 <at> bugs.debian.org
Subject: bug#18266: Bug#758105: bug#18266: grep -P and invalid exits with error
Date: Thu, 11 Sep 2014 11:37:11 -0700
On Thu, Sep 11, 2014 at 10:07 AM, Paul Eggert <eggert <at> cs.ucla.edu> wrote:
> Vincent Lefevre wrote:
>
>> I've just reported a new Debian concerning the performance problem.
>
>
> It's not clear from http://bugs.debian.org/761157 that the performance
> problem occurs only with -P, but I assume that's what is meant.
>
> Since this is a performance bug with PCRE, I suggest moving the Debian bug
> report to the Debian libpcre3 package.  Grep cannot go back to the old way,
> which could cause grep to crash, and the bug cannot be fixed in grep because
> libpcre3 does not provide a fast way to search arbitrary data that may
> include encoding errors.  It really is a problem that requires changes to
> libpcre3 to fix; grep cannot fix it.
>
> In the meantime, in order to use 'grep' to search for strings in arbitrary
> data, I suggest omitting the '-P'.  Also, I suggest using the C locale.
>
> As the GNU bug 18266 "grep -P and invalid exits with error" has been fixed,
> I'm closing that bug report.  Please feel free to open a separate GNU bug
> report for the performance issue.
>
> PS.  While composing this email I noticed another bug in grep -P and
> encoding errors, which I fixed by installing the attached patch.

Thanks for fixing yet another bug, Paul.
Would you mind adding a test to trigger that one?




This bug report was last modified 10 years and 249 days ago.

Previous Next


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