November 2025 - Wine-GitLab

Re: [PATCH v4 0/1] MR9423: ntdll: Warn when environment block is too large, instead of removing it
by Martino Fontana (＠tinozzo123) Nov. 13, 2025

Nov. 13, 2025

> there would be a clear message pointing out the issue I don't want to accuse you of https://xkcd.com/2501/, but the average user doesn't launch Wine from the terminal, and may not know what environment variables are (if they blindly follow guides, they won't understand that the log is related to their issue). > as opposed to an obscure crash [it actually isn't a crash, it's an error message] Which is what the logs of my MR are about (which I could make more specific so that they explicitly specify EA app). So, back to the beginning: better to not change anything and add a logs that explain an old problem, or to add a workaround and add logs to explain new problems created by it? Due to all the reasoning I already gave, I'd prefer the former. Now, I know you may accuse me of https://xkcd.com/1172/, but > To me it seems like an improvement. let's consider the effect of the change to the population of users who have a large environment block: - They don't use EA App -> The change may only be negative - They do use EA App -> The change is positive... but still has the negative side effects (so it's a sidegrade, not an upgrade) -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9423#note_121859

1 0

Re: [PATCH v4 0/1] MR9423: ntdll: Warn when environment block is too large, instead of removing it
by Alexandre Julliard (＠julliard) Nov. 13, 2025

Nov. 13, 2025

I'm not convinced that it would break all that much, and if it does, there would be a clear message pointing out the issue, as opposed to an obscure crash. To me it seems like an improvement. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9423#note_121853

1 0

Re: [PATCH v4 0/1] MR9423: ntdll: Warn when environment block is too large, instead of removing it
by Martino Fontana (＠tinozzo123) Nov. 13, 2025

Nov. 13, 2025

I'm aware that this is a lose-lose scenario, due to EA not fixing their issue on the end, since it's either: - Workaround the issue, but cause breaking changes outside of it - Do nothing, but EA app remains not working without manual intervention And IMO, the second choice is better. And many things can still be done, like having better documentation about the issue. For instance, Wine's wiki has a [Known Issues page](https://gitlab.winehq.org/wine/wine/-/wikis/Known-Issues), where this definitely belongs. Or maybe it could be added to the Arch Wiki as well. Or [make changes to systems that set huge environment variables to be less wasteful](https://github.com/NixOS/nixpkgs/pull/451031) (which is nice even outside of this). (And if there is still intention of doing a workaround for the issue... would it be possible to unset environment variables only if the executable `EADesktop.exe` is launched, to minimize the impact of breaking changes elsewhere?) -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9423#note_121852

1 0

[PATCH 0/3] MR9427: combobox: Adjust MEASUREITEMSTRUCT.itemHeight by 2 instead of 6.
by Paul Gofman (＠gofman) Nov. 13, 2025

Nov. 13, 2025

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=58932 -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9427

3 6

Re: [PATCH v4 0/1] MR9423: ntdll: Warn when environment block is too large, instead of removing it
by Alexandre Julliard (＠julliard) Nov. 13, 2025

Nov. 13, 2025

> It's a change that breaks configurations for little gain Obviously the gain is that a commonly-used app no longer crashes. How would you suggest to achieve that otherwise? Printing a warning may make it easier to determine the cause of the crash, but it doesn't improve the user's experience. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9423#note_121847

1 0

Re: [PATCH v4 0/1] MR9423: ntdll: Warn when environment block is too large, instead of removing it
by Martino Fontana (＠tinozzo123) Nov. 13, 2025

Nov. 13, 2025

Your change would be nice even with the warning I'm proposing, considering that it helps with trimming the environment block. I don't understand why it should still not import the environment block at all. I'll reiterate why I think conditionally not importing it is a bad idea: - If modern Windows allows unlimited environment block size, Wine should allow it too - It's a change that breaks configurations for little gain - Because it's conditional, documentation becomes much more cumbersome. Think of all the resources online saying something like "set this environment variable", which will now need to be updated with "unless your environment block is too big, you'll also have to trim the environment block too" (which is more complicated than this sentence). And that's assuming it gets updated at all. Add in users why follow guides and then don't notice the log (either because they don't use the terminal or because they are newbies and can't see that this log is related to why the guide is not working), and you get a mess. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9423#note_121846

1 0

[PATCH v3 0/5] MR9394: winex11: Lock and flush display in X11DRV_ChangeDisplaySettings.
by Rémi Bernon Nov. 13, 2025

Nov. 13, 2025

-- v3: winex11: Lock and flush display in X11DRV_ChangeDisplaySettings. winex11: Retrieve full modes before applying new display settings. winex11: Introduce a x11drv_mode structure for full modes. winex11: Always return full modes from settings handlers. winex11: Copy XF86VidModeModeInfo to the DEVMODEW driver data. https://gitlab.winehq.org/wine/wine/-/merge_requests/9394

2 9

Re: [PATCH v4 0/1] MR9423: ntdll: Warn when environment block is too large, instead of removing it
by Alexandre Julliard (＠julliard) Nov. 13, 2025

Nov. 13, 2025

I'd suggest something like !9450. You'd then be able to set WINE_HOST_XDG_DATA_DIRS to hide the distro value and keep the environment to a reasonable size. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9423#note_121838

1 0

[PATCH 0/3] MR9450: Don't replace WINE_HOST_ variables if they already exist in the environment.
by Alexandre Julliard (＠julliard) Nov. 13, 2025

Nov. 13, 2025

This makes it possible to override the host value seen by Wine of e.g. XDG_DATA_DIRS by setting WINE_HOST_XDG_DATA_DIRS directly. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9450

2 3

[PATCH 0/1] MR9439: mf: Return E_NOINTERFACE if service is missing
by Aric Stewart (＠aricstewart) Nov. 13, 2025

Nov. 13, 2025

Signed-off-by: Aric Stewart <aric(a)codeweavers.com> -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9439

3 4