search

kisslinux / kiss

KISS Linux - Package Manager
https://kisslinux.github.io
MIT License
464 stars 62 forks source link

${@:?} isn't portable #257

Closed illiliti closed 3 years ago

illiliti commented 3 years ago

This breaks mksh:

https://github.com/kisslinux/kiss/blob/554462faa7556f954dedcd0eeb6efd48ae95ab5b/kiss#L1843

Issue 8(draft) of POSIX clearly defines that expansions of $@/$* result in undefined behavior:

https://www.austingroupbugs.net/view.php?id=1478

dylanaraps commented 3 years ago

https://github.com/kisslinux/kiss/commit/bb98906055dc88e68eabe92f4ea9a94407a5b4ca

Forgot to close this. Thanks :)