Opened 17 years ago

Closed 17 years ago

Last modified 8 months ago

#7947 closed patch

WinDbg/DebugLevel/Cygwin compile

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

Description

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
(www.sysinternals.com/ntw2k/freeware/debugview.shtml).
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 17 years ago.
WinDbg and debuglevel patch
cygwin.diff (2.5 KB) - added by SF/mnepomuk 17 years ago.
Cygwin compile + _debugLevel in config

Download all attachments as: .zip

Change History (9)

Changed 17 years ago by SF/mnepomuk

Attachment: windbg.diff added

WinDbg and debuglevel patch

Changed 17 years ago by SF/mnepomuk

Attachment: cygwin.diff added

Cygwin compile + _debugLevel in config

comment:1 Changed 17 years ago by SF/mnepomuk

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

comment:2 Changed 17 years ago by Kirben

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 Changed 17 years ago by SF/mnepomuk

Status: newclosed

comment:4 Changed 17 years ago by SF/mnepomuk

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 Changed 17 years ago by Kirben

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 Changed 17 years ago by Kirben

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 Changed 8 months ago by digitall

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