October 2023 - wine-gitlab

[PATCH 0/1] MR4023: server: Add cross-platform get_path_from_fd function.
by Marc-Aurel Zent (＠marzent) 27 Oct '23

27 Oct '23

This allows *BSD systems that support kqueue (including macOS) to have working file change notifications when configuring wine `--with-inotify` and linking against https://github.com/libinotify-kqueue/libinotify-kqueue -- https://gitlab.winehq.org/wine/wine/-/merge_requests/4023

3 2

Re: [PATCH v4 0/3] MR4168: cmd: Handle caret / blocks with whitespace only lines better
by eric pouech (＠epo) 27 Oct '23

27 Oct '23

no need to apologize... getting patches into Wine is always a bit constrained because of ensuring that we mimic Windows' behavior... and cmd is even more combersome as testing cmd is even more constrained for testing, you can subscribe to testbot.winehq.org (there you can test on several windows versions) I recommend if you can test first locally (your Win10 setup is a good choice), then run the changes in testbot for all default test env (anyway the patch will be tested in the integration pipeline against these, so better test in advance), then update the MR... concerning your failing code, you could use: ``` cmd /c `block_if_eof1.cmd` ``` with a comment explaining why you need to enclose the test inside another instance of cmd (and I don't thing there's lot of value to check %errorlevel% upon cmd exit) -- https://gitlab.winehq.org/wine/wine/-/merge_requests/4168#note_49967

1 0

Re: [PATCH v4 0/3] MR4168: cmd: Handle caret / blocks with whitespace only lines better - closed
by eric pouech (＠epo) 27 Oct '23

27 Oct '23

This merge request was closed by eric pouech. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/4168

1 0

[PATCH v4 0/1] MR4133: mfmediaengine: Don't send event notify when engine is shutdown.
by Yuxuan Shui (＠yshui) 27 Oct '23

27 Oct '23

-- v4: mfmediaengine: Don't send event notify when engine is shutdown. https://gitlab.winehq.org/wine/wine/-/merge_requests/4133

3 3

[PATCH v2 0/1] MR4198: mf/tests: Test hr before checking propvar.pwszVal.
by Rémi Bernon (＠rbernon) 27 Oct '23

27 Oct '23

Wine-Bug: https://bugs.winehq.org//show_bug.cgi?id=55588 -- v2: mf/tests: Avoid crashing if MF_BYTESTREAM_EFFECTIVE_URL is missing. https://gitlab.winehq.org/wine/wine/-/merge_requests/4198

2 2

[PATCH v6 0/3] MR4073: msxml3: Stop possible typelib out of bounds access (Coverity)
by Alistair Leslie-Hughes (＠alesliehughes) 27 Oct '23

27 Oct '23

typelib has an array size of 2 (eg LibXml_Last), so a lookup of IID_NULL will result in a lookup of the third index. -- v6: msxml3: Do not leak bind context on error paths (Coverity) https://gitlab.winehq.org/wine/wine/-/merge_requests/4073

3 4

[PATCH 0/1] MR4198: mf/tests: Test hr before checking propvar.pwszVal.
by Rémi Bernon (＠rbernon) 27 Oct '23

27 Oct '23

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

3 3

Re: [PATCH v4 0/3] MR4168: cmd: Handle caret / blocks with whitespace only lines better
by userid0x0 (＠userid0x0) 27 Oct '23

27 Oct '23

Okay, oh dear. I need to apologize it's the first time I contribute to wine and as always the user learns over time. Now I have the build environment on a Windows 10 Machine set up and I can run the testbench on Windows as well. Cool feature! Lesson learned: I focussed on the unbalanced parentheses case first. Windows quits/ends the complete `cmd` execution if it do a `call block_if_eof1.cmd`. Did you have this case yet? In regards of testing this means I need to setup a own `.cmd`/`.cmd.exp` file again? Lets form a simple setup ``` echo @echo off>block_if_eof1.cmd echo echo sanity1>>block_if_eof1.cmd echo if 1 == 1 (>>block_if_eof1.cmd echo echo should not be executed>>block_if_eof1.cmd call block_if_eof1.cmd REM This command is NEVER executed as cmd exe quited. ``` -- https://gitlab.winehq.org/wine/wine/-/merge_requests/4168#note_49957

1 0

Re: [PATCH v4 0/3] MR4168: cmd: Handle caret / blocks with whitespace only lines better
by userid0x0 (＠userid0x0) 27 Oct '23

27 Oct '23

On Thu Oct 26 17:31:50 2023 +0000, eric pouech wrote: > nitpick: you could have saved one line of modif by keeping the == 0x00 > test (instead of rewritten to 0x0) Can fix that as well. No Problem must have happened while fixup. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/4168#note_49956

1 0

[PATCH v5 0/3] MR4073: msxml3: Stop possible typelib out of bounds access (Coverity)
by Alistair Leslie-Hughes (＠alesliehughes) 27 Oct '23

27 Oct '23

typelib has an array size of 2 (eg LibXml_Last), so a lookup of IID_NULL will result in a lookup of the third index. -- v5: msxml3: Free IBindCtx on error paths (Coverity) msxml3: Move tid_NULL out of possible enum values. msxml3: Dont call qsort if we have no data (Coverity). https://gitlab.winehq.org/wine/wine/-/merge_requests/4073

2 3

2025

2024

2023

2022

wine-gitlab October 2023