[PATCH v2 0/1] MR5231: ntdll: Add SDL_VIDEODRIVER to the special variable list.

[Aida Jonikienė]

Overriding the SDL_VIDEODRIVER variable (for Wayland support as an example)
on the Linux side can lead to some games under Wine failing to load (so treat
that variable as special).

--
  v2: ntdll: Add SDL_VIDEODRIVER to the special variable list.
 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5231

[PATCH v2 1/1] ntdll: Add SDL_VIDEODRIVER to the special variable list.

[Aida Jonikienė]

From: Aida Jonikienė <aidas957(a)gmail.com>

Overriding the SDL_VIDEODRIVER variable (for Wayland support as an example)
on the Linux side can lead to some games under Wine failing to load (so treat
that variable as special).
---
 dlls/ntdll/unix/env.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/dlls/ntdll/unix/env.c b/dlls/ntdll/unix/env.c
index ad9ab0dc220..e3212c827f8 100644
--- a/dlls/ntdll/unix/env.c
+++ b/dlls/ntdll/unix/env.c
@@ -341,7 +341,8 @@ static BOOL is_special_env_var( const char *var )
             STARTS_WITH( var, "TEMP=" ) ||
             STARTS_WITH( var, "TMP=" ) ||
             STARTS_WITH( var, "QT_" ) ||
-            STARTS_WITH( var, "VK_" ));
+            STARTS_WITH( var, "VK_" ) ||
+            !strcmp( var, "SDL_VIDEODRIVER" ));
 }
 
 /* check if an environment variable changes dynamically in every new process */
-- 
GitLab

https://gitlab.winehq.org/wine/wine/-/merge_requests/5231

[Aida Jonikienė]

On Tue Mar  5 08:37:17 2024 +0000, Aida Jonikienė wrote:
> So should I only hide SDL_VIDEODRIVER? That's enough to prevent issues
> in my case
> I updated the MR to only affect SDL_VIDEODRIVER (hopefully that's good enough)

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5231#note_63810