[Libreoffice-commits] .: config_host.mk.in configure.ac Makefile.in nss/ExternalProject_nss.mk nss/Module_nss.mk nss/nsinstall.py

Libreoffice Gerrit user logerrit at kemper.freedesktop.org
Mon Jan 7 01:22:35 PST 2013


 Makefile.in                |    7 -------
 config_host.mk.in          |    1 -
 configure.ac               |   17 -----------------
 nss/ExternalProject_nss.mk |   11 +++++------
 nss/Module_nss.mk          |    2 +-
 nss/nsinstall.py           |    6 +++---
 6 files changed, 9 insertions(+), 35 deletions(-)

New commits:
commit 6eff6fae6272e8ed5d74553597bb4b73db3f2864
Author: Peter Foley <pefoley2 at verizon.net>
Date:   Wed Jan 2 18:10:13 2013 -0500

    Get rid of nsinstall hackery
    
    This removes the need for using NSS Build Tools on windows.
    It also removes the nees to build nss for the build system while cross
    compiling.
    
    Change-Id: I13c9fdb575223f2940d3e4eda00e77ba9158f2b7
    Reviewed-on: https://gerrit.libreoffice.org/1534
    Reviewed-by: Tor Lillqvist <tml at iki.fi>
    Tested-by: Tor Lillqvist <tml at iki.fi>

diff --git a/Makefile.in b/Makefile.in
index fb00c38..f0b5ce1 100644
--- a/Makefile.in
+++ b/Makefile.in
@@ -301,13 +301,6 @@ cross-toolset: bootstrap fetch
 	$(GNUMAKE) gb_Side=build icu
 #
 	cd cross_toolset && $(GNUMAKE) -j $(PARALLELISM) $(GMAKE_OPTIONS)
-#
-ifneq (,$(filter DESKTOP,$(BUILD_TYPE)))
-ifneq (WNT,$(OS))
-# We need to build nss for nsinstall... See NSINSTALL=... in nss/makefile.mk
-	$(GNUMAKE) gb_Side=build nss
-endif
-endif
 
 #
 # Install
diff --git a/config_host.mk.in b/config_host.mk.in
index 70f8c4e..1c98744 100644
--- a/config_host.mk.in
+++ b/config_host.mk.in
@@ -411,7 +411,6 @@ export NSS_MINOR=@NSS_MINOR@
 export NSS_PATCH=@NSS_PATCH@
 export NSS_CFLAGS=$(gb_SPACE)@NSS_CFLAGS@
 export NSS_LIBS=$(gb_SPACE)@NSS_LIBS@
-export NSSBUILDTOOLS=@NSSBUILDTOOLS@
 export NUMBERTEXT_EXTENSION_PACK=@NUMBERTEXT_EXTENSION_PACK@
 @x_OBJCFLAGS@ export OBJCFLAGS=@OBJCFLAGS@
 @x_OBJCXXFLAGS@ export OBJCXXFLAGS=@OBJCXXFLAGS@
diff --git a/configure.ac b/configure.ac
index 29cc392..0a1fd05 100644
--- a/configure.ac
+++ b/configure.ac
@@ -8287,22 +8287,6 @@ else
 	NSS_PATCH="5"
     BUILD_TYPE="$BUILD_TYPE NSS"
     AC_MSG_RESULT([internal])
