[Bug 52132] New: DeviceCapabilities DC_PAPERSIZE return wrong sizes (Regression)
https://bugs.winehq.org/show_bug.cgi?id=52132 Bug ID: 52132 Summary: DeviceCapabilities DC_PAPERSIZE return wrong sizes (Regression) Product: Wine Version: unspecified Hardware: x86-64 OS: Linux Status: UNCONFIRMED Severity: major Priority: P2 Component: spooler Assignee: wine-bugs(a)winehq.org Reporter: incubusrk(a)gmail.com Distribution: --- Created attachment 71169 --> https://bugs.winehq.org/attachment.cgi?id=71169 test sample get system default printer and show paper names and sizes Wine 6.20 and newer has broken DeviceCapabilities fwCapability=DC_PAPERSIZE Test work fine in wine 6.15 PS All test was made with 64-bit prefix -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=52132 Konstantin <incubusrk(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |regression -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=52132 Ken Sharp <imwellcushtymelike(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Version|unspecified |6.20 Ever confirmed|0 |1 Severity|major |normal Keywords|regression |download, printing, source, | |testcase Status|UNCONFIRMED |NEEDINFO --- Comment #1 from Ken Sharp <imwellcushtymelike(a)gmail.com> --- Please run a regression test. https://wiki.winehq.org/Regression_Testing -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=52132 Gijs Vermeulen <gijsvrm(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO |NEW Summary|DeviceCapabilities |DeviceCapabilities(DC_PAPER |DC_PAPERSIZE return wrong |SIZE) returns wrong size |sizes (Regression) | Keywords| |regression --- Comment #2 from Gijs Vermeulen <gijsvrm(a)gmail.com> --- Not having a regression test done doesn't warrant a NEEDINFO imo. At least we haven't done that in the past. It would, however, be nice to know what the output for the sample is on wine-6.15 and what it is on wine-git. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=52132 --- Comment #3 from Konstantin <incubusrk(a)gmail.com> --- I check wine 6.17 it's work fine, in wine 6.18 printing was broken. I publish related bug report https://bugs.winehq.org/show_bug.cgi?id=51815 In wine 6.20 printing was partialy restored. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=52132 --- Comment #4 from Konstantin <incubusrk(a)gmail.com> --- Created attachment 71174 --> https://bugs.winehq.org/attachment.cgi?id=71174 test app logs for wine 6.17,6.18,6.19,6.20,6.22 -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=52132 --- Comment #5 from Konstantin <incubusrk(a)gmail.com> --- Created attachment 71175 --> https://bugs.winehq.org/attachment.cgi?id=71175 WINEDEBUG=+psdrv log output, look like size type cast may be сause of error -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=52132 --- Comment #6 from Konstantin <incubusrk(a)gmail.com> --- Created attachment 71176 --> https://bugs.winehq.org/attachment.cgi?id=71176 WINEDEBUG=+psdrv log for wine-6.17 where all works fine -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=52132 --- Comment #7 from Konstantin <incubusrk(a)gmail.com> --- https://source.winehq.org/git/wine.git/blob/HEAD:/dlls/wineps.drv/driver.c#l... I assume that change POINT16 to POINT will solve the problem -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=52132 --- Comment #8 from Konstantin <incubusrk(a)gmail.com> --- Created attachment 71177 --> https://bugs.winehq.org/attachment.cgi?id=71177 builld with this patch works fine for me -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=52132 Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |leslie_alistair(a)hotmail.com --- Comment #9 from Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> --- Patches arent picked from bug tracker, you need to send them for review wine-patches list. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=52132 --- Comment #10 from Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> --- (In reply to Alistair Leslie-Hughes from comment #9)
Patches arent picked from bug tracker, you need to send them for review wine-patches list.
That should of been wine-devel list. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=52132 --- Comment #11 from Hans Leidekker <hans(a)meelstraat.net> --- Is this fixed with 62e0f51e6e22deaaea6283ce541dbc7111e01faa? -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=52132 --- Comment #12 from Konstantin <incubusrk(a)gmail.com> --- Yes, patch was commited -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=52132 Gijs Vermeulen <gijsvrm(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Fixed by SHA1| |62e0f51e6e22deaaea6283ce541 | |dbc7111e01faa Resolution|--- |FIXED --- Comment #13 from Gijs Vermeulen <gijsvrm(a)gmail.com> --- (In reply to Konstantin from comment #12)
Yes, patch was commited
Marking fixed, thanks for the patch and for reporting back. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=52132 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #14 from Alexandre Julliard <julliard(a)winehq.org> --- Closing bugs fixed in 7.0-rc1. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
participants (1)
-
WineHQ Bugzilla