Commit e8842f6d authored by Ian Lynagh's avatar Ian Lynagh
Browse files

Fix bindist creation

We were running into problems like:
    for f in  LICENSE configure config.sub config.guess [...]
    make[2]: execvp: /bin/sh: Argument list too long
This patch moves the loop into make, rather than the shell.
parent b56947b3
......@@ -805,8 +805,7 @@ install_packages: libffi/package.conf.install rts/package.conf.install
# -----------------------------------------------------------------------------
# Binary distributions
# This is split in 2 because of the shell argument limit
$(eval $(call bindist,root1,\
$(eval $(call bindist,.,\
configure config.sub config.guess install-sh \ \
......@@ -827,8 +826,6 @@ $(eval $(call bindist,root1,\
$(eval $(call bindist,root2,\
$(addsuffix /*,$(INSTALL_HTML_DOC_DIRS)) \
docs/index.html \
$(wildcard libraries/*/dist-install/doc/) \
......@@ -23,6 +23,14 @@ define bindist
bindist: bindist_$1
for f in $2; do echo $(BIN_DIST_NAME)/$$$$f >> $(BIN_DIST_LIST); done
$(foreach i,$2,$(call bindist_item,$i))
define bindist_item
# $1 = the line
# The formatting of this definition (e.g. the blank line above) is
# important, in order to get make to generate the right makefile code.
for f in $(BIN_DIST_NAME)/$1; do echo $$$$f >> $(BIN_DIST_LIST); done
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