unofficial mirror of bug-guix@gnu.org 
 help / color / mirror / code / Atom feed
* bug#66753: grep 3.8 now needs pcre2 as input, not pcre
@ 2023-10-25 22:21 Matt Beshara
  2024-01-22  4:34 ` Maxim Cournoyer
  0 siblings, 1 reply; 2+ messages in thread
From: Matt Beshara @ 2023-10-25 22:21 UTC (permalink / raw)
  To: 66753

Hi Guix people,
I have been working on creating a package definition for 
pulseaudio-equalizer¹ and when built with the current definition 
of the grep package, it prints this error message when running:

grep: Perl matching not supported in a --disable-perl-regexp build
grep: write error: Broken pipe

Searching for that error message, I came across this:
https://trac.macports.org/ticket/65800

So it seems that, for version 3.8, the pcre input package for grep 
should be changed to pcre2.  I have made this change in a new 
definition which inherits grep and told my pulseaudio-equalizer 
package to use that as a propagated input, and that causes the 
error to go away.  For the sake of completeness, here’s the 
definition I used:

(define grep-fixed
  (package
    (inherit grep)
    (inputs (list pcre2))))

Best wishes,
Matt

1. https://github.com/pulseaudio-equalizer-ladspa/equalizer




^ permalink raw reply	[flat|nested] 2+ messages in thread

* bug#66753: grep 3.8 now needs pcre2 as input, not pcre
  2023-10-25 22:21 bug#66753: grep 3.8 now needs pcre2 as input, not pcre Matt Beshara
@ 2024-01-22  4:34 ` Maxim Cournoyer
  0 siblings, 0 replies; 2+ messages in thread
From: Maxim Cournoyer @ 2024-01-22  4:34 UTC (permalink / raw)
  To: Matt Beshara; +Cc: 66753-done

Hi,

Matt Beshara <m@mfa.pw> writes:

> Hi Guix people,
> I have been working on creating a package definition for
> pulseaudio-equalizer¹ and when built with the current definition of
> the grep package, it prints this error message when running:
>
> grep: Perl matching not supported in a --disable-perl-regexp build
> grep: write error: Broken pipe
>
> Searching for that error message, I came across this:
> https://trac.macports.org/ticket/65800
>
> So it seems that, for version 3.8, the pcre input package for grep
> should be changed to pcre2.  I have made this change in a new
> definition which inherits grep and told my pulseaudio-equalizer
> package to use that as a propagated input, and that causes the error
> to go away.  For the sake of completeness, here’s the definition I
> used:
>
> (define grep-fixed
>  (package
>    (inherit grep)
>    (inputs (list pcre2))))
>
> Best wishes,
> Matt

This appears to have been fixed independently by spacecadet in commit
5b0cea02358044f0cc695bacc3f44db1e220239b ("gnu: grep: Fix PCRE matches
(grep -P).").

Closing!

-- 
Thanks,
Maxim




^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2024-01-22  4:35 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-10-25 22:21 bug#66753: grep 3.8 now needs pcre2 as input, not pcre Matt Beshara
2024-01-22  4:34 ` Maxim Cournoyer

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).