beenull/ladybird
1
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2024-11-25 00:50:22 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 7

Ladybird: Look for helper processes at {app_dir}/{helper}/{helper}

Browse source 
Currently, we only look at the relative path `./{helper}/{helper}`,
which fails if the working directory is not the same as the directory
where the ladybird binary lives.
This commit is contained in:
Sigmund Lahn 2023-05-25 20:30:26 +02:00 committed by Andreas Kling
parent ce9f4c3215
commit 3bc0e7a7ca
Notes: sideshowbarker 2024-07-18 03:20:18 +09:00 
Author: https://github.com/slahn
Commit: https://github.com/SerenityOS/serenity/commit/3bc0e7a7ca
Pull-request: https://github.com/SerenityOS/serenity/pull/19041
Reviewed-by: https://github.com/ADKaster
1 changed files with 4 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
Ladybird/HelperProcess.cpp
View file

@ -25,9 +25,12 @@ ErrorOr<void> spawn_helper_process(StringView process_name, ReadonlySpan<StringV
ErrorOr<Vector<String>> get_paths_for_helper_process(StringView process_name)
{
auto application_path = TRY(ak_string_from_qstring(QCoreApplication::applicationDirPath()));
Vector<String> paths;
TRY(paths.try_append(TRY(String::formatted("./{}/{}", process_name, process_name))));
TRY(paths.try_append(TRY(String::formatted("{}/{}", TRY(ak_string_from_qstring(QCoreApplication::applicationDirPath())), process_name))));
TRY(paths.try_append(TRY(String::formatted("{}/{}/{}", application_path, process_name, process_name))));
TRY(paths.try_append(TRY(String::formatted("{}/{}", application_path, process_name))));
TRY(paths.try_append(TRY(String::formatted("./{}", process_name))));
// NOTE: Add platform-specific paths here
return paths;