gulikoza wrote:

You need directx headers for directx to be compiled. Check the configure...you should see some lines - checking for ddraw.h...

I have downloaded and extracted the latest DirectX SDK (dxsdk_feb2007.exe). I am a little unsure where to put them in my /mingw tree.

From various comments I found, it seems like I am supposed to copy the header files directly into /mingw/include, overwriting the following files:

1-rw-r--r--    1 MiniMax    100838 Jan  8 15:30 ./d3d9.h
2-rw-r--r--    1 MiniMax     22164 Jan  8 15:30 ./d3d9caps.h
3-rw-r--r--    1 MiniMax     72337 Jan  8 15:30 ./d3d9types.h
4-rw-r--r--    1 MiniMax      2919 Jan 12 19:18 ./dxerr8.h
5-rw-r--r--    1 MiniMax      2968 Jan 12 19:18 ./dxerr9.h

But for the moment, I have installed the header files from the DXSDK in a sub-directory, in /mingw/include/DX, and have defined CPPFLAGS=-I/mingw/include/DX before running configure. This seems to work - somewhat 😒
ddraw.h is found and appears to be okay. But dsound.h and dinput.h throws warnings:

1checking Win32 compiler... yes
2checking ddraw.h usability... yes
3checking ddraw.h presence... yes
4checking for ddraw.h... yes
5checking dsound.h usability... no
6checking dsound.h presence... yes
7configure: WARNING: dsound.h: present but cannot be compiled
8configure: WARNING: dsound.h:     check for missing prerequisite headers?
9configure: WARNING: dsound.h: see the Autoconf documentation
10configure: WARNING: dsound.h:     section "Present But Cannot Be Compiled"
11configure: WARNING: dsound.h: proceeding with the preprocessor's result
12configure: WARNING: dsound.h: in the future, the compiler will take precedence
13configure: WARNING:     ## ------------------------------------------ ##
14configure: WARNING:     ## Report this to the AC_PACKAGE_NAME lists.  ##
15configure: WARNING:     ## ------------------------------------------ ##
16checking for dsound.h... yes
17checking dinput.h usability... no
18checking dinput.h presence... yes
19configure: WARNING: dinput.h: present but cannot be compiled
20configure: WARNING: dinput.h:     check for missing prerequisite headers?
21configure: WARNING: dinput.h: see the Autoconf documentation
22configure: WARNING: dinput.h:     section "Present But Cannot Be Compiled"
23configure: WARNING: dinput.h: proceeding with the preprocessor's result
24configure: WARNING: dinput.h: in the future, the compiler will take precedence
25configure: WARNING:     ## ------------------------------------------ ##
26configure: WARNING:     ## Report this to the AC_PACKAGE_NAME lists.  ##
27configure: WARNING:     ## ------------------------------------------ ##
28checking for dinput.h... yes
29checking for yasm... no
30checking for nasm... no
31configure: creating ./config.status
32config.status: creating Makefile

Looking inside the confg.log for more detais I see:

1configure:20: checking ddraw.h usability
2configure:32: gcc -c -O3 -s -mms-bitfields -march=i686 -Iinclude -I/home/MiniMax/src/SDL-1.2.11-src/include -D_GNU_SOURCE=1 -I/mingw/include/DX -Iinclude -I/home/MiniMax/src/SDL-1.2.11-src/include -D_GNU_SOURCE=1 conftest.c >&5
3configure:38: $? = 0
4configure:42: test -z 
5                         || test ! -s conftest.err
6configure:45: $? = 0
7configure:48: test -s conftest.o
8configure:51: $? = 0
9configure:61: result: yes
10configure:65: checking ddraw.h presence
11configure:75: gcc -E -I/mingw/include/DX -Iinclude -I/home/MiniMax/src/SDL-1.2.11-src/include -D_GNU_SOURCE=1 conftest.c
12configure:81: $? = 0
13configure:101: result: yes
14configure:136: checking for ddraw.h
15configure:143: result: yes
16configure:162: checking dsound.h usability
17configure:174: gcc -c -O3 -s -mms-bitfields -march=i686 -Iinclude -I/home/MiniMax/src/SDL-1.2.11-src/include -D_GNU_SOURCE=1 -I/mingw/include/DX -Iinclude -I/home/MiniMax/src/SDL-1.2.11-src/include -D_GNU_SOURCE=1 conftest.c >&5
18In file included from conftest.c:124:
19C:/Spool/dxsdk_feb2007/Include/dsound.h:1899: error: redefinition of typedef 'LPDIRECTSOUNDFULLDUPLEX'
20C:/Spool/dxsdk_feb2007/Include/dsound.h:175: error: previous declaration of 'LPDIRECTSOUNDFULLDUPLEX' was here

