Skip to content

[lfs] Only check XDG_CONFIG_HOME when not on 'Windows'#716

Open
loathingKernel wants to merge 1 commit into
legendary-gl:masterfrom
loathingKernel:xdg-config-wine
Open

[lfs] Only check XDG_CONFIG_HOME when not on 'Windows'#716
loathingKernel wants to merge 1 commit into
legendary-gl:masterfrom
loathingKernel:xdg-config-wine

Conversation

@loathingKernel

@loathingKernel loathingKernel commented Sep 9, 2025

Copy link
Copy Markdown
Collaborator

When running legendary through Wine and XDG_CONFIG_HOME is set in the environment, config_path resolves to the absolute path to the Linux config directory. Wine in turn resolves the path in a way that Legendary uses the existing Linux host config directory instead of creating/using the one under Wine prefix in %USERPROFILE%\.config.

@loathingKernel

loathingKernel commented Apr 9, 2026

Copy link
Copy Markdown
Collaborator Author

FWIW, I am not sure this is useful any more, even if someone wants to run the windows binary under Wine. Wine recently started filtering env variables starting with XDG_ out of the windows env, but it doesn't hurt to have either for older wine versions.

@imLinguin

Copy link
Copy Markdown
Member

What is the use case of running legendary under wine anyway?

@loathingKernel

loathingKernel commented Apr 9, 2026

Copy link
Copy Markdown
Collaborator Author

Purely for testing purposes, it's just easier to run it in Wine when I am normally developing on Linux. Anecdotally, I know of people running Rare in Wine for Android, but this should not affect them. At least, I am not aware of any such issue occuring.

When running legendary through Wine and `XDG_CONFIG_HOME` is set in the environment,
`config_path` resolves to the absolute path to the Linux config directory. Wine in turn
resolves the path in the way that Legendary uses the existing Linux host config
directory instead of creating/using the one under Wine prefix in `%USERPROFILE%\.config`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants