Apache OpenOffice (AOO) Bugzilla – Issue 33258
Sun Workshop compiler version checking wrong
Last modified: 2006-03-14 21:02:56 UTC
Hi, SRC680_m51 contains this: _workshop_string=`$CC -V 2>&1 | grep '^cc' | sed -e 's/.* C /` It should probably read _workshop_string=`$CC -V 2>&1 | grep '^cc' | sed -e 's/.* C //'`
-> Ken. set target
The error message while configuring is: checking the SUN Workshop C compiler version... ./configure: command substitution: line 1: unexpected EOF while looking for matching `'' ./configure: command substitution: line 2: syntax error: unexpected end of file configure: error: found version "", use version 5.0 or 5.2 of the Sun Workshop C compiler
The problem was brought in by rt in 1.47, so I'll reassign to him to fix it on the master: Index: configure =================================================================== RCS file: /cvs/tools/config_office/configure,v retrieving revision 1.46 retrieving revision 1.47 diff -u -r1.46 -r1.47 --- configure 17 Jun 2004 11:59:34 -0000 1.46 +++ configure 13 Jul 2004 15:47:30 -0000 1.47 @@ -3307,10 +3307,8 @@ COMPATH=`echo $_cc | $SED -n "s/\/cc//p"` echo "$as_me:$LINENO: checking the SUN Workshop C compiler version" >&5 echo $ECHO_N "checking the SUN Workshop C compiler version... $ECHO_C" >&6 - _workshop_version=`$CC -V 2>&1 | $AWK '/^cc/ { print $7 }'` - if test "$_workshop_version" = "C"; then - _workshop_version=`$CC -V 2>&1 | $AWK '/^cc/ { print $8 }'` - fi + _workshop_string=`$CC -V 2>&1 | grep '^cc' | sed -e 's/.* C /` + _workshop_version=`echo $_workshop_string | $AWK '{ print $1 }'` _workshop_major=`echo $_workshop_version | $AWK -F. '{ print $1 }'`
rt: which compiler do you use now for Solaris builds? Is it 5.5? If it is so, please add it to the list of minor number checked (only 5.0 and 5.2 are supported now). I'm now building with 5.5 too on my new machine.
Hi, I checked in the change suggested here, but without knowing whether it's correct or not. Please not: revision 1.47 is just the result of integrating CWS ooo20040620. Real authors of the change are, according to cvs, waratah (1.44.12.2:) and rene (1.44.12.1). Integration of a CWS is a more or less automatic process, in most cases I do not know anything about the code changes introduced. Concerning you last question: we are using Sun C++ 5.5
The same change is needed for configure.in.
Hi, attached file is suggectes fix I use right now.
Created attachment 17397 [details] Suggested fix
Fixed on pj01. waratah: please verify.
Waratah puts hand up for silly error award. Thanks Pavel. I did request someone to actually test the change :-)
Closing.