Apache OpenOffice (AOO) Bugzilla – Full Text Issue Listing |
Summary: | SRC680_m244:Unassigned shell variable $GREP used in config_office/configure.in | ||
---|---|---|---|
Product: | Build Tools | Reporter: | tono <t_ono> |
Component: | code | Assignee: | rene |
Status: | CLOSED FIXED | QA Contact: | issues@tools <issues> |
Severity: | Trivial | ||
Priority: | P1 (highest) | CC: | issues, pavel, quetschke |
Version: | current | ||
Target Milestone: | OOo 2.4 | ||
Hardware: | PC | ||
OS: | Windows, all | ||
Issue Type: | DEFECT | Latest Confirmation in: | --- |
Developer Difficulty: | --- |
Description
tono
2008-01-26 15:26:32 UTC
in a masterfix of SRC680_m244, we changed all occurences of $GREP to $EGREP. This issue is then fixed. Please verify reassign for verification As Macro EGREP may not be assigned in win32 build, replacing $GREP with $EGREP does not work. Plain grep should be used (it is used everywhere) unless we explicitly specify AC_PROG_GREP or AC_PROG_EGREP. tono: you're wrong, plain GREP is not used everywhere. And as you said, plain $GREP also has problems. We need a cws to unify this. (And add the explicit check) . The error message is: checking for gperf... /usr/bin/gperf checking gperf version... ./configure: line 11406: ^GNU gperf: command not found ./configure: line 11406: test: : integer expression expected configure: error: too old, you need at least 3.0.0 @rene: I meant that not using macro variable but specify grep directly. pjanik: please refer http://www.openoffice.org/issues/show_bug.cgi?id=85590 as well. IMHO we need this part. maho: paveljanik didn't question that. in fact, he was the one who found we need it :-) fixed in cws grepcheck seen good in grepcheck, let's push for 2.4 Closing. |