[Bug 47278] New: Europa Universalis 4 Golden Century require TGA support in D3DXSaveSurfaceToFileInMemory
https://bugs.winehq.org/show_bug.cgi?id=47278 Bug ID: 47278 Summary: Europa Universalis 4 Golden Century require TGA support in D3DXSaveSurfaceToFileInMemory Product: Wine Version: 4.9 Hardware: x86-64 OS: Linux Status: UNCONFIRMED Severity: minor Priority: P2 Component: directx-d3dx9 Assignee: wine-bugs(a)winehq.org Reporter: lorenzofer(a)live.it Distribution: --- HI. As the title says, Europa Universalis 4 Golden Century (so version 1.28.3 Spain), require TGA file support for D3DXSaveSurfaceToFileInMemory function. In the log there is : 0057:fixme:d3dx:D3DXSaveSurfaceToFileInMemory File format 0x2 is not supported yet After the line the application catch a MEMORY_ACCESS_VIOLATION and close itself. In the log there is also a bunch of: 0057:fixme:d3dx:D3DXLoadSurfaceFromMemory Unhandled filter 0x80004. For now winetricks d3d9x_43 is a workaround -- 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=47278 --- Comment #1 from Lorenzo Ferrillo <lorenzofer(a)live.it> --- Created attachment 64578 --> https://bugs.winehq.org/attachment.cgi?id=64578 D3DXSaveSurfaceToFileInMemory allow TGA destination format This patch allow the game to work without native d3dx9_43.dll Note that to fully work it will require native d3dcompiler. -- 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=47278 --- Comment #2 from Lorenzo Ferrillo <lorenzofer(a)live.it> --- Created attachment 64579 --> https://bugs.winehq.org/attachment.cgi?id=64579 Add test for TGA Also the relevant test addition I submitted a job on the testbot. If it don't fail then I will try to submit the patch on devel -- 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=47278 Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |patch -- 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=47278 pattietreutel <katyaberezyaka(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |katyaberezyaka(a)gmail.com -- 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=47278 Anastasius Focht <focht(a)gmx.net> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |focht(a)gmx.net Summary|Europa Universalis 4 Golden |Multiple games and |Century require TGA support |applications require TGA |in |support in |D3DXSaveSurfaceToFileInMemo |D3DXSaveSurfaceToFileInMemo |ry |ry (Europa Universalis 4 | |Golden Century, ShaderMap | |4.x) URL| |https://web.archive.org/web | |/20201001100355/https://sha | |dermap.com/downloads/_softw | |are.php?v=4 Ever confirmed|0 |1 Keywords| |download Status|UNCONFIRMED |NEW --- Comment #3 from Anastasius Focht <focht(a)gmx.net> --- Hello folks, ShaderMap 4.x from bug 48217 also needs this: https://web.archive.org/web/20201001100355/https://shadermap.com/downloads/_... --- snip --- $ pwd /home/focht/.wine/drive_c/Program Files/ShaderMap 4/bin $ wine ./ShaderMap.exe ... wine: Call from 000000007B012AF2 to unimplemented function msvcr120.dll.?_NewCollection(a)_AsyncTaskCollection@details(a)Concurrency@@SAPEAV123(a)PEAV_CancellationTokenState@23@@Z, aborting wine: Unimplemented function msvcr120.dll.?_NewCollection(a)_AsyncTaskCollection@details(a)Concurrency@@SAPEAV123(a)PEAV_CancellationTokenState@23@@Z called at address 000000007B012AF2 (thread 00c0), starting debugger... --- snip --- After 'winetricks -q vcrun2013': --- snip --- 00c0:fixme:d3dcompiler:compile_shader Compilation target "fx_2_0" not yet supported --- snip --- -> 'winetricks -q d3dcompiler_47' --- snip --- 01f4:fixme:d3dx:D3DXLoadSurfaceFromMemory Unhandled filter 0x80004. 01f4:fixme:d3dx:D3DXSaveSurfaceToFileInMemory File format 0x2 is not supported yet --- snip --- $ sha1sum ShaderMap_v4_3_4.exe 9c375685823246e0b8d76fbdb8288a47ad01fdbd ShaderMap_v4_3_4.exe $ du -sh ShaderMap_v4_3_4.exe 46M ShaderMap_v4_3_4.exe $ wine --version wine-6.0-rc2 Regards -- 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=47278 --- Comment #4 from Gijs Vermeulen <gijsvrm(a)gmail.com> --- Created attachment 76429 --> https://bugs.winehq.org/attachment.cgi?id=76429 slightly updated patch Still present with wine-9.8. I've attached a slightly updated patch. Lorenzo, did you ever send this upstream and if not, would you mind trying now? -- 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=47278 Vijay Kamuju <infyquest(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |infyquest(a)gmail.com --- Comment #5 from Vijay Kamuju <infyquest(a)gmail.com> --- (In reply to Gijs Vermeulen from comment #4)
Created attachment 76429 [details] slightly updated patch
Still present with wine-9.8. I've attached a slightly updated patch.
Lorenzo, did you ever send this upstream and if not, would you mind trying now? It would be better you send in the MR as it would take a lot of time, mentioning that its based on patch from Lorenzo.
-- 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=47278 Connor McAdams <cmcadams(a)codeweavers.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |cmcadams(a)codeweavers.com --- Comment #6 from Connor McAdams <cmcadams(a)codeweavers.com> --- This might be fixed by 32a81ee2135c67e393e49cce60ef8ebfbaefb531, assuming the surface being saved is a direct match to one of the supported TGA saving formats. Probably worth retesting. -- 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=47278 Gijs Vermeulen <gijsvrm(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Fixed by SHA1| |32a81ee2135c67e393e49cce60e | |f8ebfbaefb531 Status|NEW |RESOLVED --- Comment #7 from Gijs Vermeulen <gijsvrm(a)gmail.com> --- ShaderMap does seem to open now, albeit with a hack to make d3dx_effect_OnResetDevice return S_OK. Can't really test that app further, as I have no idea how to use it. I don't have Europa Universalis to test, but I think we can mark this one fixed. Feel free to reopen if you can still reproduce. -- 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=47278 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #8 from Alexandre Julliard <julliard(a)winehq.org> --- Closing bugs fixed in 10.4. -- 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)
-
wine-bugs@winehq.org -
WineHQ Bugzilla