Apache OpenOffice (AOO) Bugzilla – Issue 85610
SRC680_m244:Unassigned shell variable $GREP used in config_office/configure.in
Last modified: 2009-08-30 00:48:29 UTC
Unassigned shell variable $GREP is used in config_office/configure.in. As AC_PROG_EGREP depends on AC_PROG_GREP after autoconf 2.61 and AC_HEADER_STDC depends on AC_PROG_EGREP, $GREP may happen to be assigned in some cases. However, in win32 build with MS tools AC_HEADER_STDC is never invoked and $GREP at line 2985 will always fail. Other use of $GREP is not invoked in win32 build but we should consistently avoid using $GREP or invoke AC_PROG_GREP explicitly.
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.