On Fri, Sep 17, 2010 at 7:44 PM, Ralph Versteegen <[email protected]> wrote:
> On 17 September 2010 19:56, David Gowers <[email protected]> wrote:
>>
>> (in case this appears in a double post.. this is the one with the
>> correct updated SConscript)
>
> I was really amazed when I ran scons... and the visual c++ commandline
> compiler spat back that it didn't recognise the gcc command line
> arguments. I'd completely forgotten I had VC++ installed (only used it
> once), and I certainly don't have it in my PATH. Looks like SCons'
> compiler autodetection is too clever.
>
> But no matter what I do, I can not get scons to use gcc instead! I've
> tried an awful lot of suggestions (like setting CC and CXX) I've found
> online and in the man page, and your tools = ['mingw'], but nothing
> will cause SCons to budge.
I can, don't worry.
I already did it for bas files. same thing.

>
> In the end, I can only get past this problem by commenting out the
> lines to build audwrap.
>
> The good news is that once I do so, game.exe and custom.exe build!
> They work, except I notice that you aren't specifying the correct
> FBFLAGS when linking. You need to pass whatever subset of -exx -g -s
> gui -mt, just like when compiling each source file. This also explains
> why you see a warning about inconsistent multithreading options.
ooh. So the inconsistency is from linking with wrong opts.

>
> I tried changing the backends by setting OHRGFX in my shell, and found
sorry, I broke that (somewhat intentionally) when I introduced the
'gfx' + 'music' options. I should support both. for now,
scons gfx=xyz music=abc is the way which works.

> that the script failed to rebuild modules as required (the
> 'semicommon' modules plus game.bas, custom.bas should be rebuilt). So
> I had to clean/delete build/ over and over while testing.
> After cleaning, I see that the order of backends in OHRGFX is ignored.
> The string of backends (passed to verprint) specifies not only which
> gfx backends to include, but also which order to try them in
> (specified by the order of appearance in gfx_choices).
ok.
sorry if I'm not very descriptive, just switched to colemak kb layout
and find my typing inept (+slllllllow).

>
> The following is unnecessary & broken (after fixing _ -> \, I get
> "TypeError: Command() takes exactly 4 arguments (2 given)"), so just
> remove it:
> if 'sdlpp' in used_gfx:
>    sdlpp = Command ('cd gfx_sdl_source;make old;cd ..;cd ..')
> gfx_sdl.dll is provided; I can't remember, but it may not even be
> possible to build it with gcc.
>
> In music_map, you're using libpath instead of libpaths.
>
> allegro.bi needs to be added to standard_bi for gfx_alleg to build.
>
> music_native did not build without a little help, because
> audwrap/audwrap.bi includes "../lumpfile.bi", and build/lumpfile.bi
> was missing. However it was probably missing only because I had just
> deleted build/
>
> Other than the above problems, I've tested that all of gfx_fb,
> gfx_sdl, gfx_directx, gfx_alleg, gfx_sdlpp, music_sdl, music_native,
> music_native2 link and run. Pretty good progress!

Great :)
All your comments have been noted, some are fixed.

>
> Could you please check the build files into svn? Merging my changes
> with the new file you send every half hour is becoming pretty
> annoying; we have version control to handle such things. :)
k.
Working on it.
_______________________________________________
Ohrrpgce mailing list
[email protected]
http://lists.motherhamster.org/listinfo.cgi/ohrrpgce-motherhamster.org

Reply via email to