December 2022 - Wine-GitLab

Re: [PATCH v3 0/3] MR1825: kernelbase/path.c: UrlFixupW()
by Etaash Mathamsetty (＠etaash.mathamsetty) Dec. 16, 2022

Dec. 16, 2022

I also think you should squash c4e27bd0483c0a1fb872111b6c03b9ad240d85da (mark tests as todo) and d02be967f9cca0805b5eeaef6a74b3ec51a67619 (add tests) into one commit -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1825#note_19781

1 0

Re: [PATCH v6 0/41] MR1810: winebus.sys: Fix the build without HAVE_SYS_INOTIFY_H
by Alex Henrie (＠alexhenrie) Dec. 16, 2022

Dec. 16, 2022

On Sat Dec 17 00:00:58 2022 +0000, Gerald Pfeifer wrote: > Any chance you can cherry pick > https://gitlab.winehq.org/wine/wine/-/commit/6ddda5b130481c89dae3750597452f… > ? > Git/Gitlab and me are not friends yet, and I already spent hours > (literally!) on this. :-( You've just got to learn how to use Git. One way to fix your branch: `git pull -ri https://gitlab.winehq.org/wine/wine.git master` Change all but the first commit to "squash", edit the commit message until it looks right, and then force-push. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1810#note_19775

1 0

Re: [PATCH v6 0/41] MR1810: winebus.sys: Fix the build without HAVE_SYS_INOTIFY_H
by Gerald Pfeifer (＠gerald) Dec. 16, 2022

Dec. 16, 2022

On Fri Dec 16 23:23:09 2022 +0000, Rémi Bernon wrote: > I don't have a strong opinion, although I tend to prefer having > variables declared in one place, especially for small functions like > this. In any case it's an unrelated and unnecessary change here. Any chance you can cherry pick https://gitlab.winehq.org/wine/wine/-/commit/6ddda5b130481c89dae3750597452f… ? Git/Gitlab and me are not friends yet, and I already spent hours (literally!) on this. :-( -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1810#note_19774

1 0

Re: [PATCH v6 0/41] MR1810: winebus.sys: Fix the build without HAVE_SYS_INOTIFY_H - closed
by Gerald Pfeifer (＠gerald) Dec. 16, 2022

Dec. 16, 2022

This merge request was closed by Gerald Pfeifer. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1810

1 0

[PATCH v3 0/1] MR1810: winebus.sys: Fix the build without HAVE_SYS_INOTIFY_H
by Gerald Pfeifer (＠gerald) Dec. 16, 2022

Dec. 16, 2022

find_device_from_devnode was guarded by HAVE_SYS_INOTIFY_H, alas its use in process_monitor_event was not, so linking failed. On the way reduce the scope of a variable (which makes sense per se and avoids introducing a compiler warning with this change). -- v3: winebus.sys: Fix the build without HAVE_SYS_INOTIFY_H https://gitlab.winehq.org/wine/wine/-/merge_requests/1810

3 4

Re: [PATCH 0/2] MR31: shell32: Only call HCR_GetFolderAttributes with simple pidls
by Fabian Maurer (＠DarkShadow44) Dec. 16, 2022

Dec. 16, 2022

Obsolete now. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/31#note_19768

1 0

Re: [PATCH 0/2] MR31: shell32: Only call HCR_GetFolderAttributes with simple pidls - closed
by Fabian Maurer (＠DarkShadow44) Dec. 16, 2022

Dec. 16, 2022

This merge request was closed by Fabian Maurer. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/31

1 0

Re: [PATCH v2 0/3] MR1830: Fixing some debugger tests when run on Windows. - approved
by Alexandre Julliard (＠julliard) Dec. 16, 2022

Dec. 16, 2022

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

1 0

Re: [PATCH v3 0/2] MR1829: user32/tests: Don't expect WM_DISPLAYCHANGE with the default display mode. - approved
by Alexandre Julliard (＠julliard) Dec. 16, 2022

Dec. 16, 2022

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

1 0

Re: [PATCH v2 0/3] MR1794: Improvements to ntdll.Rtl*ByteSwap(). - approved
by Alexandre Julliard (＠julliard) Dec. 16, 2022

Dec. 16, 2022

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

1 0