On 05/22/2018 01:29 PM, Que Quotion wrote:
> +for lib in "$LIBRARY/buildenv_ext/"*.sh; do
> +     source "$lib"
> +done

As mentioned in the past by dreisner, this is totally wrong. makepkg
does not set nullglob, which means this causes makepkg to error out
trying to source a nonexistent literal file named "*.sh".

This happens because makepkg does not provide any of our own
buildenv_ext files.

IMHO this whole thing is backwards. We should be moving the existing
prepare_buildenv into libmakepkg, rather than appending to it. Once
prepare_buildenv is implemented as a series of dropin buildenv/*.sh.in,
support for extensions will exist as a matter of course. This is how
every other libmakepkg feature works.

-- 
Eli Schwartz
Bug Wrangler and Trusted User

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to