418648 - configure changes required for building on windows mobile

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[Doug Turner (:dougt)]

Attachment: patch v.1

Comment 1

[Benjamin Smedberg]

Comment on attachment 304514 [details] [diff] [review]
patch v.1

>Index: configure.in

>@@ -3901,7 +3919,7 @@
>         NSPR_LIBS='$(LIBXUL_DIST)/lib/nspr'$NSPR_VERSION'.lib $(LIBXUL_DIST)/lib/plc'$NSPR_VERSION'.lib $(LIBXUL_DIST)/lib/plds'$NSPR_VERSION'.lib '$_PTHREAD_LDFLAGS''
>     elif test "$OS_ARCH" = "WINCE"; then
>         NSPR_CFLAGS='-I$(LIBXUL_DIST)/include/nspr'
>-        NSPR_LIBS='$(LIBXUL_DIST)/lib/nspr'$NSPR_VERSION'.lib $(LIBXUL_DIST)/lib/plc'$NSPR_VERSION'.lib $(LIBXUL_DIST)/lib/plds'$NSPR_VERSION'.lib '
>+        NSPR_LIBS='$(LIBXUL_DIST)/lib/libnspr'$NSPR_VERSION'.lib $(LIBXUL_DIST)/lib/libplc'$NSPR_VERSION'.lib $(LIBXUL_DIST)/lib/libplds'$NSPR_VERSION'.lib '
>     elif test "$OS_ARCH" = "WINNT"; then
>         NSPR_CFLAGS='-I$(LIBXUL_DIST)/include/nspr'
>         if test -n "$GNU_CC"; then

I'd prefer to avoid this, if we can get NSPR to produce the same import libraries for WINCE that it does for WINNT.

Comment 2

[Doug Turner (:dougt)]

Comment on attachment 304514 [details] [diff] [review]
patch v.1

i agree.  we do not have to use lib<libraryname>.

Comment 3

[Mike Beltzner [:beltzner, not reading bugmail]]

Comment on attachment 304514 [details] [diff] [review]
patch v.1

a=beltzner for 1.9

Comment 4

[Doug Turner (:dougt)]

Checking in configure.in;
/cvsroot/mozilla/configure.in,v  <--  configure.in
new revision: 1.1934; previous revision: 1.1933
done