diff options
author | Leah Rowe <leah@libreboot.org> | 2025-05-15 02:20:55 +0100 |
---|---|---|
committer | Leah Rowe <leah@libreboot.org> | 2025-05-15 02:25:02 +0100 |
commit | c8813c9a144b14a78f7328ce7b886270e1b676f3 (patch) | |
tree | 4ec4c6ba5cfe7773ca44a5b57477daad93667b67 /config/submodule/coreboot/default/arm-trusted-firmware/module.cfg | |
parent | 208dfc89bd583f4d8b757ca34e6f6a85538d5e48 (diff) |
properly exit 1 when calling fx_
in a few places, we use the presence of a file found
by fx_ to cause an exit, but the command that runs
looks something like:
exit 1 "string"
this yields an error, and a non-zero exit, because of
too many arguments to "exit", but we wanted a non-zero
exit anyway.
nevertheless, this is incorrect.
to fix it, eval is used instead. if the never-going-to-exist
condition one day exists where exit 1 actually returns, not,
you know, exits, we will use err instead, with the string
as argument.
this should be fine. it's a bit hacky, but so is fx_, and
it works. fx_ is used in several places to keep the sloccount
down, providing a common way to perform while loops on the
output of a command; that is its only purpose..
Signed-off-by: Leah Rowe <leah@libreboot.org>
Diffstat (limited to 'config/submodule/coreboot/default/arm-trusted-firmware/module.cfg')
0 files changed, 0 insertions, 0 deletions