instsetoo_native/CustomTarget_install.mk |    8 +++++---
 instsetoo_native/util/openoffice.lst.in  |    2 +-
 solenv/bin/ooinstall                     |    5 ++++-
 3 files changed, 10 insertions(+), 5 deletions(-)

New commits:
commit 4d1e53bcbd8af9655ea5d6b1cd5b03436b25feb7
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 582f797..c5ac2b0 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 LibreOffice$(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)/LibreOffice/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)/LibreOffice_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/instsetoo_native/util/openoffice.lst.in 
b/instsetoo_native/util/openoffice.lst.in
index 505c1a4..77343f3 100644
--- a/instsetoo_native/util/openoffice.lst.in
+++ b/instsetoo_native/util/openoffice.lst.in
@@ -39,7 +39,7 @@ Globals
     }
 }
 
-Collabora Office
+CollaboraOffice
 {
     Settings
     {
diff --git a/solenv/bin/ooinstall b/solenv/bin/ooinstall
index d5aa342..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

Reply via email to