-- v3: Release 8.0.2. localspl: Don't pass original document name to cupsStartDocument. wininet/tests: Update expected WineHQ certificate. winhttp/tests: Fix a test failure.
This merge request has too many patches to be relayed via email. Please visit the URL below to see the contents of the merge request. https://gitlab.winehq.org/wine/wine/-/merge_requests/3168
On Sun Jul 9 11:29:22 2023 +0000, Piotr Caban wrote:
I think it's worth adding (it fixes PDFs printing).
Added, thanks!
Added 3 commits: - 2 test fixes - 1 regression fix suggested by Piotr
This is now 8.0.2-rc1
This merge request was approved by Jan Sikorski.
I don't know if we need "mfplat: Implement MFCreatePathFromURL()." in the stable update, it was not confirmed to help anything, but probably won't hurt.
This merge request was approved by Nikolay Sivov.
This merge request was approved by Piotr Caban.
On Tue Jul 11 18:25:32 2023 +0000, Nikolay Sivov wrote:
I don't know if we need "mfplat: Implement MFCreatePathFromURL()." in the stable update, it was not confirmed to help anything, but probably won't hurt.
Yeah, I have been pulling those stub to implementation changes in if they come via a bugzilla bug. Even if there is a follow up bug that blocks the original application. Those patches are safe ("definite crash" ==> "possible crash later" is always an improvement) and easy to review.
I'm not sure about the choice of MSHTML leaks, especially the ones that are mostly theoretical. But they are safe enough, so I don't mind.
This merge request was approved by Jacek Caban.
On Tue Jul 11 20:03:49 2023 +0000, Jacek Caban wrote:
I'm not sure about the choice of MSHTML leaks, especially the ones that are mostly theoretical. But they are safe enough, so I don't mind.
Well, I stopped when those became unsafe and kinda got out of hand. But by that time I already had those reviewed and picked so I kept them.
But if you insist I can kick them out.
On Tue Jul 11 20:03:49 2023 +0000, Michael Stefaniuc wrote:
Well, I stopped when those became unsafe and kinda got out of hand. But by that time I already had those reviewed and picked so I kept them. But if you insist I can kick them out.
I reviewed them already as well, it's fine with me to keep them. I just wanted to suggest to be a bit more picky about patches like that in the future.
This merge request was approved by Michael Stefaniuc.
-
Jacek Caban (＠jacek) -
Jan Sikorski (＠jsikorski)
-
Michael Stefaniuc (＠mstefani)
-
Nikolay Sivov (＠nsivov)
-
Piotr Caban (＠piotr)