( /mingw/include/DX is mounted from C:\Spool\dxsdk_feb2007/Include)

Checking dsound.h shows that configure (gcc) is correct:

1$ nl dsound.h | egrep '175      typedef|1899    typedef'
2   175  typedef struct IDirectSoundFullDuplex       *LPDIRECTSOUNDFULLDUPLEX;
3  1899  typedef struct IDirectSoundFullDuplex *LPDIRECTSOUNDFULLDUPLEX;

Any ideas why this is happening? Is it simply a case of bad coding by Microsoft, defining the same typedef twice (even if it is an identical definition)?

MiniMax wrote:

This means, that to build and run DOSBox, I need […]
Show full quote

This means, that to build and run DOSBox, I need

1. a modified version of SDL (which explains the path-names listed in Qbix's diffs (SDL-1.2.11rel and SDL-1.2.11 - one for the official release, and one for an DOSBox-adapted version)
2. to make sure that I don't accidential install the modified SDL.dll in my ordinary build envinronment, e.g. /mingw/bin.

You can be as strict as you want to be. (but it's not really needed)
in fact the sdl.dll is compiled on a different version of mingw then the release of dosbox. (I maked install to a different directory and edit the sdl-config)
the other support libs are even compiled on a different computer using an ancient version of mingw. (sdl_sound and related libs)

You can mix the compiled versions of SDL.dll without problems unless SDL changes their exported api. (which happened between the 0.63/65 sdl.dll version and the 0.70 version)
You can even compile dosbox and compile a new sdl.dll afterwards (for example having those patches in.) It depends a bit on what you change, but for small things it doesn't matter.

There are 2 directories mentioned in my patch as it was the easiest way of making the diff for you..

I don't deny that I am a control freak when it comes to configuration management. I spend so much time at work, trying to maintain a build-environment that will allow us to re-create our applications and databases from scratch. I want consistency, repeatability, script-ability, and documentation of the process. Keeping programs and files in non-standard location makes life a little complicated, but when it is all put into build-scripts it also serves a documentation for what is needed for the build, where it is located, how it is created, etc, etc.

If I can do this for DOSBox, then the next person that wants to have a look, or contribute code, will have a much easier task.

Qbix wrote:

There are 2 directories mentioned in my patch as it was the easiest way of making the diff for you..

The diff itself was fine, no problem. I just wanted some rationale for the changes.

As of now, I am stuck with a problem compiling SDL_dx5events:

1/bin/sh ./libtool --mode=compile gcc -O3 -s -mms-bitfields -march=i686 -L/mingw/lib/DX -I/mingw/include/DX -Iinclude -I/home/MiniMax/src/SDL-1.2.11-src/include -D_GNU_SOURCE=1 -c /home/MiniMax/src/SDL-1.2.11-src/src/video/windx5/SDL_dx5events.c  -o build/SDL_dx5events.lo
2 gcc -O3 -s -mms-bitfields -march=i686 -L/mingw/lib/DX -I/mingw/include/DX -Iinclude -I/home/MiniMax/src/SDL-1.2.11-src/include -D_GNU_SOURCE=1 -c /home/MiniMax/src/SDL-1.2.11-src/src/video/windx5/SDL_dx5events.c  -DDLL_EXPORT -DPIC -o build/.libs/SDL_dx5events.o
3In file included from C:/MSYS/1.0/home/MiniMax/src/SDL-1.2.11-src/src/video/windx5/directx.h:82,
4                 from C:/MSYS/1.0/home/MiniMax/src/SDL-1.2.11-src/src/video/windx5/SDL_dx5events.c:26:
5C:/Spool/dxsdk_feb2007/Include/dinput.h:1298: error: redefinition of typedef 'DIDEVICEINSTANCE'
6C:/Spool/dxsdk_feb2007/Include/dinput.h:1288: error: previous declaration of 'DIDEVICEINSTANCE' was here
7C:/Spool/dxsdk_feb2007/Include/dinput.h:1301: error: redefinition of typedef 'LPCDIDEVICEINSTANCE'
8C:/Spool/dxsdk_feb2007/Include/dinput.h:1299: error: previous declaration of 'LPCDIDEVICEINSTANCE' was here
9C:/MSYS/1.0/home/MiniMax/src/SDL-1.2.11-src/src/video/windx5/SDL_dx5events.c: In function `DX5_CreateWindow':
10C:/MSYS/1.0/home/MiniMax/src/SDL-1.2.11-src/src/video/windx5/SDL_dx5events.c:868: warning: cast to pointer from integer of different size
11make: *** [build/SDL_dx5events.lo] Error 1

Okay - I discarded the February 2007 DX-SDK from Microsoft, and installed these headers and libs from the SDL-site:

http://www.libsdl.org/extras/win32/common/dir … tx-devel.tar.gz

Now SDL compiles, and I can invoke DOSBox with SDL_VIDEODRIVER=directx.

Microsoft has some typedefs repeated in their headers. I assume their compiler happily accepts that so they haven't noticed, but gcc doesn't. This is a patch I apply to dx headers....
That SDL_dx5video change is needed for output=ddraw to work.

just use the SDL file included in http://prdownloads.sourceforge.net/dosbox/DOS … er.exe?download

gulikoza wrote:

Microsoft has some typedefs repeated in their headers. I assume their compiler happily accepts that so they haven't noticed, but gcc doesn't. This is a patch I apply to dx headers....

Thanks gulikoza - just what I needed. Together with the tip to use reimp (for all the Microsoft DX libraries?) that I found on your web-page, I believe there is a chance to get the MS DX SDK to work. But it will have to wait a bit - first I want to streamline the process using the files from libsdl.org, then I want to add SDL_net to my build, and whatever is need to make something that is equivalent to the official DOSBox distribution.

gulikoza wrote:

That SDL_dx5video change is needed for output=ddraw to work.

You are refering to this patch?

1-#if 0 /* FIXME: enable this when SDL consistently reports lost surfaces */
2+#if 1 /* FIXME: enable this when SDL consistently reports lost surfaces */
3 		if ( (flags & SDL_HWSURFACE) == SDL_HWSURFACE ) {
4 			video->flags |= SDL_HWSURFACE;
5 		} else {

So specifying output=ddraw in the DOSBox configuration will expose this problem with lost surfaces?

Thanks - that is the kind of explanation I was looking for.

franpa wrote:

just use the SDL file included in http://prdownloads.sourceforge.net/dosbox/DOS … er.exe?download

My ambitions are a little higher. I might break my neck while climbing high, but I can enjoy the view while I climb.

Reimp converts Visual Studio libs to gcc libs. It can be used for anything...
You actually should not need any DirectX libraries, unless you want to compile my Direct3D patch with shader support.

Yes, that was the patch I was referring to. Without it, dosbox will fail to create ddraw output ("Failed to create ddraw surface, back to normal surface.") because SDL_HWSURFACE flag is not set.

SDL_VIDEODRIVER=windib
is needed for NT4.0 sp 6 as well.

(email conversation with Stephan)