Build break in mozilla/configure.in

RESOLVED FIXED

Status

SeaMonkey
Build Config
RESOLVED FIXED
14 years ago
14 years ago

People

(Reporter: Rupesh, Assigned: dbaron)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

14 years ago
The build is failed in "configure" with the below error.

***************************************************************************

checking for X... (cached) no
checking whether ld has archive extraction flags... (cached) no
checking for 64-bit OS... no
I:/builds/moz175/mozilla/configure: 7233: Syntax error: "!" unexpected
*** Fix above errors and then restart with "make.exe -f client.mk build"
make: *** [I:/builds/moz175/mozilla/objret/Makefile] Error 1

*****************************************************************************
(Reporter)

Comment 1

14 years ago
The similar error occurs with mozilla/configure.in file also. The below patch
may work.

*************************Start***********************************
Index: configure.in
===================================================================
RCS file: /cvsroot/mozilla/configure.in,v
retrieving revision 1.1399
diff -u -8 -r1.1399 configure.in
--- configure.in	15 Dec 2004 05:52:34 -0000	1.1399
+++ configure.in	22 Dec 2004 13:41:54 -0000
@@ -1950,17 +1950,17 @@
                      [cat > conftest.c <<EOF
 #pragma GCC visibility push(hidden)
                       int foo_hidden = 1;
 #pragma GCC visibility push(default)
                       int foo_default = 1;
 EOF
                       ac_cv_visibility_pragma=no
                       if ${CC-cc} -Werror -S conftest.c -o conftest.s
>/dev/null 2>&1; then
-                        if grep '\.hidden.*foo_hidden' conftest.s >/dev/null &&
! grep '\.hidden.*foo_default' conftest.s > /dev/null ; then
+                        if grep '\.hidden.*foo_hidden' conftest.s >/dev/null &&
!grep '\.hidden.*foo_default' conftest.s > /dev/null ; then
                           ac_cv_visibility_pragma=yes
                         fi
                       fi
                       rm -f conftest.[cs]
                     ])
       if test $ac_cv_visibility_pragma = yes; then
         AC_DEFINE(HAVE_VISIBILITY_PRAGMA)
         VISIBILITY_FLAGS='-I$(DIST)/include/system_wrappers -include
$(topsrcdir)/config/gcc_hidden.h'

Index: configure.in
===================================================================
RCS file: /cvsroot/mozilla/nsprpub/configure.in,v
retrieving revision 1.83.2.95
diff -u -8 -r1.83.2.95 configure.in
--- configure.in	15 Dec 2004 05:52:39 -0000	1.83.2.95
+++ configure.in	22 Dec 2004 13:40:26 -0000
@@ -511,17 +511,17 @@
                      [cat > conftest.c <<EOF
 #pragma GCC visibility push(hidden)
                       int foo_hidden = 1;
 #pragma GCC visibility push(default)
                       int foo_default = 1;
 EOF
                       ac_cv_visibility_pragma=no
                       if ${CC-cc} -Werror -S conftest.c -o conftest.s
>/dev/null 2>&1; then
-                        if grep '\.hidden.*foo_hidden' conftest.s >/dev/null &&
! grep '\.hidden.*foo_default' conftest.s > /dev/null ; then
+                        if grep '\.hidden.*foo_hidden' conftest.s >/dev/null &&
!grep '\.hidden.*foo_default' conftest.s > /dev/null ; then
                           ac_cv_visibility_pragma=yes
                         fi
                       fi
                       rm -f conftest.[cs]
                     ])
      if test $ac_cv_visibility_pragma = yes; then
        AC_DEFINE(HAVE_VISIBILITY_PRAGMA)
        VISIBILITY_FLAGS='-I$(dist_includedir)/system_wrappers -include
$(topsrcdir)/config/gcc_hidden.h'
************************************End*************************************
No, the space is required.
fixed by splitting into two ifs.
Assignee: dbaron → nobody
Component: Tinderbox Platforms → Build Config
Product: mozilla.org → Mozilla Application Suite
QA Contact: mcafee → core.build-config
Version: other → Trunk
Assignee: nobody → dbaron
Status: UNCONFIRMED → NEW
Ever confirmed: true
Status: NEW → RESOLVED
Last Resolved: 14 years ago
Resolution: --- → FIXED

Comment 4

14 years ago
Not yet. You need to make the same change in nsprpub/configure.in.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Status: REOPENED → RESOLVED
Last Resolved: 14 years ago14 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.