Kaydet (Commit) ec8d4b1d authored tarafından Michael Stahl's avatar Michael Stahl

gbuild: use zip --must-match to fail if input files are missing

With the last "find" call to determine inputs removed, this should
hopefully work reliably now.

Change-Id: Ie725e29d0889ec40cefc961ff61b2cf29839fb66
üst 5c57a102
...@@ -291,7 +291,7 @@ $(call gb_CustomTarget_get_workdir,extras/source/autotext)/%.dat : ...@@ -291,7 +291,7 @@ $(call gb_CustomTarget_get_workdir,extras/source/autotext)/%.dat :
$(call gb_Output_announce,$*.dat,$(true),ZIP,2) $(call gb_Output_announce,$*.dat,$(true),ZIP,2)
$(call gb_Helper_abbreviate_dirs,\ $(call gb_Helper_abbreviate_dirs,\
cd $(EXTRAS_AUTOCORR_DIR) && \ cd $(EXTRAS_AUTOCORR_DIR) && \
zip -qrX --filesync $@ $(EXTRAS_AUTOCORR_FILES) \ zip -qrX --filesync --must-match $@ $(EXTRAS_AUTOCORR_FILES) \
) )
define extras_Autocorr_make_file_deps define extras_Autocorr_make_file_deps
......
...@@ -116,7 +116,7 @@ $(call gb_ExtensionTarget_get_target,%) : \ ...@@ -116,7 +116,7 @@ $(call gb_ExtensionTarget_get_target,%) : \
$(if $(LICENSE),cp -f $(LICENSE) $(call gb_ExtensionTarget_get_rootdir,$*)/registration &&) \ $(if $(LICENSE),cp -f $(LICENSE) $(call gb_ExtensionTarget_get_rootdir,$*)/registration &&) \
$(if $(and $(gb_ExtensionTarget_TRANS_LANGS),$(DESCRIPTION)),cp $(foreach lang,$(gb_ExtensionTarget_TRANS_LANGS),$(call gb_ExtensionTarget_get_workdir,$*)/description-$(lang).txt) $(call gb_ExtensionTarget_get_rootdir,$*) &&) \ $(if $(and $(gb_ExtensionTarget_TRANS_LANGS),$(DESCRIPTION)),cp $(foreach lang,$(gb_ExtensionTarget_TRANS_LANGS),$(call gb_ExtensionTarget_get_workdir,$*)/description-$(lang).txt) $(call gb_ExtensionTarget_get_rootdir,$*) &&) \
cd $(call gb_ExtensionTarget_get_rootdir,$*) && \ cd $(call gb_ExtensionTarget_get_rootdir,$*) && \
$(gb_ExtensionTarget_ZIPCOMMAND) -rX --filesync \ $(gb_ExtensionTarget_ZIPCOMMAND) -rX --filesync --must-match \
$(call gb_ExtensionTarget_get_target,$*) \ $(call gb_ExtensionTarget_get_target,$*) \
$(sort $(FILES))) $(sort $(FILES)))
......
...@@ -50,7 +50,7 @@ $(call gb_Zip_get_target,%) : ...@@ -50,7 +50,7 @@ $(call gb_Zip_get_target,%) :
RESPONSEFILE=$(call var2file,$(shell $(gb_MKTEMP)),500,\ RESPONSEFILE=$(call var2file,$(shell $(gb_MKTEMP)),500,\
$(FILES)) && \ $(FILES)) && \
mkdir -p $(dir $(call gb_Zip_get_target,$*)) && \ mkdir -p $(dir $(call gb_Zip_get_target,$*)) && \
cd $(LOCATION) && cat $${RESPONSEFILE} | tr "[:space:]" "\n" | $(gb_Zip_ZIPCOMMAND) -@rX --filesync $(call gb_Zip_get_target,$*) && \ cd $(LOCATION) && cat $${RESPONSEFILE} | tr "[:space:]" "\n" | $(gb_Zip_ZIPCOMMAND) -@rX --filesync --must-match $(call gb_Zip_get_target,$*) && \
rm -f $${RESPONSEFILE} ) rm -f $${RESPONSEFILE} )
# the final target is a touch target; we use it as registered targets should be in workdir, not in outdir # the final target is a touch target; we use it as registered targets should be in workdir, not in outdir
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment