Hi,
Brandon Casey <casey@xxxxxxxxxxxxxxx> writes:
> ksh does not like it when the list argument is missing in a for loop. This
> can happen when NO_CURL is set which causes REMOTE_CURL_ALIASES to be unset.
> In this case, the for loop in the Makefile expands to look like this:
>
> for p in ; do
>
> and ksh complains like this:
>
> /bin/ksh: syntax error at line 15 : `;' unexpected
[snip]
> - { test x"$(REMOTE_CURL_ALIASES)" = x || \
> - for p in $(REMOTE_CURL_ALIASES); do \
> + remote_curl_aliases="$(REMOTE_CURL_ALIASES)" && \
> + for p in $$remote_curl_aliases; do \
I believe the idiom
test x'$(foo)' = x || for p in ''$(foo); do
works equally well in this case, and is less invasive.
- Hari
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html