system/serenity
system/serenity
1
0
Fork 0
mirror of https://github.com/SerenityOS/serenity synced 2024-09-16 06:30:41 +00:00
Code Issues Packages Projects Releases Wiki Activity

rm: Allow empty paths if -f is specified

Browse source 
On most (if not all) systems rm ignores an empty paths array if -f or
--force is specified. This helps with scripts that may pass an empty
variable where the file paths are supposed to go.
This commit is contained in:
Tim Schumacher 2021-06-05 03:08:02 +02:00 committed by Andreas Kling
parent aec941b46c
commit f295ac3c0b
1 changed files with 6 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
Userland/Utilities/rm.cpp
View file

@ -27,9 +27,14 @@ int main(int argc, char** argv)
args_parser.add_option(recursive, "Delete directories recursively", "recursive", 'r');
args_parser.add_option(force, "Force", "force", 'f');
args_parser.add_option(verbose, "Verbose", "verbose", 'v');
args_parser.add_positional_argument(paths, "Path(s) to remove", "path");
args_parser.add_positional_argument(paths, "Path(s) to remove", "path", Core::ArgsParser::Required::No);
args_parser.parse(argc, argv);
if (!force && paths.is_empty()) {
args_parser.print_usage(stderr, argv[0]);
return 1;
}
bool had_errors = false;
for (auto& path : paths) {
auto result = Core::File::remove(path, recursive ? Core::File::RecursionMode::Allowed : Core::File::RecursionMode::Disallowed, force);