After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 313531 - gnome-autogen.sh uses grep syntax solaris doesn't understand
gnome-autogen.sh uses grep syntax solaris doesn't understand
Status: RESOLVED FIXED
Product: gnome-common
Classification: Core
Component: general
git master
Other All
: Normal normal
: ---
Assigned To: Gnome Common Maintainer(s)
Gnome Common Maintainer(s)
: 313539 313540 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2005-08-15 15:51 UTC by James Andrewartha
Modified: 2005-08-17 00:41 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description James Andrewartha 2005-08-15 15:51:52 UTC
Please describe the problem:
I'm tinderboxing gnome, and gnome-autogen.sh uses syntax that solaris's
/usr/bin/grep doesn't understand (it's incredibly brain-dead).

Steps to reproduce:
1. 
2. 
3. 


Actual results:


Expected results:


Does this happen every time?


Other information:
The solution is to either use /usr/xpg4/bin/grep or ggrep. The question is,
should I just work around this by fiddling my $PATH, or is it an actual bug that
needs to be fixed, by munging $PATH, or introducing $grep which uses ggrep by
preference ...
Comment 1 James Andrewartha 2005-08-15 17:01:02 UTC
*** Bug 313539 has been marked as a duplicate of this bug. ***
Comment 2 James Andrewartha 2005-08-15 17:01:31 UTC
*** Bug 313540 has been marked as a duplicate of this bug. ***
Comment 3 James Henstridge 2005-08-16 03:14:51 UTC
Can you point out which particular syntax Solaris Grep is getting stuck on?
Comment 4 James Henstridge 2005-08-16 10:53:26 UTC
Is this the grep call causing the problems?

    if grep "^\(AC\|IT\)_PROG_INTLTOOL" $configure_ac >/dev/null; then

That is using extended regexps, which aren't usually turned on unless you use
"grep -E" or egrep.  If that's the only problem you are running into, then we
can fix that.
Comment 5 James Andrewartha 2005-08-16 12:51:40 UTC
Yeah, that's the syntax. Solaris egrep doesn't like it either - it doesn't want
the (, ), and | escaped. GNU egrep doesn't want them escaped either, so changing
the line to 
if egrep "^(AC|IT)_PROG_INTLTOOL" $configure_ac >/dev/null; then
should be the correct fix.
This actaully occurs twice, at lines 230 and 361.
Comment 6 James Henstridge 2005-08-17 00:41:57 UTC
Fixed in CVS.