September 2023 - Wine-gitlab

[PATCH 0/3] MR3771: srvsvc: Add LanmanServer stub service.
by Dmitry Timoshkov (＠dmitry) 08 Sep '23

08 Sep '23

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

3 4

[PATCH 0/1] MR3770: include: Add IPackage2 definition in windows.applicationmodel.idl.
by Biswapriyo Nath (＠Biswa96) 08 Sep '23

08 Sep '23

Required for GNOME/glib project. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3770

3 2

[PATCH 0/3] MR3746: test fixes for fvwm3
by Stefan Dösinger 08 Sep '23

08 Sep '23

These tests make the ddraw-d3d9 tests pass inside Xephyr with fvwm3 for me. fvwm3 still randomly fails in a lot of places if it is run on a real display. It also fails in Xephyr if I wiggle the mouse too much. Those two issues might even be related. A virtual X server should be enough for CI though. fvwm2 has a lot of random failures even inside Xephyr or xf86-video-dummy. I will not attempt to fix them. I tested the SW_SHOWMINNOACTIVE in patches 2 and 3 on Windows XP, Windows 11, the testbot, kwin, openbox. It seems to work without breaking any of those systems. With those things being flaky changes though I can never fully gurantee this though. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3746

4 8

[PATCH v2 0/3] MR3737: winegstreamer: Implement wg_muxer.
by Ziqing Hui (＠zhui) 08 Sep '23

08 Sep '23

This patch set is part of !3303 -- v2: winegstreamer: Create wg_muxer for media sink. winegstreamer: Implement seeking query for wg_muxer sink pad. winegstreamer: Introduce new wg_muxer struct. https://gitlab.winehq.org/wine/wine/-/merge_requests/3737

3 7

[PATCH 0/1] MR3767: msvcp110: Add std::_Throw_future_error.
by Daniel Lehman (＠dlehman25) 08 Sep '23

08 Sep '23

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

3 2

[PATCH 0/2] MR3769: ws2_32/tests: Call GetAdaptersAddresses() in a loop.
by Hans Leidekker (＠hans) 08 Sep '23

08 Sep '23

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

2 2

Re: [PATCH v5 0/4] MR3672: appxdeploymentclient: Add IPackageManager2 stub interface.
by Rémi Bernon 08 Sep '23

08 Sep '23

On Fri Sep 8 06:44:32 2023 +0000, Mohamad Al-Jaf wrote: > I've removed it, seems like it's not really necessary to test for it. > Though, I don't understand why Office queries for `IAgileObject`. It's a > Microsoft program so they should know it's not supported. Minecraft also > queries for an interface that returns `E_NOINTERFACE`, it seems bizarre. You should still remove the `broken(...)` condition, it will otherwise let every test succeed silently on Windows. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3672#note_44665

1 0

Re: [PATCH v3 0/27] MR332: include: Fill in missing enum/struct/interfaces in vkd3d_d3d12.idl
by Martin Storsjö 08 Sep '23

08 Sep '23

On Fri Sep 8 06:50:30 2023 +0000, Henri Verbeet wrote: > At first sight this looks fine, thanks. Unfortunately I probably won't > be able to properly review this this week, but I should be able to get > to it early next week. > Note that this currently fails the CI though. Probably because it's > intended to be applied on top of !325, so hopefully that will get > resolved once those patches actually get committed. It still fails CI after !325 was merged (and even before - as it was based on top of !325, those commits were part of this branch so testing would just have iterated over those too) - not quite sure what is wrong here... -- https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/332#note_44664

1 0

Re: [PATCH v5 0/4] MR3672: appxdeploymentclient: Add IPackageManager2 stub interface.
by Mohamad Al-Jaf (＠maljaf) 08 Sep '23

08 Sep '23

On Thu Sep 7 13:49:19 2023 +0000, Rémi Bernon wrote: > I missed this before, but I think the check_interface check is a bit > pointless as it will always succeed on Windows. > Something like that would be better (using `check_interface_broken` only > where it is needed): > ```suggestion:-14+0 > #define check_interface_broken( obj, iid, supported ) check_interface_( > __LINE__, obj, iid, supported, TRUE ) > #define check_interface( obj, iid, supported ) check_interface_( > __LINE__, obj, iid, supported, FALSE ) > static void check_interface_( unsigned int line, void *obj, const IID > *iid, BOOL supported, BOOL is_broken ) > { > HRESULT hr, expected_hr, broken_hr; > IUnknown *iface = obj; > IUnknown *unk; > expected_hr = supported ? S_OK : E_NOINTERFACE; > broken_hr = supported ? E_NOINTERFACE : S_OK; > hr = IUnknown_QueryInterface( iface, iid, (void **)&unk ); > ok_(__FILE__, line)( hr == expected_hr || broken( is_broken && hr == > broken_hr ), "got hr %#lx.\n", hr ); > if (SUCCEEDED(hr)) IUnknown_Release( unk ); > } > ``` > Or, maybe even better, use the basic `check_interface`, without the > broken result and remove the broken check for IAgileObject interface > which probably doesn't really matter on the factory. I've removed it, seems like it's not really necessary to test for it. Though, I don't understand why Office queries for `IAgileObject`. It's a Microsoft program so they should know it's not supported. Minecraft also queries for an interface that returns `E_NOINTERFACE`, it seems bizarre. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3672#note_44663

1 0

[PATCH v4 0/4] MR3672: appxdeploymentclient: Add IPackageManager2 stub interface.
by Mohamad Al-Jaf (＠maljaf) 08 Sep '23

08 Sep '23

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=55403 -- v4: appxdeploymentclient: Add IPackageManager2 stub interface. include: Add Windows.Management.Deployment.IPackageManager2 interface definition. appxdeploymentclient: Implement IActivationFactory::ActivateInstance(). appxdeploymentclient: Add stub DLL. https://gitlab.winehq.org/wine/wine/-/merge_requests/3672

2 4