Björn,

What good is that backport to 3.4? For one, the fix turns out to not work yet (as executables already built using gbuild but only used during the build itself use a "NONEBIN" token in solenv/gbuild/platform/*.mk instead of the "NONE" token used in solenv/inc/*.mk, so that macosx-change-install-names.pl complains and fails). For another, are there any executables in 3.4 that are built using gbuild but are not only used during the build itself (i.e., for which the missing call to macosx-change-install-names.pl actually matters)?

-Stephan

On 09/19/2011 11:06 PM, Bjoern Michaelsen wrote:
  solenv/gbuild/platform/macosx.mk |    2 ++
  1 file changed, 2 insertions(+)

New commits:
commit a87a8ee5a1919f6f6b4b35668456c03bdd499c85
Author: Stephan Bergmann<sberg...@redhat.com>
Date:   Mon Sep 19 21:39:46 2011 +0200

     Call macosx-change-install-names on executables in gbuild, too.

     Signed-off-by: Bjoern Michaelsen<bjoern.michael...@canonical.com>

diff --git a/solenv/gbuild/platform/macosx.mk b/solenv/gbuild/platform/macosx.mk
index 2def0c9..ff73333 100755
--- a/solenv/gbuild/platform/macosx.mk
+++ b/solenv/gbuild/platform/macosx.mk
@@ -274,6 +274,8 @@ $(call gb_Helper_abbreviate_dirs,\
                $(foreach lib,$(LINKED_STATIC_LIBS),$(call 
gb_StaticLibrary_get_target,$(lib))) \
                -o $(1) \
                `cat $${DYLIB_FILE}`&&  \
+       $(if $(filter Executable,$(TARGETTYPE)),\
+               $(PERL) $(SOLARENV)/bin/macosx-change-install-names.pl Executable 
$(LAYER) $(1)&&) \
        $(if $(filter Library CppunitTest,$(TARGETTYPE)),\
                $(PERL) $(SOLARENV)/bin/macosx-change-install-names.pl Library $(LAYER) 
$(1)&&  \
                ln -sf $(1) $(patsubst %.dylib,%.jnilib,$(1))&&) \
_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice

Reply via email to