September 2023 - wine-gitlab

[PATCH v7 0/6] MR3870: mspatcha: Add support for 32-bit file patching and normalization
by Aidan Khoury (＠ajkhoury) 21 Sep '23

21 Sep '23

This clears up much of the todo list for mspatcha. GetFilePatchSignature* and NormalizeFileForPatchSignature have now been implemented in these changes. These changes bring better support for things like the Adobe Acrobat installer without the need for things like the winetricks mspatcha.dll native dll override. Still needed is support for interleaved streams in the LZXD decompression logic, however this is a great start to better supporting software installers that use the Windows interface for creating and applying patches to files. This is the beginning of the fixes required for bug 12501: https://bugs.winehq.org/show_bug.cgi?id=12501 -- v7: mspatcha: Fix binary to hex string conversions for GetFilePatchSignatureByBuffer https://gitlab.winehq.org/wine/wine/-/merge_requests/3870

3 6

[PATCH v3 0/1] MR3648: server: Set the default thread exit code to 1.
by Brendan Shanks (＠bshanks) 21 Sep '23

21 Sep '23

The default exit code is only used when a process is killed by the system or through a signal. These are exceptional cases where exit code 1 (killed by Task Manager) is more appropriate than the normal termination code of 0. Chromium does not restart helper processes that exit with code 0 (see [kill.h](https://source.chromium.org/chromium/chromium/src/+/main:base/proce…), with this fix they are restarted when killed by the system or by a signal. -- v3: server: Set the default thread exit code to 1. https://gitlab.winehq.org/wine/wine/-/merge_requests/3648

4 5

Re: [PATCH 0/1] MR3908: wineserver: Report non-zero exit code for abnormal process termination
by Adam Rehn (＠adamrehn) 21 Sep '23

21 Sep '23

On Thu Sep 21 00:46:47 2023 +0000, Jinoh Kang wrote: > I'm not sure if it is correct to flip the "violent delth" flag > parameter. Anyways, this looks like a duplicate of > https://gitlab.winehq.org/wine/wine/-/merge_requests/3648, so you might > want to ask about its status? @iamahuman thanks for pointing out that older thread, I'll follow up on that! With regards to the `violent_death` parameter, are there any guidelines around when it is appropriate to set that to a value of 1? From what I can see, it's primarily used when a protocol error is encountered (e.g. [here](https://gitlab.winehq.org/wine/wine/-/blob/wine-8.16/server/request.c… and [here](https://gitlab.winehq.org/wine/wine/-/blob/wine-8.16/server/request.c…) and the server can no longer communicate with the client process. That seems fairly similar to the abnormal process termination scenario (with the main difference being that the server isn't trying to send or receive a message at the time it detects the pipe close), but there might be some nuance there that I'm overlooking? If there's a more appropriate mechanism to set the non-zero exit code in the abnormal process termination scenario then I'd certainly be happy to update the patch to use that instead. The current implementation was designed to be as minimally intrusive as possible whilst still achieving the desired behaviour, but I'm not familiar enough with the internal workings of the Wine server to be completely confident in its correctness, so any guidance here is much appreciated! (I'm also happy to add comments if desired, since I recognise that the current code is not particularly explicit in communicating its intent to readers.) -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3908#note_46121

1 0

Re: [PATCH 0/6] MR3909: winewayland.drv: part 7: Basic window management
by Shmerl (＠shmerl) 21 Sep '23

21 Sep '23

libdecor makes sense as a workaround for Gnome which doesn't support server side decorations. I.e. server side decorations can be used as a primary option and if it's not available, Wine can fall back to libdecor. It shouldn't matter much for games I think, since games normally run in fullscreen if you care about performance, so window decorations are really irrelevant in gaming use case. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3909#note_46120

1 0

Re: [PATCH 0/6] MR3909: winewayland.drv: part 7: Basic window management
by Gabriele (＠gabriele2000) 20 Sep '23

20 Sep '23

I wonder if from this MR, we'll be able to somehow test those changes, by using an environment variable or something similar. I'd be very interested in testing it out as soon as it gets merged! -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3909#note_46119

1 0

Re: [PATCH v2 0/1] MR3913: user32/tests: Fix WM_WINDOWPOSCHANGED entries. - approved
by Alexandre Julliard (＠julliard) 20 Sep '23

20 Sep '23

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3913

1 0

Re: [PATCH v2 0/5] MR3910: uiautomationcore: More MSAA proxy provider patches. - approved
by Alexandre Julliard (＠julliard) 20 Sep '23

20 Sep '23

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3910

1 0

Re: [PATCH v2 0/2] MR3906: dmusic: Keep original instrument patch around. - approved
by Alexandre Julliard (＠julliard) 20 Sep '23

20 Sep '23

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3906

1 0

Re: [PATCH v2 0/4] MR3905: dmime/tests: Test DMUS_PMSG with band and sequence tracks. - approved
by Alexandre Julliard (＠julliard) 20 Sep '23

20 Sep '23

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3905

1 0

Re: [PATCH v2 0/1] MR3912: msvcp90: Silence FIXME in _Locinfo::_Locinfo_ctor and _Locinfo::_Locinfo_Addcats functions. - approved
by Alexandre Julliard (＠julliard) 20 Sep '23

20 Sep '23

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3912

1 0

2025

2024

2023

2022

wine-gitlab September 2023