[Bug 14118] New: wine needs its own dxdiag

http://bugs.winehq.org/show_bug.cgi?id=14118

Summary: wine needs its own dxdiag
           Product: Wine
           Version: CVS/GIT
          Platform: Other
               URL: http://www.gameshadow.com/
        OS/Version: other
            Status: NEW
          Keywords: download
          Severity: enhancement
          Priority: P2
         Component: directx-d3d
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]

Hi, some apps like gameshadow expect dxdiag.exe to be present so wine needs
it's own to satisfy these apps. 
I fixed something together, but somehow i cannot send it to wine-patches, so
i'll just attach it to this bug (i'll try link it from here.)

[Bug 14118] wine needs its own dxdiag

http://bugs.winehq.org/show_bug.cgi?id=14118

--- Comment #1 from Louis Lenders   2008-06-25 02:15:44 ---
Created an attachment (id=14342)
 --> (http://bugs.winehq.org/attachment.cgi?id=14342)
how it looks like

[Bug 14118] wine needs its own dxdiag

http://bugs.winehq.org/show_bug.cgi?id=14118

--- Comment #2 from Louis Lenders   2008-06-25 02:17:16 ---
Created an attachment (id=14343)
 --> (http://bugs.winehq.org/attachment.cgi?id=14343)
the patch

patch contains binary code for bitmap, so to apply again i had to do
git apply dx.patch
git add programs/dxdiag/
tools/make_makefiles
autoconf

maybe there's a simpler way, but i dunno

[Bug 14118] wine needs its own dxdiag

http://bugs.winehq.org/show_bug.cgi?id=14118

Austin English  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |patch

[Bug 14118] wine needs its own dxdiag

http://bugs.winehq.org/show_bug.cgi?id=14118

Austin English  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #14343|0                           |1
        is obsolete|                            |

--- Comment #3 from Austin English   2008-06-27 15:15:27 ---
Created an attachment (id=14400)
 --> (http://bugs.winehq.org/attachment.cgi?id=14400)
dxdiag patch, with configure, etc. updated

Here's the patch with configure/.gitignore/etc. updated. No need to run
autoconf/make_makefiles/etc.

[Bug 14118] wine needs its own dxdiag

http://bugs.winehq.org/show_bug.cgi?id=14118

Louis Lenders  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |ABANDONED

--- Comment #4 from Louis Lenders   2008-07-02 18:04:00 ---
apparently there seems to be no need to have a dxdiag in wine, so i'll abandon
the bug.

[Bug 14118] wine needs its own dxdiag

http://bugs.winehq.org/show_bug.cgi?id=14118

Louis Lenders  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED

--- Comment #5 from Louis Lenders   2008-07-02 18:04:16 ---
closing

[Bug 14118] wine needs its own dxdiag

http://bugs.winehq.org/show_bug.cgi?id=14118

Dan Kegel  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]
             Status|CLOSED                      |REOPENED
         Resolution|ABANDONED                   |

--- Comment #6 from Dan Kegel   2008-12-28 18:58:09 ---
Louis was just discouraged about getting his patch in
(see discussion at 
http://marc.info/?l=wine-devel&m=121483752431741&w=2 
for a list of issues with the patch).

[Bug 14118] wine needs its own dxdiag

http://bugs.winehq.org/show_bug.cgi?id=14118

--- Comment #7 from Dan Kegel   2008-12-28 19:03:59 ---
Created an attachment (id=18284)
 --> (http://bugs.winehq.org/attachment.cgi?id=18284)
Louis' patch updated to apply against current git, plus commandline processing

I updated the patch and added the commandline
parsing needed by GameShadow.  Man, that app needs
a lot of overrides.  I stopped at
  winetricks vbrun60 wsh56 msxml3 mdac27 wininet ie6 
Since I couldn't get the app to run at all, I
didn't implement the options, I just parsed them.

Are there other apps that need native dxdiag.exe ?

[Bug 14118] wine needs its own dxdiag

http://bugs.winehq.org/show_bug.cgi?id=14118

Austin English  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
            Version|CVS/GIT                     |unspecified

--- Comment #8 from Austin English   2009-01-20 02:39:18 ---
Removing deprecated CVS/GIT version tag. Please retest in current git. If the
bug is still present in today's wine, but was not present in some earlier
version of wine, please update version field to earliest known version of wine
that had the bug. Thanks!

[Bug 14118] wine needs its own dxdiag

http://bugs.winehq.org/show_bug.cgi?id=14118

Austin English  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #14400|0                           |1
        is obsolete|                            |
  Attachment #18284|0                           |1
        is obsolete|                            |

--- Comment #9 from Austin English   2009-08-06 21:05:36 ---
Created an attachment (id=22871)
 --> (http://bugs.winehq.org/attachment.cgi?id=22871)
ucla dxdiag

Cleaned up version of the dxdiag made by the UCLA students. Currently doesn't
work...

[Bug 14118] wine needs its own dxdiag

http://bugs.winehq.org/show_bug.cgi?id=14118

Vitaliy Margolen  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #22871|text/plain                  |application/octet-stream
          mime type|                            |
  Attachment #22871|1                           |0
           is patch|                            |

[Bug 14118] wine needs its own dxdiag

http://bugs.winehq.org/show_bug.cgi?id=14118

--- Comment #10 from Austin English   2009-08-24 14:06:52 ---
I sent a small stub, which just returns 0, but makes GameShadow happier:
http://www.winehq.org/pipermail/wine-patches/2009-August/077580.html

while googling, I found that 3DS Max's benchmark tool may use dxdiag /x (what
Dan's patch added):
http://auriterminal.com/Benchmark/ManualSteps.htm

[Bug 14118] wine needs its own dxdiag

http://bugs.winehq.org/show_bug.cgi?id=14118

Deepak Barua  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #11 from Deepak Barua   2009-09-25 21:54:23 ---
(In reply to comment #10)
> I sent a small stub, which just returns 0, but makes GameShadow happier:
> http://www.winehq.org/pipermail/wine-patches/2009-August/077580.html
>
> while googling, I found that 3DS Max's benchmark tool may use dxdiag /x (what
> Dan's patch added):
> http://auriterminal.com/Benchmark/ManualSteps.htm

Hi Austin,
         I would like to help you resolve this bug ... 
could you send me the latest dxdiag sources...? ([e-mail])

Regards
Deepak

[Bug 14118] wine needs its own dxdiag

http://bugs.winehq.org/show_bug.cgi?id=14118

Bruno Jesus  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #12 from Bruno Jesus  2011-10-05 22:20:43 CDT ---
Friendly reminder of 2 years without new posts.

[Bug 14118] wine needs its own dxdiag

http://bugs.winehq.org/show_bug.cgi?id=14118

--- Comment #13 from Austin English  2011-10-05 23:42:44 CDT ---
(In reply to comment #12)
> Friendly reminder of 2 years without new posts.

Semi implemented. It has support for command line processing, and can output
some basic info to a text/xml file.

The GUI is unimplemented.

[Bug 14118] wine needs its own dxdiag

http://bugs.winehq.org/show_bug.cgi?id=14118

--- Comment #14 from Dan Kegel  2011-10-05 23:44:47 CDT ---
Louis, does the current implementation make gameshadow happy?

[Bug 14118] wine needs its own dxdiag

http://bugs.winehq.org/show_bug.cgi?id=14118

Louis Lenders  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|REOPENED                    |RESOLVED
         Resolution|                            |FIXED

--- Comment #15 from Louis Lenders  2011-10-06 13:23:27 CDT ---
(In reply to comment #14)
> Louis, does the current implementation make gameshadow happy?

I don't have the old version anymore, so i downloaded new one from
http://gameshadow.vo.llnwd.net/o1/ltd/patches/download/gameshadow/GameShadow...

The app has some problem with wine's vbscript and msxml3, but it seems to pass
fine across the original missing dxdiag:

.
.
trace:loaddll:load_builtin_dll Loaded L"C:\windows\system32\dxdiag.exe" at
0x68310000: builtin
.
.

Still, it would be nice if we had some nifty GUI, to test ddraw, d3d and
directplay etc like native has, but that would deserve a new bug i guess.

So resolving fixed

[Bug 14118] wine needs its own dxdiag

http://bugs.winehq.org/show_bug.cgi?id=14118

--- Comment #16 from Austin English  2011-10-06 13:59:04 CDT ---
(In reply to comment #15)
> Still, it would be nice if we had some nifty GUI, to test ddraw, d3d and
> directplay etc like native has, but that would deserve a new bug i guess.

Agreed, but that should be a new bug, priority=>enhancement.

[Bug 14118] wine needs its own dxdiag

http://bugs.winehq.org/show_bug.cgi?id=14118

Alexandre Julliard  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED

--- Comment #17 from Alexandre Julliard  2011-10-10 13:13:34 CDT ---
Closing bugs fixed in 1.3.30.