Bug 37388

Summary: Compiling: configure uses incorrect grep reference: grep vs $GREP
Product: cairo Reporter: Mike Cavanagh <isa>
Component: generalAssignee: Carl Worth <cworth>
Status: RESOLVED FIXED QA Contact: cairo-bugs mailing list <cairo-bugs>
Severity: normal    
Priority: medium CC: dam
Version: 1.10.2   
Hardware: SPARC   
OS: Solaris   
Whiteboard:
i915 platform: i915 features:

Description Mike Cavanagh 2011-05-19 17:51:23 UTC
I hope this is not a duplicate.  I did a quick search for it and did not find.

Configure finds grep in /usr/xpg4/bin/grep (not in my $PATH)
But, when it checks for "Warnings", it references grep and not $GREP
So, the error grep: illegal option -- e  is flagged.

I replaced grep with $GREP and the compile continues normally.
    line: 17388

I am using Solaris 9.

I do see other instances using grep vs. $GREP, but I am not a Perl programmer so I did not study if they are valid or not.

Mike
Comment 1 Mike Cavanagh 2011-05-20 10:37:53 UTC
Due to non-related reasons, today, I started over.  I rm'd the cairo directory and untar'd the downloaded package.  Ran the configure command and I did not have the error this time (no change to the configure file).

So, not sure why it occurred yesterday, but does not look like an issue today.
Thanks,
Mike
Comment 2 Mike Cavanagh 2011-05-20 10:57:39 UTC
SOrry.  Ignore this previous post (delete it if you can).  The error still exists.  I overlooked it in the output.

(In reply to comment #1)
> Due to non-related reasons, today, I started over.  I rm'd the cairo directory
> and untar'd the downloaded package.  Ran the configure command and I did not
> have the error this time (no change to the configure file).
> So, not sure why it occurred yesterday, but does not look like an issue today.
> Thanks,
> Mike
Comment 3 Benjamin Otte 2011-05-20 12:05:54 UTC
Should be fixed in git master and the 1.10 branch, see http://cgit.freedesktop.org/cairo/commit/?id=e2c558c435b8c8f9a9057b466020e93d7d61fd70 for the fix.

I $GREPped through the code and didn't find any other places that used grep -e. So I'm assuming it's the only one.
Comment 4 Mike Cavanagh 2011-05-20 13:27:30 UTC
Thanks.
Comment 5 Benjamin Otte 2011-06-08 05:59:53 UTC
*** Bug 38069 has been marked as a duplicate of this bug. ***

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.