Ah, I see... Sorry, I'm not very familiar with linking stuff on Windows (it
was really not fun before we had scons), anyway here is the patch to
SConscript...
It turns sets flags_msvcdebug to 0 by default...
It builds and runs dep free. So I'll check it in. :D
Thanks again
Index: SConscript
===================================================================
--- SConscript (revision 2269)
+++ SConscript (working copy)
@@ -700,13 +700,15 @@
else:
print "Vinyl Control... disabled"
-flags_msvcdebug = getFlags(env, 'msvcdebug', 1)
+flags_msvcdebug = getFlags(env, 'msvcdebug', 0)
if int(flags_msvcdebug) and platform == 'win32':
+ env.Append(CCFLAGS = '/MDd')
env.Append(LINKFLAGS = '/DEBUG')
env.Append(CXXFLAGS = '/ZI')
print "MSVC Debugging... enabled"
build_flags += 'msvcdebug '
else:
+ env.Append(CCFLAGS = '/MD')
print "MSVC Debugging... disabled"
flags_script = getFlags(env, 'script', 0)
On Sun, Aug 24, 2008 at 6:26 PM, Michael Pujos <[EMAIL PROTECTED]>wrote:
> Garth Dahlstrom a écrit :
>
>> The thing if I check in the one that depends on the debug assembly, nobody
>> accept you will be able to do a windows release build. :P
>> (since the rest of us sorry suckers don't have the patched source, project
>> files, etc for sndfile... )
>>
>> Could you please just compile it so it doesn't require the debug stuff, if
>> it's not too much trouble?
>>
>>
>> if platform == 'win32':
>> if int(flags_msvcdebug):
>> env.Append(CCFLAGS = '/MDd')
>> else
>> env.Append(CCFLAGS = '/MD')
>>
>>
>>
>>
> The above snippet links mixxx with the MSVCRT debug library if msvcdebug is
> defined (/MDd) or the release MSVCRT otherwise (/MD).
> The compiled libsndfile is a release compile and works with both debug or
> release compile of mixxx.
> Previously the MSVCRT code was into the libsndfile.lib import library,
> which is a hack that must have add some purpose (?), but I think is not
> necessary and got me confused for a while :).
> It's a good idea to link to the debug version of MSVCRT when compiling
> mixxx in debug because it can catch some things like memory corruption.
> But if you wish you can just add
>
> if platform == 'win32':
> env.Append(CCFLAGS = '/MD')
>
>
> it will always link mixxx to the release version of MSVRT.
>
>
>
>
>
--
__
--- == __/ t.O ==--
http://stacktrace.org/
-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
Mixxx-devel mailing list
Mixxx-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mixxx-devel
