On 19 September 2010 11:37, David Gowers (kampu) <[email protected]> wrote:
> On Sun, Sep 19, 2010 at 8:49 AM, James Paige <[email protected]> wrote:
>> On Sat, Sep 18, 2010 at 04:05:49PM -0700, [email protected] 
>> wrote:
>>> kampu
>>> 2010-09-18 16:05:49 -0700 (Sat, 18 Sep 2010)
>>> 112
>>> SConstruct based build system. Builds game, custom, and bam2mid currently.
>>>
>>> Still needs more testing on Windows.
>>> ---
>>> A   wip/SConscript
>>> A   wip/SConstruct
>>> A   wip/ohrbuild.py
>>
>> Fails for me on compiling audwrap
>>
>> I don't know if audwrap has ever been compileable on my machine, as I
>> test with music_sdl almost 100% of the time.
>>
> Good point. It is a little overzealous. I've attached a fixed version
> which only adds it as a default target when it's actually going to be
> used. If it works for you, I'll commit.

I see you forgot music_silence in the list of supported backends.

When I try to compile, gcc crashes. gcc of course runs just fine if I
run it with the same options from the commandline. I added -v to the
gcc options and saw that it crashes right before it prints out its
commandline arguments and environmental information. I managed to fix
the problem by going back to ENV = os.environ. After a minute, and
thinking back to the man page, the problem is obvious: your fix
overwrites all the other environmental variables.

Committed those two fixes.

I can't figure out why at the moment, but the gfx commandline option
doesn't cause gver.txt to change. build/gver.txt appears to be a copy
of gver.txt: it has the same modified time. For this reason, building
anything other than the default backends either fails to link or
doesn't let me run them.

I see you added a rule to force gcc to be used for .c files. However,
audwrap is built completely differently, and you create an environment
for its build where you don't set CC and CXX, so it tries to run VC++.
I can't understand why you created another environment for it, so I
left this problem for you to fix.

Switching from extra_env to env , I get:
g++ -c build\audwrap\audwrap.cpp -o build\audwrap\audwrap.o -c -g -O3
build\audwrap\audwrap.cpp:9:21: error: audwrap.h: No such file or directory

OK, now the killer problem: it doesn't rebuild anything when I edit a
file!! Even if I run scons -c, it doesn't rebuild anything afterwards
(but maybe -c is completely unsupported?) I still haven't tested on
Linux. Is rebuilding working for you?

Also, any reason to not place built executables in wip/ instead of
wip/build? It probably more common in other projects, but we already
have all the dlls there, and various distribution scripts which assume
that location.

Oh, and also there is no distver.sh because distrib.sh doesn't need
it; it can easily grab the data itself.
_______________________________________________
Ohrrpgce mailing list
[email protected]
http://lists.motherhamster.org/listinfo.cgi/ohrrpgce-motherhamster.org

Reply via email to