> Wine and forks only use them through getenv imported from <stdlib.h>, so it shouldn't matter whether they are in the Windows environment if I'm correct.
Yes, the issue would be Windows apps that use a PE build of SDL. I don't know if there are variables that would translate meaningfully across Unix and Windows environments.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/9423#note_122011
> Sure, how about something like MR !9461?
Yes. (Only nitpick is that there should be more comments explaining the why.)
> Any other variables we should add there?
Nothing comes to mind. More can always be added later.
> Should we ignore all the SDL_ variables?
Wine and forks only use them through `getenv` imported from `<stdlib.h>`, so it shouldn't matter whether they are in the Windows environment if I'm correct.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/9423#note_122009
## Summary
Tests for UrlFixupW and semi stub function implementaion.
## Test results
- Tests are 100% ok on my local WIN 10 pc. Wine test bot seems to be okay too.
- Wine tests on debian machine
| function type | result |
| ------ | ------ |
| stub | 0020:url: 69802 tests executed (0 marked as todo, 0 as flaky, **200** failures), 0 skipped. |
| semi stub| 0020:url: 69802 tests executed (0 marked as todo, 0 as flaky, **100** failures), 0 skipped. |
## Todos
For follow up merge request(s).
- [ ] Add more tests for priority investigation, who url scheme fix up is working on windows.
- [ ] Implement missing parts of fix up.
## Conclusion
Semi stub implementation is an improvement to the existing solution. A grep showed that wine internals do not call this function, therefore it should not introduce any new bugs. Windows applications using/relying on the function should now work better, if not they may reveal already existing bugs.
--
v6: shlwapi: fixup leading and trailing typos in UrlFixupW
shlwapi: check for colon in UrlFixupW
kernelbase: Restructure UrlFixupW()
shlwapi/tests: Add tests for UrlFixupW.
https://gitlab.winehq.org/wine/wine/-/merge_requests/1825
> Importing something is better than nothing, and this achieves the goal of keeping popular configuration envars like DXVK_FRAME_RATE. IMO, it is absolutely worth the code bloat.
Code bloat is not an issue, I'm more concerned about the unpredictability of which variables will be removed.
Maybe now that we have b1e94119f07919991edb1e629090b04e7e8b34a9 we could be more aggressive in removing unneeded vars. For instance it's very unlikely that NIXPKGS variables would be needed by a Windows app.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/9423#note_121997
As the release approaches, I remain concerned about the breaking changes.
I use configuration envars much more frequently than the EA App, meaning this change would require me to apply workarounds everywhere (including the EA App itself) rather than just for one application. This increases the manual intervention burden significantly for users like me.
Plus, that EA itself is disincentivizing the use EA App their push of anti-Linux anticheats, so the percentage of users with a big envar block that don't use the EA App is likely substantial.
The previous discussion established that [Wine doesn't do breaking changes or workarounds](https://gitlab.winehq.org/wine/wine/-/merge_requests/6140#note…. While other solutions could be explored longer-term, for now it would be best if the next Wine release ships without this breaking change.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/9423#note_121996