system/serenity
system/serenity
1
0
Fork 0
mirror of https://github.com/SerenityOS/serenity synced 2024-07-21 18:15:58 +00:00
Code Issues Actions 5 Packages Projects Releases Wiki Activity

open: Display a meaningful error if the specified file doesn't exist

Browse source 
Previously, the error given for any non-existent file was:
"Failed to open ':'".
This commit is contained in:
Tim Ledbetter 2023-10-03 21:50:32 +01:00 committed by Andreas Kling
parent 1f2ab4ebfc
commit 0068e91aad
1 changed files with 11 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
Userland/Utilities/open.cpp
View file

@ -25,8 +25,17 @@ ErrorOr<int> serenity_main(Main::Arguments arguments)
bool all_ok = true;
for (auto& url_or_path : urls_or_paths) {
auto path = FileSystem::real_path(url_or_path);
auto url = URL::create_with_url_or_path(path.is_error() ? url_or_path : path.value());
auto path_or_error = FileSystem::real_path(url_or_path);
URL url;
if (path_or_error.is_error()) {
url = url_or_path;
if (!url.is_valid()) {
warnln("Failed to open: '{}': {}", url_or_path, strerror(path_or_error.error().code()));
continue;
}
} else {
url = URL::create_with_url_or_path(path_or_error.value().to_deprecated_string());
}
if (!Desktop::Launcher::open(url)) {
warnln("Failed to open '{}'", url);