Opened 19 years ago

Closed 19 years ago

Last modified 2 years ago

#7947 closed patch

WinDbg/DebugLevel/Cygwin compile

Reported by: SF/mnepomuk Owned by:
Priority: normal Component: Port: Win32
Version: Keywords:
Cc: Game:


This patch consists of

a) a new define USE_WINDBG Debugging under Win9x is nearly impossible: you see only the last 25 lines. The Win32-API function OutputDebugString makes browsing of debug/warning/error messages perfect using WinDbg ( You can even log to file.

b) using -d switch as debug level Allows changing verbosity of debugging output without recompile.

c) make debug output in some resource functions (getResourceAddress etc.) more verbose Print resource type instead of numeric type number where reasonable. Crash safer than #480648.

d) Make it compile under latest cygwin Adding a Makefile.cygwin and some changes in scummsys.h, simon.h

It applies to latest CVS (patch -p2 < xyz.diff in scummvm directory).

Ticket imported from: #546768. Ticket imported from: patches/52.

Attachments (2)

windbg.diff (11.8 KB ) - added by SF/mnepomuk 19 years ago.
WinDbg and debuglevel patch
cygwin.diff (2.5 KB ) - added by SF/mnepomuk 19 years ago.
Cygwin compile + _debugLevel in config

Download all attachments as: .zip

Change History (9)

by SF/mnepomuk, 19 years ago

Attachment: windbg.diff added

WinDbg and debuglevel patch

by SF/mnepomuk, 19 years ago

Attachment: cygwin.diff added

Cygwin compile + _debugLevel in config

comment:1 by SF/mnepomuk, 19 years ago

Thanks for applying! I made a new "cygwin compile" and "debug_level in config file" things against the actual cvs.

comment:2 by Kirben, 19 years ago

What is the point of a Cygwin compile ? there is no need to compile scummvm as a Cygwin program. The makefile you provided has several mistakes, mingw libs (ie winmm) should not be linked to Cygwin programs! and you only need to define COMPRESSED_SOUND_FILE and don't need USE_COMPRESSED_SOUND If you want to compile under Cygwin, use mingw compiler switch (gcc -mno-cygwin) and define WIN32 too.

comment:3 by SF/mnepomuk, 19 years ago

Status: newclosed

comment:4 by SF/mnepomuk, 19 years ago

Why a cygwin compile? I like that highly supported POSIX stuff that gives the feeling of a somehow real *nix environment if you have to develop under Win32 for other reasons (someone told me, it's even possible to compile the X11 version of SCUMMVM under Cygwin).

For the error: only trust the source, never trust the "readme" ;-) I pasted USE_COMPRESSED_SOUND unchecked from "tools/readme.txt".

Help me: Why is the mingw-Guy linking against winsock32 ? Is *this* necessary (until SCUMMVM becomes a multiuser network game) ?

Closing this!

comment:5 by Kirben, 19 years ago

If you just like to develop under unix like shell then use the mingw compiler switch of Cygwin GCC as I mentioned before, or try out msys for mingw. The disadvantages of Cygwin native compile is that it will require the Cygwin dll, be slowed down by Cygwin and won't support midi or mp3. An x11 version under Cygwin and Xfree86 would be very very slow... The reason I'm against linking in winmm and wsock32 (Notice they are usually stored in /lib/mingw/ ?) is because you are compiling a Cygwin native version of scummvm and linking in mingw specific libraries.

comment:6 by Kirben, 19 years ago

And the winsock32 library was been linked in mingw builds due to scummvm compile requiring it at one point (Timidity driver was been compiled in win32 and needed it). I will double check later today and will remove it from Makefile.mingw if it is no longer required.

comment:7 by digitall, 2 years ago

Component: Port: Win32
Note: See TracTickets for help on using tickets.