Repository.mk                              |    2 +-
 external/Module_external.mk                |    7 -------
 solenv/gbuild/Helper.mk                    |    1 +
 solenv/gbuild/Module.mk                    |    2 ++
 solenv/gbuild/extensions/pre_BuildTools.mk |    3 +++
 5 files changed, 7 insertions(+), 8 deletions(-)

New commits:
commit 52d409f0d657f314a53f945c9ffb5b8025bea034
Author: Matúš Kukan <matus.ku...@gmail.com>
Date:   Wed Nov 30 14:36:51 2016 +0100

    gbuild: Prevent wrong use of gb_Helper_optional_for_host
    
    It's empty, unless gb_Side=build.
    This fixes commit 1193443b15b0e38cbfb41573b31acf9a8d90ce2c
    
    Change-Id: I83efad36d8f40c460d09ee1bd1addd7fb26c177b
    Reviewed-on: https://gerrit.libreoffice.org/31419
    Tested-by: Jenkins <c...@libreoffice.org>
    Reviewed-by: Michael Stahl <mst...@redhat.com>

diff --git a/Repository.mk b/Repository.mk
index 6ed227c..33bf6d0 100644
--- a/Repository.mk
+++ b/Repository.mk
@@ -992,7 +992,7 @@ $(eval $(call gb_Helper_register_resources,\
        eur \
        for \
        forui \
-       $(call gb_Helper_optional_for_host,DESKTOP,fps_office) \
+       $(call gb_Helper_optional,DESKTOP,fps_office) \
        frm \
        fwe \
        gal \
diff --git a/external/Module_external.mk b/external/Module_external.mk
index 3b54b52..1dbf97b 100644
--- a/external/Module_external.mk
+++ b/external/Module_external.mk
@@ -101,13 +101,6 @@ $(eval $(call gb_Module_add_moduledirs,external,\
        $(call gb_Helper_optional,XSLTML,xsltml) \
        $(call gb_Helper_optional,ZLIB,zlib) \
        $(call gb_Helper_optional,ZMF,libzmf) \
-       $(call gb_Helper_optional_for_host,DESKTOP, \
-               $(if $(WITH_GALLERY_BUILD), \
-                       $(call gb_Helper_optional,HARFBUZZ,harfbuzz) \
-                       $(call gb_Helper_optional,LCMS2,lcms2) \
-                       $(call gb_Helper_optional,NSS,nss) \
-               ) \
-       ) \
 ))
 
 # vim: set noet sw=4 ts=4:
diff --git a/solenv/gbuild/Helper.mk b/solenv/gbuild/Helper.mk
index 4e914d1..1464411 100644
--- a/solenv/gbuild/Helper.mk
+++ b/solenv/gbuild/Helper.mk
@@ -299,6 +299,7 @@ endef
 endif
 
 define gb_Helper_optional_for_host
+$(if $(filter build,$(gb_Side)),,$(call 
gb_Output_error,gb_Helper_optional_for_host: Use only when gb_Side=build))
 $(if $(filter $(1),$(BUILD_TYPE_FOR_HOST)),$(2))
 endef
 
diff --git a/solenv/gbuild/Module.mk b/solenv/gbuild/Module.mk
index eb558a8..6af2b5c 100644
--- a/solenv/gbuild/Module.mk
+++ b/solenv/gbuild/Module.mk
@@ -144,10 +144,12 @@ $(call gb_Module_get_target,%) :
 
 all : build $(if $(CROSS_COMPILING),,unitcheck $(if 
$(gb_PARTIAL_BUILD),,slowcheck))
 
+ifeq ($(gb_Side),build)
 build-tools : $(gb_BUILD_TOOLS)
        $(call gb_Output_announce,loaded tools: $(gb_BUILD_TOOLS),$(true),ALL,6)
        $(call gb_Output_announce_title,build-tools done.)
        $(call gb_Output_announce_bell)
+endif
 
 build :
        $(call gb_Output_announce,top level modules: $(foreach 
module,$(filter-out deliverlog $(WORKDIR)/bootstrap,$^),$(notdir 
$(module))),$(true),ALL,6)
diff --git a/solenv/gbuild/extensions/pre_BuildTools.mk 
b/solenv/gbuild/extensions/pre_BuildTools.mk
index f387df6..5daf671 100644
--- a/solenv/gbuild/extensions/pre_BuildTools.mk
+++ b/solenv/gbuild/extensions/pre_BuildTools.mk
@@ -8,6 +8,7 @@
 #
 
 # Tools we need to build for cross-compiling
+ifeq ($(gb_Side),build)
 gb_BUILD_TOOLS = \
        $(foreach executable, \
                bestreversemap \
@@ -44,4 +45,6 @@ gb_BUILD_TOOLS = \
        ,$(call gb_ExternalExecutable_get_dependencies,$(executable))) \
        $(INSTROOT)/$(LIBO_URE_ETC_FOLDER)/$(call gb_Helper_get_rcfile,uno) \
 
+endif
+
 # vim: set noet sw=4 ts=4:
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to