diff options
author | Andrew Robbins <contact@andrewrobbins.info> | 2017-07-22 01:51:01 -0400 |
---|---|---|
committer | Andrew Robbins <contact@andrewrobbins.info> | 2017-07-22 01:51:01 -0400 |
commit | 5e3dcd8f29cf4ad653e69d50e1500a330bba8ca8 (patch) | |
tree | 6ab2c8545571229d2f4c0dda5ee7976d47cf078f /docs | |
parent | 6e92244e0cc9a3b08cfcca8bd85990f795abb83e (diff) | |
download | librebootfr-5e3dcd8f29cf4ad653e69d50e1500a330bba8ca8.tar.gz librebootfr-5e3dcd8f29cf4ad653e69d50e1500a330bba8ca8.zip |
Remove IFS binding masking a non-zero status code
In project_action_arguments_recursive(), IFS bindings needed to be
reworked in order to avoid a situation where, for example,
project_extract() would fail to extract source archives (because they
didn't exist), returning a status code of 1 only for it to be masked
by a rebinding of IFS--which would always succeed.
ifs_save and IFS were made local variables in
project_action_arguments_recursive() in order to avoid the need to
rebind IFS after the for loop returns.
This patch makes './libreboot sources <project>' functional.
Diffstat (limited to 'docs')
0 files changed, 0 insertions, 0 deletions