-    if test "$build_os" = "cygwin"; then
-        AC_MSG_CHECKING([for nss build tooling]) # coming from mozilla
-        if test -z "$NSSBUILDTOOLS"; then
-            AC_MSG_ERROR([nss build tooling not provided
-Use the --with-nss-build-tools option after installing the tools obtained
-from http://ftp.mozilla.org/pub/mozilla.org/mozilla/libraries/win32])
-        else
-            if test \( "$WITH_MINGW" = "yes" -a  ! -d "$NSSBUILDTOOLS" \) \
-                    -o ! -d "$NSSBUILDTOOLS/moztools" \
-                    -o ! -d "$NSSBUILDTOOLS/msys" ; then
-                AC_MSG_ERROR([nss build tooling incomplete!])
-            else
-                AC_MSG_RESULT([ok])
-            fi
-        fi
-    fi
 fi # system nss
 AC_SUBST(SYSTEM_NSS)
 AC_SUBST(NSS_MAJOR)
@@ -8310,7 +8294,6 @@ AC_SUBST(NSS_MINOR)
 AC_SUBST(NSS_PATCH)
 AC_SUBST(NSS_CFLAGS)
 AC_SUBST(NSS_LIBS)
-AC_SUBST(NSSBUILDTOOLS)
 AC_SUBST(MINGW_SMIME3_DLL)
 
 dnl ===================================================================
diff --git a/nss/ExternalProject_nss.mk b/nss/ExternalProject_nss.mk
index 330a6e6..7ddbcae 100644
--- a/nss/ExternalProject_nss.mk
+++ b/nss/ExternalProject_nss.mk
@@ -34,15 +34,15 @@ $(call gb_ExternalProject_get_state_target,nss,configure):
 
 ifeq ($(OS),WNT)
 ifeq ($(COM),MSC)
-$(call gb_ExternalProject_get_state_target,nss,build): $(call gb_ExternalProject_get_state_target,nss,configure)
+$(call gb_ExternalProject_get_state_target,nss,build): $(call gb_ExternalProject_get_state_target,nss,configure) $(call gb_ExternalExecutable_get_dependencies,python)
 	cd $(EXTERNAL_WORKDIR)/mozilla/security/nss \
 	&& $(if $(debug),,BUILD_OPT=1) \
 	MOZ_MSVCVERSION=9 OS_TARGET=WIN95 \
-	PATH="$(NSSBUILDTOOLS)/msys/bin:$(NSSBUILDTOOLS)/moztools/bin:$(PATH)" \
 	$(if $(filter X,$(CPU)),USE_64=1) \
 	LIB="$(ILIB)" \
 	XCFLAGS="$(SOLARINC)" \
 	$(MAKE) -j1 nss_build_all RC="rc.exe $(SOLARINC)" \
+	NSINSTALL='$(call gb_ExternalExecutable_get_command,python) $(SRCDIR)/nss/nsinstall.py' \
 	&& touch $@
 
 
@@ -53,7 +53,7 @@ $(call gb_ExternalProject_get_state_target,nss,build): $(call gb_ExternalProject
 	CXX="$(CXX) $(if $(filter YES,$(MINGW_SHARED_GCCLIB)),-shared-libgcc)" \
 	OS_LIBS="-ladvapi32 -lws2_32 -lmwsock -lwinm $(if $(filter YES,$(MINGW_SHARED_GXXLIB)),$(MINGW_SHARED_LIBSTDCPP))" \
 	OS_TARGET=WINNT RC="$(WINDRES)" OS_RELEASE="5.0" \
-	PATH="$(NSSBUILDTOOLS)/bin:$(PATH)" IMPORT_LIB_SUFFIX=dll.a \
+	IMPORT_LIB_SUFFIX=dll.a \
 	NSPR_CONFIGURE_OPTS="--build=$(BUILD_PLATFORM) --host=$(HOST_PLATFORM) --enable-shared --disable-static" \
 	NSINSTALL="$(PYTHON_FOR_BUILD) $(SRCDIR)/nss/nsinstall.py" \
 	$(MAKE) -j1 nss_build_all \
@@ -69,10 +69,9 @@ $(call gb_ExternalProject_get_state_target,nss,build): $(call gb_ExternalProject
 	$(if $(filter SOLARIS,$(OS)),NS_USE_GCC=1) \
 	$(if $(filter YES,$(CROSS_COMPILING)),\
 	$(if $(filter MACOSXP,$(OS)$(CPU)),CPU_ARCH=ppc) \
-	NSINSTALL="$(subst $(INPATH),$(INPATH_FOR_BUILD),\
-	$(call gb_UnpackedTarball_get_dir,nss)/mozilla/security/coreconf/nsinstall/out/nsinstall)") \
+	NSINSTALL="$(PYTHON_FOR_BUILD) $(SRCDIR)/nss/nsinstall.py") \
 	NSDISTMODE=copy \
-	$(MAKE) -j1 $(if $(filter build,$(gb_Side)),build_coreconf,nss_build_all) \
+	$(MAKE) -j1 nss_build_all \
 	&& touch $@
 
 endif
diff --git a/nss/Module_nss.mk b/nss/Module_nss.mk
index 5455fc0..c1619ff 100644
--- a/nss/Module_nss.mk
+++ b/nss/Module_nss.mk
@@ -13,7 +13,7 @@ ifeq ($(SYSTEM_NSS),NO)
 ifeq ($(filter ANDROID IOS,$(OS)),)
 $(eval $(call gb_Module_add_targets,nss,\
 	UnpackedTarball_nss \
-	$(if $(filter build,$(gb_Side)),,ExternalPackage_nss) \
+	ExternalPackage_nss \
 	ExternalProject_nss \
 ))
 endif
diff --git a/nss/nsinstall.py b/nss/nsinstall.py
index 39d26fe..e1728a9 100644
--- a/nss/nsinstall.py
+++ b/nss/nsinstall.py
@@ -86,11 +86,11 @@ def nsinstall(argv):
   if options.m:
     # mode is specified
     try:
-      options.m = int(options.m, 8)
       # I have no idea why nss insists on using this mode for installed headers.
       # It causes problems with updating the files during a rebuild.
-      if options.m == 0444:
-        options.m = 0644
+      if options.m == 444:
+        options.m = 644
+      options.m = int(options.m)
     except:
       sys.stderr.write('nsinstall: ' + options.m + ' is not a valid mode\n')
       return 1


More information about the Libreoffice-commits mailing list