[Libreoffice-commits] core.git: instsetoo_native/CustomTarget_install.mk solenv/bin

[Jan Holesovsky]

 instsetoo_native/CustomTarget_install.mk |    8 +++++---
 solenv/bin/ooinstall                     |    5 ++++-
 2 files changed, 9 insertions(+), 4 deletions(-)

New commits:
commit 010a2a8ed3214380d514be56171c974f99010bc1
Author: Jan Holesovsky <ke...@collabora.com>
Date:   Tue Aug 2 09:15:37 2016 +0100

    Don't use spaces in the product names when packaging.
    
    Change-Id: I55f332117cc7530eb6d562846db379d95e628a22

diff --git a/instsetoo_native/CustomTarget_install.mk 
b/instsetoo_native/CustomTarget_install.mk
index 0fb97da..59a4500 100644
--- a/instsetoo_native/CustomTarget_install.mk
+++ b/instsetoo_native/CustomTarget_install.mk
@@ -59,6 +59,8 @@ export LOCAL_COMMON_OUT := $(instsetoo_OUT)
 
 instsetoo_native_WITH_LANG := en-US $(filter-out en-US,$(gb_WITH_LANG))
 
+PRODUCTNAME_no_spaces := $(subst $(WHITESPACE),,$(PRODUCTNAME))
+
 ifeq (WNT,$(OS))
 define instsetoo_native_msitemplates
 
@@ -80,7 +82,7 @@ $(foreach pkgformat,$(5),\
 && $(PERL) -w $< \
        -f $(BUILDDIR)/instsetoo_native/util/openoffice.lst \
        -l $(subst $(WHITESPACE),$(COMMA),$(strip $(2))) \
-       -p $(PRODUCTNAME)$(3) \
+       -p $(PRODUCTNAME_no_spaces)$(3) \
        -u $(instsetoo_OUT) \
        -buildid $(if $(filter deb0 
rpm0,$(pkgformat)$(LIBO_VERSION_PATCH)),1,$(LIBO_VERSION_PATCH)) \
        $(if $(filter WNT,$(OS)), \
@@ -98,13 +100,13 @@ $(call 
gb_CustomTarget_get_workdir,instsetoo_native/install)/install.phony:
        rm -rf $(instsetoo_OUT)
 ifeq (TRUE,$(LIBO_TEST_INSTALL))
        $(call instsetoo_native_install_command,openoffice,en-US,,,archive)
-       unzip -q -d $(TESTINSTALLDIR) 
$(instsetoo_OUT)/$(PRODUCTNAME)/archive/install/en-US/LibreOffice*_archive.zip
+       unzip -q -d $(TESTINSTALLDIR) 
$(instsetoo_OUT)/$(PRODUCTNAME_no_spaces)/archive/install/en-US/LibreOffice*_archive.zip
        mv $(TESTINSTALLDIR)/LibreOffice*_archive/LibreOffice*/* 
$(TESTINSTALLDIR)/
        rmdir $(TESTINSTALLDIR)/LibreOffice*_archive/LibreOffice*
        rmdir $(TESTINSTALLDIR)/LibreOffice*_archive
 ifeq (ODK,$(filter ODK,$(BUILD_TYPE)))
        $(call instsetoo_native_install_command,sdkoo,en-US,_SDK,,archive)
-       unzip -q -d $(TESTINSTALLDIR) 
$(instsetoo_OUT)/$(PRODUCTNAME)_SDK/archive/install/en-US/LibreOffice*_archive_sdk.zip
+       unzip -q -d $(TESTINSTALLDIR) 
$(instsetoo_OUT)/$(PRODUCTNAME_no_spaces)_SDK/archive/install/en-US/LibreOffice*_archive_sdk.zip
        mv $(TESTINSTALLDIR)/LibreOffice*_archive_sdk/LibreOffice*_SDK/sdk \
         $(TESTINSTALLDIR)/
        rmdir $(TESTINSTALLDIR)/LibreOffice*_archive_sdk/LibreOffice*_SDK
diff --git a/solenv/bin/ooinstall b/solenv/bin/ooinstall
index f9034cd..edd3d83 100755
--- a/solenv/bin/ooinstall
+++ b/solenv/bin/ooinstall
@@ -89,11 +89,14 @@ if ($destdir && "$ENV{DESTDIR}" ne "/" && -d 
"$ENV{DESTDIR}") {
 
 print "Running LibreOffice installer\n";
 
+my $PRODUCTNAME_no_spaces = $ENV{PRODUCTNAME};
+$PRODUCTNAME_no_spaces =~ s/ //g;
+
 system ("cd $ENV{SRC_ROOT}/instsetoo_native/util ; " .
         "perl " .
         (scalar keys(%DB::sub) ? "-d " : "") .
         "-w $ENV{SRCDIR}/solenv/bin/make_installer.pl " .
-        "-f $ENV{BUILDDIR}/instsetoo_native/util/openoffice.lst -l $langs -p 
$ENV{PRODUCTNAME}" . " " .
+        "-f $ENV{BUILDDIR}/instsetoo_native/util/openoffice.lst -l $langs -p 
$PRODUCTNAME_no_spaces " .
         "-u $tmp_dir " .
         "-buildid $BUILD $destdir $strip $msi " .
         "-simple $path") && die "Failed to install: $!";
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits