[Bug 49453] New: ExtractNow: buttons do nothing / don't work / function (needs comctl32 version 6)
https://bugs.winehq.org/show_bug.cgi?id=49453 Bug ID: 49453 Summary: ExtractNow: buttons do nothing / don't work / function (needs comctl32 version 6) Product: Wine Version: 5.11 Hardware: x86 URL: https://www.extractnow.com/ OS: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: comctl32 Assignee: wine-bugs(a)winehq.org Reporter: spoon0042(a)hotmail.com Distribution: Debian This came up on the forum and I was bored. ExtractNow is a simple looking archive extractor thing. Problem is the buttons in the main window don't do anything on mainline wine. Works on staging though, tracked it down to this patch: commit 14c33edf03750722193658b9f9a72fc21b7fb988 Author: Dmitry Timoshkov <dmitry(a)baikal.ru> Date: Tue Nov 12 18:13:20 2019 +0800 comctl32: Bump version to 6.0. An application that I have here checks comctl32.dll version information and refuses to run, changing DLL version to 6.0 makes it run. Signed-off-by: Dmitry Timoshkov <dmitry(a)baikal.ru> :040000 040000 2e914614641b09648642ff928d0777005f8ad070 99f4dbca57a429a7d038e014d0dd1b68a6b3ffaf M dlls :040000 040000 89a1c48068de16510fc9ca1d1eb1800849fc1065 a501c8d94188aa8d9c5f03435b4ee7dd370045d7 M include -- 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=49453 --- Comment #1 from Nikolay Sivov <bunglehead(a)gmail.com> --- Does it run on Windows without embedded manifest? (using comctl32 v5) -- 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=49453 Michael McGuire <spoon0042(a)hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |dmitry(a)baikal.ru --- Comment #2 from Michael McGuire <spoon0042(a)hotmail.com> --- I'm not sure what that means? Would I need to try the program on an XP install? Or whatever version, I don't know what windows version has which comctl version. -- 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=49453 --- Comment #3 from Dmitry Timoshkov <dmitry(a)baikal.ru> --- (In reply to Michael McGuire from comment #2)
I'm not sure what that means? Would I need to try the program on an XP install? Or whatever version, I don't know what windows version has which comctl version.
This needs an investigation: if an application checks the DLL version what does it do then? Perhaps it depends on some specific behaviour? Does the application work under XP? -- 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=49453 --- Comment #4 from Michael McGuire <spoon0042(a)hotmail.com> --- (In reply to Dmitry Timoshkov from comment #3)
This needs an investigation: if an application checks the DLL version what does it do then? Perhaps it depends on some specific behaviour? Does the application work under XP?
My hunch is this particular application isn't checking at all. Whatever that's worth. I did fire up an XP SP3 vm and it does work. I think though that's already using version 6. I found a page detailing comctl32 versions and if I'm reading it right any version of XP has v6 available. I could try something older assuming I still have any. Or if there's some way to force v5. Don't know. https://www.geoffchappell.com/studies/windows/shell/comctl32/history/index.h... -- 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=49453 --- Comment #5 from Michael McGuire <spoon0042(a)hotmail.com> --- Or I guess that doesn't make sense since all that patch does is change the version. So yeah, don't know. -- 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=49453 joaopa <jeremielapuree(a)yahoo.fr> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jeremielapuree(a)yahoo.fr --- Comment #6 from joaopa <jeremielapuree(a)yahoo.fr> --- Confirming. Can be tested with https://www.01net.com/telecharger/windows/Utilitaire/compression_et_decompre... -- 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=49453 --- Comment #7 from Nikolay Sivov <bunglehead(a)gmail.com> --- XP is a first release to provide version 6 along with version 5, and all followed releases do the same, so version 5 remains supported in all current Windows 10 builds. What I was asking as a test, was to remove manifest resource from inside main application exe, that specifies comctl32 v6 dependency. Without manifest it will load version 5 module, and the question is if application is able to deal with this properly. That of course is not a hard proof, because presence of the context is visible to application too and might trigger different path, but at least it gives some insight. -- 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=49453 --- Comment #8 from joaopa <jeremielapuree(a)yahoo.fr> ---
What I was asking as a test, was to remove manifest resource from inside main application exe
And how do you do that? -- 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=49453 --- Comment #9 from Michael McGuire <spoon0042(a)hotmail.com> --- Seems to be there in plain text, amazingly enough. Even more amazingly, I naively snipped the Common Controls dependency bit out and extractnow didn't seem to mind and appears to work. Hopefully this is useful information? -- 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.
http://bugs.winehq.org/show_bug.cgi?id=49453 Andrew Nguyen <arethusa26@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |RESOLVED Resolution|--- |FIXED Fixed by SHA1| |541d9be5ecda1bc0c8fce22f131 | |8bf5d72572327 Keywords| |download --- Comment #10 from Andrew Nguyen <arethusa26@gmail.com> --- I confirm the issue with the buttons in the ExtractNow main window occurs with wine-5.11 and no longer occurs with wine-11.2. For whatever reason, the first commit that allows the buttons to work is deaf0891b096628cf99e58834c39a45a6c99c54c, available in wine-7.10. However, there are other issues with the application controls that are not fully addressed until the comctl32 separation with commit 541d9be5ecda1bc0c8fce22f1318bf5d72572327, available in wine-10.17. As the ExtractNow application controls appear to be fully working with the latest Wine version, I'm resolving this bug as fixed. -- 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.
http://bugs.winehq.org/show_bug.cgi?id=49453 Alexandre Julliard <julliard@winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #11 from Alexandre Julliard <julliard@winehq.org> --- Closing bugs fixed in 11.3. -- 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 (2)
-
WineHQ Bugzilla -
WineHQ Bugzilla