[Bug 14939] New: DXT1 support not implemented
http://bugs.winehq.org/show_bug.cgi?id=14939 Summary: DXT1 support not implemented Product: Wine Version: 1.1.2 Platform: PC-x86-64 OS/Version: Linux Status: UNCONFIRMED Severity: minor Priority: P2 Component: directx-d3d AssignedTo: wine-bugs(a)winehq.org ReportedBy: shacklein(a)gmail.com Running Black & White, regardless of in-game settings, the following fixme occurs: fixme:d3d_surface:surface_convert_format Cannot find a conversion function from format WINED3DFMT_DXT1 to WINED3DFMT_A4R4G4B4 Although the game is still playable, the lack of DXT1 support does seriously affect the graphics and gameplay. On "lowest" settings, the terrain is transparent and some objects are untextured and 100% black; on "highest", some textures are corrupted, or untextured as in "lowest" settings. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 --- Comment #1 from Roderick Colenbrander <thunderbird2k(a)gmx.net> 2008-08-21 18:00:06 --- DXTC requires the opengl s3tc extension. On open source drivers you need to install a custom library for mesa to get it as the compression isn't shipped by default since it is patented. For that reason we can't also add it to Wine. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 --- Comment #2 from Ben Klein <shacklein(a)gmail.com> 2008-08-21 18:09:44 --- (In reply to comment #1)
DXTC requires the opengl s3tc extension. On open source drivers you need to install a custom library for mesa to get it as the compression isn't shipped by default since it is patented. For that reason we can't also add it to Wine.
I see. And presumably, native dlls wouldn't help here either. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 --- Comment #3 from Ben Klein <shacklein(a)gmail.com> 2008-08-21 18:47:02 --- I think I misunderstood you before. The problem is legal/availability issues of S3TC, correct? I don't understand the underlying technology very much, but am I right in saying that S3TC is required for DXTC hardware support? If so, would it be possible (as in, legally) to implement a DXTC converter in software? -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 Lei Zhang <thestig(a)google.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|DXT1 support not implemented|Black & White: DXT1 support | |not implemented -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 Chris Meier <cm(a)chrismeier.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |NEW Ever Confirmed|0 |1 --- Comment #4 from Chris Meier <cm(a)chrismeier.de> 2008-09-07 18:50:27 --- *** This bug has been confirmed by popular vote. *** -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 --- Comment #5 from Ben Klein <shacklein(a)gmail.com> 2008-09-07 22:16:03 --- After some research, I now understand more about the problem here. Wine can't implement a DXTC/S3TC encoder without violating the S3TC patent, and probably the same goes for a decoder. A while ago in #winehackers someone suggested that this particular decoding function could be offloaded to the GPU (when the appropriate functionality is available). The process would be essentially: 1. Load S3TC texture into a PBO 2. GPU converts to ARGB 3. Wine reads back the converted texture from PBO Of course, this won't work when the GPU doesn't support S3TC/DXTC textures. I've been meaning to write a PoC for this, but haven't had the time. Comments? -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 --- Comment #6 from Roderick Colenbrander <thunderbird2k(a)gmx.net> 2008-09-20 08:32:45 --- Likely you could implement the algorithm that way but the drivers which offers PBOs are also the ones which offer S3TC. (Only Nvidia and ATI offer this at this point and I doubt PBOs will be supported on much older GPUs though if they support VBOs it is possible as PBOs are an extension to those) -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 John Haywards <normandy(a)web.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |normandy(a)web.de --- Comment #7 from John Haywards <normandy(a)web.de> 2008-09-20 11:19:43 --- *** Bug 15340 has been marked as a duplicate of this bug. *** -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 Ben Klein <shacklein(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|Black & White: DXT1 support |Black & White: DXTC to ARGB |not implemented |converter not implemented --- Comment #8 from Ben Klein <shacklein(a)gmail.com> 2008-09-20 18:52:30 --- (In reply to comment #6)
Likely you could implement the algorithm that way but the drivers which offers PBOs are also the ones which offer S3TC. (Only Nvidia and ATI offer this at this point and I doubt PBOs will be supported on much older GPUs though if they support VBOs it is possible as PBOs are an extension to those)
What I'm wondering is, is it worth it to implement this algorithm that only works on particular GPU and driver pairs? I would vote "yes" on this, but it would have to be done in such a way that it wouldn't be more broken if S3TC support is not found in the driver. Also, adjusted description to be more accurate. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 --- Comment #9 from Austin English <austinenglish(a)gmail.com> 2009-03-30 12:54:10 --- Is this still an issue in current (1.1.18 or newer) wine? -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 --- Comment #10 from Ben Klein <shacklein(a)gmail.com> 2009-03-30 21:17:23 --- Created an attachment (id=20214) --> (http://bugs.winehq.org/attachment.cgi?id=20214) Tail of wine crash output on 1.1.18, including the backtrace -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 --- Comment #11 from Ben Klein <shacklein(a)gmail.com> 2009-03-30 21:28:42 --- There's been a regression in Wine that causes B&W to crash shortly after starting. I've attached the backtrace (above). It's in Wine's texturing code, interesting :) And regarding whether the bug has been fixed, it can *only* be fixed by one of the following solutions: 1) Resolving the patented s3tc/DXTC issues and implementing a software DXTC to ARGB converter 2) Deferring decoding s3tc/DXTC textures to the graphics card where s3tc is supported, then read back the texture and tell the app "here is your converted texture" (like what I suggest in Comment #2, which in turn was suggested by people in #winehackers) I don't think either of these things have been done. In the duplicate Bug 15340, Roderick mentions the s3tc patches for mesa. They don't help here (thought they would for solution #2 if it was implemented) because the app isn't trying to load an s3tc texture into the graphics card, it's got an s3tc/DXTC texture that it wants converted to ARGB by DirectX and/or the video driver, then load the ARGB texture into the graphics card. I'll be trying to determine where the regression that causes the current crashiness occurs :) -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 --- Comment #12 from Ben Klein <shacklein(a)gmail.com> 2009-03-31 11:16:10 --- I can now confirm that there is still no DXTC to ARGB converter as of 1.1.18. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 Austin English <austinenglish(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |taril_laszlo(a)yahoo.co.uk --- Comment #13 from Austin English <austinenglish(a)gmail.com> 2009-04-05 14:34:52 --- *** Bug 15308 has been marked as a duplicate of this bug. *** -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 Paul "TBBle" Hampson <Paul.Hampson(a)Pobox.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |Paul.Hampson(a)Pobox.com --- Comment #14 from Paul "TBBle" Hampson <Paul.Hampson(a)Pobox.com> 2009-04-19 02:38:30 --- Why not copy the Mesa route described at http://dri.freedesktop.org/wiki/S3TC and try using an external library which it is up to the user or packager to supply, assuming they have complied with the relevant licensing requirements? If we can't dlopen the library, then fallback to whatever we do now. Feeding it off the GPU could be done either before or after the library, basically depending on performance I'd think, but neither path depends on the other that I can see. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 Vitaliy Margolen <vitaliy(a)kievinfo.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |xamaniqinqu(a)gmail.com --- Comment #15 from Vitaliy Margolen <vitaliy(a)kievinfo.com> 2009-09-27 11:30:07 --- *** Bug 20183 has been marked as a duplicate of this bug. *** -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 --- Comment #16 from Austin English <austinenglish(a)gmail.com> 2010-05-26 15:12:09 --- Still present. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 --- Comment #17 from Tuomas Lukinmaa <winebugs(a)tumu.iki.fi> 2010-10-06 00:50:55 CDT --- Also affects Orbiter 2010. The source library provided in #14 is 404, so the workaround is slowly turning extinct. http://orbit.medphys.ucl.ac.uk/ -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 --- Comment #18 from Itzamna <xamaniqinqu(a)gmail.com> 2010-12-04 12:03:38 CST --- Created an attachment (id=32355) --> (http://bugs.winehq.org/attachment.cgi?id=32355) Implements DXT1 to ARGB4444 texture conversion. Tested against commit b898130a114f1baf24f717ab0c5f9a65030c5283. I have written a patch implementing DXT1 to ARGB4444 texture conversion. I pushed for inclusion in mainline, but was told this algorithm is patented. I decided to give VIA and S3 the privilege of sitting on my finger and write this patch anyway. Then I pushed the patent to the ground and had my way with it thoroughly. It wasn't as pleasant for the patent as it was for me. More patches are coming, most notably to implement XRGB1555 to DXT1, DXT1 to XRGB1555 and DXT3 to ARGB4444 texture conversion. The patches handling XRGB1555 textures will fix the crash upon finishing the temple in land 1. I advise anyone who wishes to try this to check out commit b898130a114f1baf24f717ab0c5f9a65030c5283, since the commit after it ( 7e3985e1fea60da73bae2b9465b94e5d110909fc) causes left / right mouse button clicks to be ignored in Black & White. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 --- Comment #19 from Itzamna <xamaniqinqu(a)gmail.com> 2010-12-04 12:16:47 CST --- To use this patch and not be affected by bug 25417 (http://bugs.winehq.org/show_bug.cgi?id=25417), copy the following commands in a terminal: mkdir ~/src cd src git clone git://source.winehq.org/git/wine.git ~/wine cd wine git checkout b898130a114f1baf24f717ab0c5f9a65030c5283 wget -O wine-dxt1-to-argb4444-conversion.patch http://bugs2.winehq.org/attachment.cgi?id=32355 patch -p1 < wine-dxt1-to-argb4444-conversion.patch ./configure --prefix=/usr make depend make sudo make install By the way: DXT1 textures are used as low resolution textures in Black & White, whereas DXT3 textures are used as high resolution textures. Until I post a DXT3 -> ARGB4444 patch, go to the Black & White installation directory and type 'wine Setup.exe'. Then go to the 'Graphics Detail Level' slider and select 'Custom Detail'. Then click the 'Custom...' button, uncheck "Use hi-res textures" and check "Use detail textures". -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 --- Comment #20 from Itzamna <xamaniqinqu(a)gmail.com> 2010-12-04 15:52:20 CST --- Created an attachment (id=32358) --> (http://bugs.winehq.org/attachment.cgi?id=32358) Implements DXT1 to XRGB1555 texture conversion. This incremental patch adds support for DXT1 -> XRGB1555 texture conversion, required for the temple surface. Once I post an XRGB1555 -> DXT1 patch, the crash will be solved. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 --- Comment #21 from Ben Klein <shacklein(a)gmail.com> 2010-12-04 17:42:52 CST --- (In reply to comment #18)
Created an attachment (id=32355) --> (http://bugs.winehq.org/attachment.cgi?id=32355) [details] Implements DXT1 to ARGB4444 texture conversion. Tested against commit b898130a114f1baf24f717ab0c5f9a65030c5283.
I have written a patch implementing DXT1 to ARGB4444 texture conversion. I pushed for inclusion in mainline, but was told this algorithm is patented.
Rude language aside, thanks for looking in to this. Let's just hope that there are no nasty legal consequences! -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 --- Comment #22 from Itzamna <xamaniqinqu(a)gmail.com> 2010-12-04 17:57:59 CST --- (In reply to comment #21)
Rude language aside, thanks for looking in to this. Let's just hope that there are no nasty legal consequences!
Rude language? What? :p I just managed to tell S3 to go (...) themselves without resorting to the F-word, that's just classy. Have you tried out the patch(es) yet? Tell me if you find any problems. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 Itzamna <xamaniqinqu(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #32355|0 |1 is obsolete| | Attachment #32358|0 |1 is obsolete| | --- Comment #23 from Itzamna <xamaniqinqu(a)gmail.com> 2010-12-05 19:38:19 CST --- Created an attachment (id=32374) --> (http://bugs.winehq.org/attachment.cgi?id=32374) Implements DXT1, DXT3 to ARGB4444 and XRGB1555 texture conversion. This is a full (non-incremental) patch. Preferrably apply against commit b898130a114f1baf24f717ab0c5f9a65030c5283. I have now implemented DXT3 -> ARGB4444 texture conversion, enabling high resolution textures. There is only one bug left to tackle (15984), that one requires an XRGB1555 -> DXT1 conversion function. When that has been completed, likely tomorrow, Black & White will no longer crash when saving or entering a skirmish game and will be eligible for platinum status in the AppDB. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 Dan Kegel <dank(a)kegel.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |dank(a)kegel.com --- Comment #24 from Dan Kegel <dank(a)kegel.com> 2010-12-05 19:56:12 CST --- Platinum's only for games that run well on unpatched wine. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 --- Comment #25 from Itzamna <xamaniqinqu(a)gmail.com> 2010-12-05 20:15:06 CST --- (In reply to comment #24)
Platinum's only for games that run well on unpatched wine.
Whoops. Well, I suppose a gold rating accompanied with a thorough howto and .deb and .rpm packages built from the patched source will be almost as snazzy. Working on that XRGB1555 -> DXT1 patch now, almost there :) -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 jannis <kripton(a)kripserver.net> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |kripton(a)kripserver.net -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 James Le Cuirot <chewi(a)aura-online.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |chewi(a)aura-online.co.uk -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 Fabio <fabio.ped(a)libero.it> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |fabio.ped(a)libero.it -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 nh2(a)deditus.de changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |nh2(a)deditus.de -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 --- Comment #26 from Ben Klein <shacklein(a)gmail.com> 2011-03-23 06:14:26 CDT --- (In reply to comment #25)
(In reply to comment #24)
Platinum's only for games that run well on unpatched wine.
Whoops. Well, I suppose a gold rating accompanied with a thorough howto and .deb and .rpm packages built from the patched source will be almost as snazzy.
Working on that XRGB1555 -> DXT1 patch now, almost there :)
Just had the chance to try out your patch attached here. Amazing work! Two things: 1) What happened to your XRGB1555 encoder? 2) Is there any way you can implement this so it defers the actual decompression algorithm to OpenGL/Mesa? At the moment, there's no way this patch will be included in upstream wine. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 --- Comment #27 from Itzamna <xamaniqinqu(a)gmail.com> 2011-03-23 07:16:46 CDT --- Hello Ben, I hope you're enjoying Black & White - it's been a long time coming. 1) The XRGB1555 -> DXT135 compression algorithm is no longer essential to running Black & White, as it's only used to compress the screenshots into thumbnails when one saves a game. The crashes were caused by a shoddy implementation of DirectDraw blitting (used when making these savegame screenshots), Zdenek Behan has provided a hack that bypasses blitting. In the HowTo I have included all 3 patches needed for playing Black & White on Wine. 2) Personally I think this patent fear is nonsense, since only a backwards, oppressive and capitalist legal climate as in the U.S. would allow patents to kill innovation. If you like, I could write a patch to hook into libtxc_dxtn, though there are two concerns: A) the overhead produced by converting the 16 bit textures to 32 bit (libtdx_dxtn works with 32 bit textures) and then performing a function call to libtdx_dxtn.so would be just silly and B) the Mesa guys are finally turning on the light in their attics and are discussing providing DXT support natively in Mesa ( http://lists.freedesktop.org/archives/mesa-dev/2011-March/005826.html ). I'll push a patch to provide fast, direct compression / decompression of 16 bit (A)RGB textures to Mesa when they've completed the merger. For now, just follow the HowTo on the AppDB page and you're good to go. If you want an online match sometime, give me a yell! My rhino is feeling lonesome. =( -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 --- Comment #28 from Ben Klein <shacklein(a)gmail.com> 2011-03-23 07:50:29 CDT --- (In reply to comment #27)
Hello Ben, I hope you're enjoying Black & White - it's been a long time coming.
1) The XRGB1555 -> DXT135 compression algorithm is no longer essential to running Black & White, as it's only used to compress the screenshots into thumbnails when one saves a game. The crashes were caused by a shoddy implementation of DirectDraw blitting (used when making these savegame screenshots), Zdenek Behan has provided a hack that bypasses blitting. In the HowTo I have included all 3 patches needed for playing Black & White on Wine.
Thanks :)
2) Personally I think this patent fear is nonsense, since only a backwards, oppressive and capitalist legal climate as in the U.S. would allow patents to kill innovation.
I agree with you. I'd even argue that implementing the S3TC algorithms from scratch is not a patent violation as long as you don't reverse-engineer. But I'm not a patent expert, an nor are the wine devs, so as it stands this sort of software decoder won't get into wine upstream.
If you like, I could write a patch to hook into libtxc_dxtn, though there are two concerns: A) the overhead produced by converting the 16 bit textures to 32 bit (libtdx_dxtn works with 32 bit textures) and then performing a function call to libtdx_dxtn.so would be just silly and
I suggested it mainly because it'd be more likely to be accepted by AJ.
B) the Mesa guys are finally turning on the light in their attics and are discussing providing DXT support natively in Mesa ( http://lists.freedesktop.org/archives/mesa-dev/2011-March/005826.html ). I'll push a patch to provide fast, direct compression / decompression of 16 bit (A)RGB textures to Mesa when they've completed the merger.
This is good news then :) Something to keep an eye on.
For now, just follow the HowTo on the AppDB page and you're good to go. If you want an online match sometime, give me a yell! My rhino is feeling lonesome. =(
Poor rhino! -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 --- Comment #29 from James Le Cuirot <chewi(a)aura-online.co.uk> 2011-03-26 17:29:16 CDT --- I just want to say thank you to Itzamna for his hard work. It works perfectly now and my wife has been playing for hours! -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 Itzamna <xamaniqinqu(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #32374|0 |1 is obsolete| | --- Comment #30 from Itzamna <xamaniqinqu(a)gmail.com> 2011-03-30 19:47:15 CDT --- Created an attachment (id=33871) --> (http://bugs.winehq.org/attachment.cgi?id=33871) Rebase DXT patch on latest git commit (da4ed4707683687254d4abcb63f8abdd0ba7b313). More recent git commits broke the patch, here's an update. Also, you may have noticed DXT3 textures sometimes get decompressed with jagged edges... I'll look into it in the weekend, it may have something to do with alpha pixels or with blending. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 --- Comment #31 from Fabio <fabio.ped(a)libero.it> 2011-05-04 08:46:41 CDT --- (In reply to comment #27)
B) the Mesa guys are finally turning on the light in their attics and are discussing providing DXT support natively in Mesa ( http://lists.freedesktop.org/archives/mesa-dev/2011-March/005826.html ).
It looks like this will never happen. They merged the floating textures (enabled with the --enable-float configure flag, but Linux distribution are unlikely to enable it) because it was infeasible to provide them in an external library, but the DXT texture support will stay in libtxc-dxtn, at least up until the S3 patent is valid. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 --- Comment #32 from nickstuckert(a)gmail.com <nickstuckert(a)gmail.com> 2011-05-08 23:49:35 CDT --- Created an attachment (id=34559) --> (http://bugs.winehq.org/attachment.cgi?id=34559) Rebase DXT patch May-09-2011 This is the new patch to address the name change from surface_base.c to surface.c in the dlls/wined3d folder -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 nickstuckert(a)gmail.com <nickstuckert(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |nickstuckert(a)gmail.com --- Comment #33 from nickstuckert(a)gmail.com <nickstuckert(a)gmail.com> 2011-05-08 23:50:32 CDT --- Hey Itzamna, the file name changed for the patch so I remade it. Thanks for all your help. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 San Zamoyski <san(a)plusnet.pl> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |san(a)plusnet.pl --- Comment #34 from San Zamoyski <san(a)plusnet.pl> 2012-05-14 02:25:54 CDT --- Which one source version should i download to get it to work? I downloaded few wersions of wine's source. 1.1.2, 1.1.27, 1.1.5, 1.5.4 But cannot patch or compile it. Tried to use both patches (33871 and 34559). -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 Damian Ivanov <damianatorrpm(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |damianatorrpm(a)gmail.com --- Comment #35 from Damian Ivanov <damianatorrpm(a)gmail.com> 2012-06-26 02:27:58 CDT --- Is still an issue in wine 1.5.6 or newer? -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 Stefan B. <steve56(a)web.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |steve56(a)web.de --- Comment #36 from Stefan B. <steve56(a)web.de> 2012-12-31 12:36:36 CST --- Can't start it even on wine 1.5.20 on ubuntu 12.10 ... tried it with black and white version 1.0 and patched 1.3 ... even with no-cd crack. Sometimes it begins to start, shows the two splash screens, but freezes and does only show a black screen :-( -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 --- Comment #37 from nh2(a)deditus.de 2013-01-01 20:41:25 CST --- I have no clue about the exact details of this, but by now texture compression is AFAIK provided by the package libtxc-dxtn-s2tc0 (this is its name in Ubuntu), which happens to fix many black-screen issues. Please give it a try, install it and tell us if it changes something. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 zippy <gohabsgo60(a)yahoo.ca> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |gohabsgo60(a)yahoo.ca -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email 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=14939 Linus Unnebäck <linus(a)folkdatorn.se> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |linus(a)folkdatorn.se --- Comment #38 from Linus Unnebäck <linus(a)folkdatorn.se> --- This is still an issue in wine 1.7.5. Haven't been able to try in 1.7.13, see Bug 35721. Building with the patch fixes most of the issues but leaves a few. 1. All the sprites for trees haven't any alpha, making them rectangular with a greenish background. 2. Some textures still seem to be missing, for example the Celtic temple and some parts of Yin and Yan. The game still tries to use WINED3DFMT_B5G5R5X1_UNORM to WINED3DFMT_DXT1 conversion, I don't know for what but maybe implementing it could help with the missing textures. I also get lots and lots of these output lines: fixme:d3d_surface:surface_load_location Unimplemented copy from SFLAG_INTEXTURE to SFLAG_INSYSMEM for depth/stencil buffers. and err:ddraw:d3d_device_prepare_vertex_buffer (0x1b6b70) wined3d_buffer_create_vb failed with hr = 8876017c -- 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=14939 Vladimir <didenkovladimir12(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |didenkovladimir12(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=14939 Sebastian Lackner <sebastian(a)fds-team.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |michael(a)fds-team.de, | |sebastian(a)fds-team.de --- Comment #39 from Sebastian Lackner <sebastian(a)fds-team.de> --- Patch was added to Wine-Staging: https://github.com/wine-compholio/wine-staging/tree/master/patches/wined3d-D... (needs 'autoreconf -f' and 32-bit libtxc-s2tc to be present at ./configure time) It might be a bit slower than directly implementing DXTn support in Wine, but is a bit more safe from a legal perspective. -- 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=14939 Alec Teal <alec.teal(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |alec.teal(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=14939 --- Comment #40 from Alec Teal <alec.teal(a)gmail.com> --- (In reply to Sebastian Lackner from comment #39)
Patch was added to Wine-Staging:
https://github.com/wine-compholio/wine-staging/tree/master/patches/wined3d- DXTn (needs 'autoreconf -f' and 32-bit libtxc-s2tc to be present at ./configure time)
It might be a bit slower than directly implementing DXTn support in Wine, but is a bit more safe from a legal perspective.
Please do not give up on this! I'd love to play this game! -- 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=14939 Sam <sam.h.russell(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |sam.h.russell(a)gmail.com --- Comment #41 from Sam <sam.h.russell(a)gmail.com> --- Created attachment 52372 --> https://bugs.winehq.org/attachment.cgi?id=52372 Rebase patch for 1.7.51 -- 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=14939 Sam Russell <sam.h.russell(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #52372|0 |1 is obsolete| | --- Comment #42 from Sam Russell <sam.h.russell(a)gmail.com> --- Created attachment 52384 --> https://bugs.winehq.org/attachment.cgi?id=52384 DXTC-ARGB converter with fixed DXT3 Alpha for 1.7.51 from your wine directory, run patch -p1 < wine-dxt13-to-argb4444-xrgb1555-conversion-dxt3-fixed-alpha-1.7.51.patch This fixes the alpha problem for trees in Black and White 1 with high quality textures -- 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=14939 Michael Müller <michael(a)fds-team.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |STAGED Staged patchset| |https://github.com/wine-com | |pholio/wine-staging/tree/ma | |ster/patches/wined3d-DXTn -- 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=14939 --- Comment #43 from Linus Unnebäck <linus(a)folkdatorn.se> --- I would LOVE for this to be included in mainline wine. Is there something I can do to help that process? -- 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=14939 Tiago Medeiros <tiago(a)azoreancoder.pw> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |tiago(a)azoreancoder.pw --- Comment #44 from Tiago Medeiros <tiago(a)azoreancoder.pw> --- (In reply to Linus Unnebäck from comment #43)
I would LOVE for this to be included in mainline wine. Is there something I can do to help that process?
This bug is staged, should only be a matter of time now, right? -- 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=14939 --- Comment #45 from Sebastian Lackner <sebastian(a)fds-team.de> --- (In reply to Tiago Medeiros from comment #44)
(In reply to Linus Unnebäck from comment #43)
I would LOVE for this to be included in mainline wine. Is there something I can do to help that process?
This bug is staged, should only be a matter of time now, right?
Not necessarily. The staging tree intentionally has less strict rules regarding patch acceptance. The fact that a staged patch is available, only means that there is at least one proposed solution, not necessarily that this is the solution which will be used in the development tree later. For users who need such STAGED features immediately, I would recommend to use either a prebuilt package of Wine Staging, or compile it manually with all patches you want. -- 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=14939 Austin English <austinenglish(a)gmail.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=14939 Józef Kucia <joseph.kucia(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |https://bugs.winehq.org/sho | |w_bug.cgi?id=25486 -- 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=14939 Mike McQuaid <mike(a)mikemcquaid.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |mike(a)mikemcquaid.com --- Comment #46 from Mike McQuaid <mike(a)mikemcquaid.com> --- Hi, Mike McQuaid, lead Homebrew maintainer here. Is there any chance of this patch getting merged for a future Wine stable release? Homebrew doesn't want to carry patches like this indefinitely as we consider upstream a better judge than us on the suitability of patches. If there's no chance in this getting merged we'll consider removing it and direct people to this bug report instead. Thanks for all your work on Wine! -- 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=14939 --- Comment #47 from Henri Verbeet <hverbeet(a)gmail.com> --- (In reply to Mike McQuaid from comment #46)
Hi, Mike McQuaid, lead Homebrew maintainer here. Is there any chance of this patch getting merged for a future Wine stable release? Homebrew doesn't want to carry patches like this indefinitely as we consider upstream a better judge than us on the suitability of patches. If there's no chance in this getting merged we'll consider removing it and direct people to this bug report instead. Thanks for all your work on Wine!
Which patch in particular are you referring to? The staging patch has pretty much no chance of getting merged. I think using libtxc_dxtn when available is fine, but the way this is implemented in staging isn't. That said, it's not clear to me whether that patch is even needed with current Wine. The last release anyone reported this occurring in was Wine 1.7.5 from 2013, in comment 38. -- 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=14939 --- Comment #48 from Mike McQuaid <mike(a)mikemcquaid.com> --- Thanks, I was referring to the staging patch. We'll remove it unless/until one of our users reconfirms it on 2.0.0 and even then I'd probably want to see it submitted in a way that could be accepted before we reinclude it. -- 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=14939 --- Comment #49 from Sam Russell <sam.h.russell(a)gmail.com> --- (In reply to Mike McQuaid from comment #48)
Thanks, I was referring to the staging patch. We'll remove it unless/until one of our users reconfirms it on 2.0.0 and even then I'd probably want to see it submitted in a way that could be accepted before we reinclude it.
Hi Mike, Henry, I submitted the updated version of the patch as I was experiencing this bug at the time (at the end of 2015). I can give it another test for you this weekend if you like, but I can't imagine how this could be resolved without this patch. The tl;dr is that there are fears around a software patent for this technology, which appears to be shipped on every graphics card released in the last decade, but requires a license to implement in software. The patent in question appears to expire in October 2017 (at which point there shouldn't be an issue with Wine merging this into stable?) In the meantime, the patch does serve a purpose, although it's debatable as to whether homebrew is the right place to host it. The current recipe does make things *very* easy to end users though. What are your thoughts? -- 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=14939 --- Comment #50 from Mike McQuaid <mike(a)mikemcquaid.com> --- Homebrew is happy to carry this patch if the merge concerns are patents but not if the patch isn't technically suitable for merging for other reasons. -- 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=14939 --- Comment #51 from Sam Russell <sam.h.russell(a)gmail.com> --- I'm not close enough to the project to know either way, but my guess would be that the methods that this patch adds line up with the other methods in the file (convert_yuy2_r5g6b5 etc). I'd want someone on the project to confirm though. -- 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=14939 --- Comment #52 from Matteo Bruni <matteo.mystral(a)gmail.com> --- (In reply to Sam Russell from comment #49)
I submitted the updated version of the patch as I was experiencing this bug at the time (at the end of 2015).
There seems to be some disagreement between Sam and Mike? Sam is probably referring to attachment 52384, from comment 42, which is not the staging patch series linked from this bug. Either way, the attached patch is not acceptable for upstreaming at the moment, if nothing else for the patent reasons mentioned.
I can give it another test for you this weekend if you like, but I can't imagine how this could be resolved without this patch.
425e84dac5c87f6b1e7d89b363017a045e4c0626 might have made the patch (whatever that is) unnecessary for Black & White. -- 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=14939 --- Comment #53 from Sam Russell <sam.h.russell(a)gmail.com> --- Confirming this patch is still necessary and effective as-is in Wine 2.0 Before patching: http://i.imgur.com/bu308Yc.png After patching: http://i.imgur.com/CY0Wg2A.jpg -- 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=14939 --- Comment #54 from Henri Verbeet <hverbeet(a)gmail.com> --- (In reply to Sam Russell from comment #53)
Confirming this patch is still necessary and effective as-is in Wine 2.0
Before patching: http://i.imgur.com/bu308Yc.png After patching: http://i.imgur.com/CY0Wg2A.jpg
What does the terminal output say? -- 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=14939 --- Comment #55 from Sam Russell <sam.h.russell(a)gmail.com> --- I'm not sure I follow your question. The textures never worked, they continue to not work. I don't know Wine well enough to debug further, but I am happy to confirm that I'm seeing identical symptoms on Wine 2.0 as I saw on Wine 1.7. -- 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=14939 --- Comment #56 from Henri Verbeet <hverbeet(a)gmail.com> --- (In reply to Sam Russell from comment #55)
I'm not sure I follow your question.
What does Wine 2.0 write to stderr when running the application? -- 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=14939 --- Comment #57 from Mike McQuaid <mike(a)mikemcquaid.com> ---
Either way, the attached patch is not acceptable for upstreaming at the moment, if nothing else for the patent reasons mentioned.
Could you suggest how the patent reasons could be resolved and what would need to be done for this patch to be upstreamed? Thanks. -- 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=14939 winetest(a)luukku.com changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |winetest(a)luukku.com --- Comment #58 from winetest(a)luukku.com --- (In reply to Mike McQuaid from comment #57)
Either way, the attached patch is not acceptable for upstreaming at the moment, if nothing else for the patent reasons mentioned.
Could you suggest how the patent reasons could be resolved and what would need to be done for this patch to be upstreamed? Thanks.
I quess it would be better to send a question to wine-devel mailing 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=14939 --- Comment #59 from Austin English <austinenglish(a)gmail.com> --- (In reply to Mike McQuaid from comment #57)
Either way, the attached patch is not acceptable for upstreaming at the moment, if nothing else for the patent reasons mentioned.
Could you suggest how the patent reasons could be resolved and what would need to be done for this patch to be upstreamed? Thanks.
https://en.wikipedia.org/wiki/S3_Texture_Compression#Patent Though it looks like it's close to running out (Oct 2, 2017). -- 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=14939 --- Comment #60 from Henri Verbeet <hverbeet(a)gmail.com> --- I think I was pretty clear in saying that I think using an external library like libtxc-dxtn-s2tc would be fine. Of course if that can be avoided that would be preferred, which is why I asked for debug output. -- 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=14939 --- Comment #61 from Sam Russell <sam.h.russell(a)gmail.com> --- I didn't save the log output sorry. Can reproduce by running black and white (any version) under wine and getting into the game, the textures start getting hit fairly early on. -- 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=14939 Bruno Jesus <00cpxxx(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |00cpxxx(a)gmail.com --- Comment #62 from Bruno Jesus <00cpxxx(a)gmail.com> --- Created attachment 57121 --> https://bugs.winehq.org/attachment.cgi?id=57121 screenshot of textureless stuff Not sure what log can be useful, attached log is from wine-git. I can reproduce the textureless people/buildings, in the log I'm facing the problem in the screen (don't know if that is relevant). +d3d,+tid (3.2mb unpacks to 300mb): http://alexa.pro.br/~bruno/wine/bw.txt.bz2 standard log: http://alexa.pro.br/~bruno/wine/bwstd.txt -- 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=14939 --- Comment #63 from Bruno Jesus <00cpxxx(a)gmail.com> --- Created attachment 57125 --> https://bugs.winehq.org/attachment.cgi?id=57125 comparison Staging vs Vanilla 2.0 Since there were no screenshots I attached the previous comment screenshot and now a comparison taken from the same angle of a house. Staging is almost perfect, but looks like the windows are not stretched as they should. -- 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=14939 --- Comment #64 from Henri Verbeet <hverbeet(a)gmail.com> --- Created attachment 57187 --> https://bugs.winehq.org/attachment.cgi?id=57187 patch (In reply to Bruno Jesus from comment #62)
Created attachment 57121 [details] screenshot of textureless stuff
Not sure what log can be useful, attached log is from wine-git. I can reproduce the textureless people/buildings, in the log I'm facing the problem in the screen (don't know if that is relevant).
+d3d,+tid (3.2mb unpacks to 300mb): http://alexa.pro.br/~bruno/wine/bw.txt.bz2
standard log: http://alexa.pro.br/~bruno/wine/bwstd.txt
That's helpful, thanks. Could you give the attached patch a try, and create a new log with +ddraw included? -- 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=14939 --- Comment #65 from Bruno Jesus <00cpxxx(a)gmail.com> --- Created attachment 57188 --> https://bugs.winehq.org/attachment.cgi?id=57188 comparison vanilla and patched The patch works for me on Debian, thanks Henri. The screenshot shows clearly every building roof in black, while patched is OK. People also have correct textures. -- 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=14939 --- Comment #66 from Bruno Jesus <00cpxxx(a)gmail.com> --- Created attachment 57189 --> https://bugs.winehq.org/attachment.cgi?id=57189 +d3d,+ddraw with patch 287mb uncompressed. Game loaded to the point the screen is drawn correctly. The trace lines added are present in the log. -- 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=14939 Józef Kucia <joseph.kucia(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|STAGED |RESOLVED Fixed by SHA1| |5a6efe632918fd18758897c3991 | |0d52791d46053 CC| |joseph.kucia(a)gmail.com Resolution|--- |FIXED --- Comment #67 from Józef Kucia <joseph.kucia(a)gmail.com> --- (In reply to Bruno Jesus from comment #65)
The patch works for me on Debian, thanks Henri.
The patch was committed. This should be now fixed with commit 5a6efe632918fd18758897c39910d52791d46053. -- 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=14939 Henri Verbeet <hverbeet(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Fixed by SHA1|5a6efe632918fd18758897c3991 |f576e5c02857a506e63c33e3303 |0d52791d46053 |e2a0a4bc24e1a --- Comment #68 from Henri Verbeet <hverbeet(a)gmail.com> --- (In reply to Józef Kucia from comment #67)
The patch was committed. This should be now fixed with commit 5a6efe632918fd18758897c39910d52791d46053. The patch that's supposed to fix it is f576e5c02857a506e63c33e3303e2a0a4bc24e1a. 5a6efe632918fd18758897c39910d52791d46053 is about handling failure to convert in surface_convert_format() in a more reasonable way, but the application isn't supposed to reach that anymore.
-- 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=14939 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #69 from Alexandre Julliard <julliard(a)winehq.org> --- Closing bugs fixed in 2.2. -- 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=14939 Michael Stefaniuc <mstefani(a)redhat.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|--- |2.0.x -- 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=14939 Michael Stefaniuc <mstefani(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|2.0.x |--- --- Comment #70 from Michael Stefaniuc <mstefani(a)winehq.org> --- Removing the 2.0.x milestone from bugs included in 2.0.1. -- 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=14939 Matteo Bruni <matteo.mystral(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |lminiero(a)gmail.com --- Comment #71 from Matteo Bruni <matteo.mystral(a)gmail.com> --- *** Bug 13610 has been marked as a duplicate of this bug. *** -- 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)
-
wine-bugs＠winehq.org