333406 - Fail to build seamonkey on solaris sparc.

Status: RESOLVED FIXED

(Directory :: LDAP C SDK, defect)

Description

[Leon Sha]

cc -G -h libplc4.so -z combreloc -z defs -z ignore -o libplc4.so -M ./plcmap.sun -R '$ORIGIN' ./plvrsion.o ./strlen.o ./strcpy.o ./strdup.o ./strcat.o ./strcmp.o ./strccmp.o ./strchr.o ./strpbrk.o ./strstr.o ./strcstr.o ./strtok.o ./base64.o ./plerror.o ./plgetopt.o   -L/export/home/tinderbox/builds/seamonkey-ports/SunOS_5.10_Clobber/mozilla/dist/lib -lnspr4 -lc
../../../config/./nsinstall -R -m 444 ./libplc4.a ./libplc4.so /export/home/tinderbox/builds/seamonkey-ports/SunOS_5.10_Clobber/mozilla/dist/lib
../../../config/./nsinstall -R -m 444 ./libplc4.so /export/home/tinderbox/builds/seamonkey-ports/SunOS_5.10_Clobber/mozilla/dist/bin
gmake[6]: Leaving directory `/export/home/tinderbox/builds/seamonkey-ports/SunOS_5.10_Clobber/mozilla/nsprpub/lib/libc/src'
gmake[5]: Leaving directory `/export/home/tinderbox/builds/seamonkey-ports/SunOS_5.10_Clobber/mozilla/nsprpub/lib/libc'
gmake[4]: Leaving directory `/export/home/tinderbox/builds/seamonkey-ports/SunOS_5.10_Clobber/mozilla/nsprpub/lib'
gmake[3]: Leaving directory `/export/home/tinderbox/builds/seamonkey-ports/SunOS_5.10_Clobber/mozilla/nsprpub'
gmake[2]: Leaving directory `/export/home/tinderbox/builds/seamonkey-ports/SunOS_5.10_Clobber/mozilla'
/export/home/tinderbox/sysbin/bin/gmake ldap
gmake[2]: Entering directory `/export/home/tinderbox/builds/seamonkey-ports/SunOS_5.10_Clobber/mozilla'
/export/home/tinderbox/sysbin/bin/gmake -C directory/c-sdk
gmake[3]: Entering directory `/export/home/tinderbox/builds/seamonkey-ports/SunOS_5.10_Clobber/mozilla/directory/c-sdk'
cd config; /export/home/tinderbox/sysbin/bin/gmake export
gmake[4]: Entering directory `/export/home/tinderbox/builds/seamonkey-ports/SunOS_5.10_Clobber/mozilla/directory/c-sdk/config'
cc -xarch=v9 -xstrconst -o now.o -c     -I/usr/openwin/include -I/usr/sfw/include -I/usr/sfw/include/freetype2 -I/usr/openwin/include -I/usr/sfw/include -I/usr/sfw/include/freetype2 -O -KPIC  -UDEBUG -DPACKAGE_NAME=\"\" -DPACKAGE_TARNAME=\"\" -DPACKAGE_VERSION=\"\" -DPACKAGE_STRING=\"\" -DPACKAGE_BUGREPORT=\"\" -DMOZILLA_CLIENT=1 -DNDEBUG=1 -DXP_UNIX=1 -DSVR4=1 -DSYSV=1 -D__svr4=1 -D__svr4__=1 -DSOLARIS=1 -DHAVE_FCNTL_FILE_LOCKING=1 -D_PR_HAVE_OFF64_T=1 -DHAVE_LCHOWN=1 -DHAVE_STRERROR=1 -D_REENTRANT=1 -DHAVE_POINTER_LOCALTIME_R=1  -DFORCE_PR_LOG -D_PR_PTHREADS -UHAVE_CVAR_BUILT_ON_SEM   now.c
cc -xarch=v9 -xstrconst  now.o  -o now
cc -xarch=v9 -xstrconst     -I/usr/openwin/include -I/usr/sfw/include -I/usr/sfw/include/freetype2 -I/usr/openwin/include -I/usr/sfw/include -I/usr/sfw/include/freetype2 -O -KPIC  -UDEBUG -DPACKAGE_NAME=\"\" -DPACKAGE_TARNAME=\"\" -DPACKAGE_VERSION=\"\" -DPACKAGE_STRING=\"\" -DPACKAGE_BUGREPORT=\"\" -DMOZILLA_CLIENT=1 -DNDEBUG=1 -DXP_UNIX=1 -DSVR4=1 -DSYSV=1 -D__svr4=1 -D__svr4__=1 -DSOLARIS=1 -DHAVE_FCNTL_FILE_LOCKING=1 -D_PR_HAVE_OFF64_T=1 -DHAVE_LCHOWN=1 -DHAVE_STRERROR=1 -D_REENTRANT=1 -DHAVE_POINTER_LOCALTIME_R=1  -DFORCE_PR_LOG -D_PR_PTHREADS -UHAVE_CVAR_BUILT_ON_SEM    -lpthread -M /usr/lib/ld/map.noexstk -xildoff -z lazyload -z combreloc  -R/usr/openwin/lib -R/usr/sfw/lib -R/usr/openwin/lib:/usr/openwin/sfw/lib -L/usr/openwin/lib -L/usr/sfw/lib -L/usr/openwin/sfw/lib -lXft -lfreetype -lXrender -lfontconfig -R/usr/openwin/lib -R/usr/sfw/lib -R/usr/openwin/lib:/usr/openwin/sfw/lib -L/usr/openwin/lib -L/usr/sfw/lib -L/usr/openwin/sfw/lib -lXft -lfreetype -lXrender -lfontconfig -R/usr/openwin/lib -R/usr/sfw/lib -R/usr/openwin/lib:/usr/openwin/sfw/lib -L/usr/openwin/lib -L/usr/sfw/lib -L/usr/openwin/sfw/lib -lXft -lfreetype -lXrender -lfontconfig -R/usr/openwin/lib -R/usr/sfw/lib -R/usr/openwin/lib:/usr/openwin/sfw/lib -L/usr/openwin/lib -L/usr/sfw/lib -L/usr/openwin/sfw/lib -lXft -lfreetype -lXrender -lfontconfig -z ignore -R 'RIGIN:RIGIN/..'    nsinstall.c   -o nsinstall
ld: fatal: file /usr/sfw/lib/libfreetype.so: wrong ELF class: ELFCLASS32
ld: fatal: file /usr/sfw/lib/libXrender.so: wrong ELF class: ELFCLASS32
ld: fatal: file /usr/sfw/lib/libfreetype.so: wrong ELF class: ELFCLASS32
ld: fatal: file /usr/sfw/lib/libXrender.so: wrong ELF class: ELFCLASS32
ld: fatal: file /usr/sfw/lib/libfreetype.so: wrong ELF class: ELFCLASS32
ld: fatal: file /usr/sfw/lib/libXrender.so: wrong ELF class: ELFCLASS32
ld: fatal: file /usr/sfw/lib/libfreetype.so: wrong ELF class: ELFCLASS32
ld: fatal: file /usr/sfw/lib/libXrender.so: wrong ELF class: ELFCLASS32
ld: fatal: File processing errors. No output written to nsinstall
gmake[4]: *** [nsinstall] Error 1
gmake[4]: Leaving directory `/export/home/tinderbox/builds/seamonkey-ports/SunOS_5.10_Clobber/mozilla/directory/c-sdk/config'
gmake[3]: *** [export] Error 2
gmake[3]: Leaving directory `/export/home/tinderbox/builds/seamonkey-ports/SunOS_5.10_Clobber/mozilla/directory/c-sdk'
gmake[2]: *** [ldap] Error 2
gmake[2]: Leaving directory `/export/home/tinderbox/builds/seamonkey-ports/SunOS_5.10_Clobber/mozilla'
gmake[1]: *** [alldep] Error 2
gmake[1]: Leaving directory `/export/home/tinderbox/builds/seamonkey-ports/SunOS_5.10_Clobber/mozilla'
gmake: *** [alldep] Error 2

