solenv/gbuild/ExtensionTarget.mk | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-)
New commits: commit 751f3aea6805140775bb6ad28f92a1f2795373a6 Author: David Tardon <dtar...@redhat.com> Date: Sun Sep 23 16:29:10 2012 +0200 fix build of dictionaries with l10n enabled It seems that foo : BAR := xyz foo $$(BAR) does not really work, i.e., it does not generate a dep on xyz, but $$(BAR) expands into an empty string. It is unclear to me whether it is a bug in make or it does not work by design. Info documentation for make only says that target-specific variable _assignments_ are evaluated in the context of the target, so previously defined target-specific variables shall be in scope. There is no mention whether this should work for use of such a variable too (like in the example above). I have always expected it does; experimental evidence shows it does not .-) Change-Id: If99bdcacea403aa925127dd993a7e94ed7d57ae4 diff --git a/solenv/gbuild/ExtensionTarget.mk b/solenv/gbuild/ExtensionTarget.mk index d68f79e..7c5d3d3 100644 --- a/solenv/gbuild/ExtensionTarget.mk +++ b/solenv/gbuild/ExtensionTarget.mk @@ -132,7 +132,7 @@ $(call gb_ExtensionTarget_get_target,$(1)) : PRJNAME := $(firstword $(subst /, , $(call gb_ExtensionTarget_get_workdir,$(1))/description.xml : $(SRCDIR)/$(2)/description.xml ifneq ($(strip $(gb_WITH_LANG)),) $(call gb_ExtensionTarget_get_target,$(1)) : SDF := $(gb_SDFLOCATION)/$(2)/localize.sdf -$(call gb_ExtensionTarget_get_workdir,$(1))/description.xml : $$(SDF) +$(call gb_ExtensionTarget_get_workdir,$(1))/description.xml : $(gb_SDFLOCATION)/$(2)/localize.sdf endif $(foreach lang,$(gb_ExtensionTarget_ALL_LANGS), \ @@ -219,7 +219,7 @@ $(call gb_ExtensionTarget_get_target,$(1)) : FILES += $(2) ifneq ($(strip $(gb_WITH_LANG)),) $(call gb_ExtensionTarget_get_target,$(1)) : FILES += $(foreach lang,$(subst -,_,$(gb_ExtensionTarget_TRANS_LANGS)),$(subst en_US,$(lang),$(2))) $(call gb_ExtensionTarget_get_rootdir,$(1))/$(2) : SDF := $(gb_SDFLOCATION)$(subst $(SRCDIR),,$(dir $(3)))localize.sdf -$(call gb_ExtensionTarget_get_rootdir,$(1))/$(2) : $$(SDF) +$(call gb_ExtensionTarget_get_rootdir,$(1))/$(2) : $(gb_SDFLOCATION)$(subst $(SRCDIR),,$(dir $(3)))localize.sdf endif $(call gb_ExtensionTarget_get_target,$(1)) : $(call gb_ExtensionTarget_get_rootdir,$(1))/$(2) $(call gb_ExtensionTarget_get_rootdir,$(1))/$(2) : $(3) \ @@ -288,7 +288,8 @@ $(call gb_ExtensionTarget_get_rootdir,$(1))/help/$(5).done : \ $(call gb_ExtensionTarget_get_workdir,$(1))/help/$(5)/$(3) $(call gb_ExtensionTarget_get_workdir,$(1))/help/$(5)/$(3) : \ SDF := $(gb_SDFLOCATION)$(subst $(SRCDIR),,$(subst $(WORKDIR)/CustomTarget,,$(2)/$(dir $(or $(4),$(3)))))localize.sdf -$(call gb_ExtensionTarget_get_workdir,$(1))/help/$(5)/$(3) : $$(SDF) +$(call gb_ExtensionTarget_get_workdir,$(1))/help/$(5)/$(3) : \ + $(gb_SDFLOCATION)$(subst $(SRCDIR),,$(subst $(WORKDIR)/CustomTarget,,$(2)/$(dir $(or $(4),$(3)))))localize.sdf $(call gb_ExtensionTarget_get_workdir,$(1))/help/$(5)/$(3) : \ $(if $(filter-out en-US,$(5)),$(gb_ExtensionTarget_HELPEXTARGET)) | \ $(call gb_ExtensionTarget_get_workdir,$(1))/help/.dir @@ -320,7 +321,8 @@ $(call gb_ExtensionTarget_get_rootdir,$(1))/help/$(5).done : \ $(call gb_ExtensionTarget_get_rootdir,$(1))/help/$(5)/$(3) $(call gb_ExtensionTarget_get_rootdir,$(1))/help/$(5)/$(3) : \ SDF := $(gb_SDFLOCATION)$(subst $(SRCDIR),,$(subst $(WORKDIR)/CustomTarget,,$(2)/$(dir $(or $(4),$(3)))))localize.sdf -$(call gb_ExtensionTarget_get_rootdir,$(1))/help/$(5)/$(3) : $$(SDF) +$(call gb_ExtensionTarget_get_rootdir,$(1))/help/$(5)/$(3) : \ + $(gb_SDFLOCATION)$(subst $(SRCDIR),,$(subst $(WORKDIR)/CustomTarget,,$(2)/$(dir $(or $(4),$(3)))))localize.sdf $(call gb_ExtensionTarget_get_rootdir,$(1))/help/$(5)/$(3) : \ $(if $(filter-out en-US,$(WITH_LANG)),$(gb_ExtensionTarget_UPDATETREETARGET)) | \ $(2)/$(4) _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits