remove Win32 dependancy on directX headers from DXSDK

[David Bussenschutt <davidbuzz@…>]

Opend this Trac so as we don't forget to look into this as some stage:

Buzz's initial suggestion:

• regarding the DXSDK header files- my suggestion is that we just wack

the 3 needed header files,a README.txtexplaining whatit isand a LICENCE file into a zip file , and put it on a server somewhere. Given that .h files document the API in the rest of the software/sdk (which we don't need), I'd think that they fall under the "documentation" section of the EULA which says that: "4. DOCUMENTATION. Any person that has valid access to your computer or internal network may copy and use the documentation for your internal, reference purposes.". Which translated says to methat "any valid mythtv developer may use it for mythtv development purposes by accessing it from a mythtv related server (like trac or svn)". I am not a lawyer.

Andrei's response: We only need a handful of function declarations from dsound.h, I think the easiest thing to do might be to place these declarations in compat.h and get rid of this dependency altogether.

...

Another option is not to compile audiooutputdx at all - in its current state it's not usable anyway (I wrote and tested audiooutputwin and I am satisfied with the way it works).

[David Bussenschutt (aka buzz) <davidbuzz@…>]

in the Win32-packager.pl script, I have used an alternative source for the header files, which we could use in the wiki doco regarding this.... but the proper solution is still what Andrei suggested, so the ticket should stay open for now.

[Buzz <davidbuzz@…>]

quick update: the win32-packager.pl actually does (and has for quite some time) what I initially suggested (ie the few needed .h files are just stored in a .zip file on a website, without the rest of the DXSDK, which isn't needed), but it'd still be nice to do andre's suggestion (using compat.h, and removing the need for these headers altogether.

[stuartm]

Can this ticket be closed yet?

[anonymous]

So long as we have directx code in mythtv ( despite being deprecated) , and we still have the following code block in the build script, then this ticket is here to remind us to come up with a better solution( ie get rid of the directx code from mythtv that uses dsound.h) : 643 # ( save bandwidth compare to the above full SDK where they came from: 644 [ archive => $sources.'DX9SDK_dsound_Include_subset.zip', 645 'fetch' => ' http://davidbuzz.googlepages.com/'. 646 'DX9SDK_dsound_Include_subset.zip', 647 comment => 'We download just the required Include files for DX9' ], 648 [ dir => $sources.'DX9SDK_dsound_Include_subset', 649 extract => $sources.'DX9SDK_dsound_Include_subset.zip' ], 650 [ filesame => [$mingw.'include/dsound.h',$sources. 651 "DX9SDK_dsound_Include_subset/dsound.h"], 652 copy => [=>] ], 653 [ filesame => [$mingw.'include/dinput.h',$sources. 654 "DX9SDK_dsound_Include_subset/dinput.h"], 655 copy => [=>] ], 656 [ filesame => [$mingw.'include/ddraw.h', $sources. 657 "DX9SDK_dsound_Include_subset/ddraw.h"], 658 copy => [=>] ], 659 [ filesame => [$mingw.'include/dsetup.h',$sources. 660 "DX9SDK_dsound_Include_subset/dsetup.h"], 661 copy => [=>] ], 662 ;

[Jeff Lu <jll544@…>]

Suggested approach: check for dsound.h in configure

[danielk]

Mark K & Nigel have Windows computers, I'm assigning windows tickets semi-randomly to one of them.

[robertm]

Is anyone using DX sound output that has a critical reason for needing it? It seems that standardizing on the better support audiooutputwin ("Windows:") method would be both more intuitive and better supported.

[stuartm]

Milestone 0.25 deleted