Comment 1

[Leon Sha]

This is caused by check in of bug 325625.

Comment 2

[Leon Sha]

Attachment: Patch

We'd better let user to set the option if they want to build a 64 bit application.

Comment 3

[Rich Megginson]

You can also do configure --disable-64bit to force it to build 32 bit binaries on a 64 bit platform.  I don't know if is easier than the proposed diff - it just depends on how hard it is to modify the build process.  If the diff is the right way to go, then we also need to remove the previous line that sets the build to automatically use 64 bits on other 64 bit arches, such as itanium and x86_64.

Comment 4

[Mark Smith [:mcs]]

How is this issue handled for NSPR and NSS?

Comment 5

[Rich Megginson]

Given the error report, I suspect that with nspr and nss you use configure --enable-64bit and/or make USE_64=1 to force 64 bit builds on those platforms that support 64 bit, and just default to 32 bit if the platform supports both 32 bit and 64 bit (e.g. Solaris, HPUX, others?).

So, given that, perhaps the best option is to make ldapcsdk work like nspr and nss.

Comment 6

[Mark Banner (:standard8)]

(In reply to comment #5)
> Given the error report, I suspect that with nspr and nss you use configure
> --enable-64bit and/or make USE_64=1 to force 64 bit builds on those platforms
> that support 64 bit, and just default to 32 bit if the platform supports both
> 32 bit and 64 bit (e.g. Solaris, HPUX, others?).
> 
> So, given that, perhaps the best option is to make ldapcsdk work like nspr and
> nss.
> 
The main configure script also seems to expect to be able to use --enable-64bit, for configuring ldap see http://lxr.mozilla.org/seamonkey/source/configure.in#7738

Comment 7

[Rich Megginson]

*** Bug 333224 has been marked as a duplicate of this bug. ***

Comment 8

[Mark Smith [:mcs]]

Reassigned to Rich.

Comment 9

[Rich Megginson]

Attachment: diffs for fix

This supersedes the previous diff - it removes auto-64bit support for all platforms, in accordance with nspr and nss builds.

Comment 10

[Rich Megginson]

Can I get a review of my proposed patch - https://bugzilla.mozilla.org/attachment.cgi?id=217964 ?

Comment 11

[Mark Smith [:mcs]]

Comment on attachment 217964 [details]
diffs for fix

Looks OK to me.

Comment 12

[Rich Megginson]

Fixed.  Mark, you can either reassign to me or resolve it.

Checked in fix to HEAD:
Checking in configure;
/cvsroot/mozilla/directory/c-sdk/configure,v  <--  configure
new revision: 5.49; previous revision: 5.48
done
Checking in configure.in;
/cvsroot/mozilla/directory/c-sdk/configure.in,v  <--  configure.in
new revision: 5.49; previous revision: 5.48
done

Checked in fix to ldapcsdk_5_17_client_branch:
Checking in configure;
/cvsroot/mozilla/directory/c-sdk/configure,v  <--  configure
new revision: 5.47.2.2; previous revision: 5.47.2.1
done
Checking in configure.in;
/cvsroot/mozilla/directory/c-sdk/configure.in,v  <--  configure.in
new revision: 5.47.2.2; previous revision: 5.47.2.1
done

Comment 13

[Mark Smith [:mcs]]

-> Rich.

Comment 14

[Mark Smith [:mcs]]

I am not sure it is a big deal in this case, but before landing on ldapcsdk_5_17_client_branch a super review should be obtained (any code that lands on the what is effectively the trunk for TBird, Firefox, etc. requires an sr).

Comment 15

[Rich Megginson]

Ok, sorry about that.  Mark B., can you superreview, or should I get Dan M.?