diff options
author | Andrew Robbins <contact@andrewrobbins.info> | 2019-11-02 20:51:24 -0500 |
---|---|---|
committer | Andrew Robbins <contact@andrewrobbins.info> | 2019-11-02 22:07:31 -0500 |
commit | 7c58ae4cdf4c31188a581ca7685bda889132dda9 (patch) | |
tree | f82621cca0c84a10491c84b1ee8e24f267d40250 /projects/ich9gen | |
parent | 0c736cf027a90c0540834c53b957b8c577591c91 (diff) | |
download | librebootfr-7c58ae4cdf4c31188a581ca7685bda889132dda9.tar.gz librebootfr-7c58ae4cdf4c31188a581ca7685bda889132dda9.zip |
Remove redundant subshells in project actions
Since an action is performed within a subshell there is no need
for a second subshell to handle errors.
Diffstat (limited to 'projects/ich9gen')
-rwxr-xr-x | projects/ich9gen/ich9gen | 17 |
1 files changed, 7 insertions, 10 deletions
diff --git a/projects/ich9gen/ich9gen b/projects/ich9gen/ich9gen index 71567329..5ba9ae2f 100755 --- a/projects/ich9gen/ich9gen +++ b/projects/ich9gen/ich9gen @@ -58,23 +58,20 @@ build() { fi local sources_path="$(project_sources_path "$project" "$repository" "$@")" - local build_path="$(project_build_path "$project" "$@")" mkdir -p "$build_path" make -C "$sources_path" -j"$TASKS" ich9gen - ( - local macaddress="$(ich9gen_macaddress)" + local macaddress="$(ich9gen_macaddress)" - cd "$build_path" + cd "$build_path" - if [[ -n $macaddress ]]; then - "$sources_path"/ich9gen --macaddress "$macaddress" - else - "$sources_path"/ich9gen - fi - ) + if [[ -n $macaddress ]]; then + "$sources_path"/ich9gen --macaddress "$macaddress" + else + "$sources_path"/ich9gen + fi cp "$sources_path/ich9gen" "$build_path" make -C "$sources_path" clean |