LadybirdBrowser/ladybird
1
1
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2025-01-04 01:05:58 +03:00
Code Issues Packages Projects Releases Wiki Activity

Shell: Make the 'not' builtin return the correct exit code for functions

Browse Source
This commit is contained in:
AnotherTest 2021-03-07 09:55:02 +03:30 committed by Andreas Kling
parent fec8d7d699
commit 4f6bf2931c
Notes: sideshowbarker 2024-07-18 21:40:00 +09:00 
Author: https://github.com/alimpfard
Commit: https://github.com/SerenityOS/serenity/commit/4f6bf2931c5
Pull-request: https://github.com/SerenityOS/serenity/pull/5672
1 changed files with 5 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
Userland/Shell/Builtin.cpp
View File

@ -892,10 +892,15 @@ int Shell::builtin_not(int argc, const char** argv)
auto commands = expand_aliases({ move(command) }); auto commands = expand_aliases({ move(command) });
int exit_code = 1; int exit_code = 1;
auto found_a_job = false;
for (auto& job : run_commands(commands)) { for (auto& job : run_commands(commands)) {
found_a_job = true;
block_on_job(job); block_on_job(job);
exit_code = job.exit_code(); exit_code = job.exit_code();
} }
// In case it was a function.
if (!found_a_job)
exit_code = last_return_code;
return exit_code == 0 ? 1 : 0; return exit_code == 0 ? 1 : 0;
} }