github-mirror/fish-shell
2
0
Fork 0
mirror of https://github.com/fish-shell/fish-shell.git synced 2024-11-23 06:41:42 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fix actual issue with allow_use_posix_spawn.

Browse Source 
We were testing the function pointer, not evaluating the function.

This should be the proper fix. Thanks @ridiculousfish
This commit is contained in:
Aaron Gyes 2022-08-22 13:30:51 -07:00
parent 9c896e1990
commit 85b9f3c71f
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/env_dispatch.cpp
View File

@ -270,7 +270,7 @@ static constexpr bool allow_use_posix_spawn() {
/// Whether to use posix_spawn when possible.
static relaxed_atomic_bool_t g_use_posix_spawn{false};
bool get_use_posix_spawn() { return allow_use_posix_spawn() && g_use_posix_spawn; }
bool get_use_posix_spawn() { return g_use_posix_spawn; }
static void handle_fish_use_posix_spawn_change(const environment_t &vars) {
// If the variable is missing or empty, we default to true if allowed.
@ -327,7 +327,7 @@ static std::unique_ptr<const var_dispatch_table_t> create_dispatch_table() {
var_dispatch_table->add(L"fish_history", handle_fish_history_change);
var_dispatch_table->add(L"fish_autosuggestion_enabled", handle_autosuggestion_change);
var_dispatch_table->add(L"TZ", handle_tz_change);
if (allow_use_posix_spawn) {
if (allow_use_posix_spawn()) {
var_dispatch_table->add(L"fish_use_posix_spawn", handle_fish_use_posix_spawn_change);
}
var_dispatch_table->add(L"fish_trace", handle_fish_trace);