GNU bug report logs - #25908
[PATCH, WIP] Add godot.

Previous Next

Full log

Message #14 received at 25908 <at> debbugs.gnu.org (full text, mbox):

From: ludo <at> gnu.org (Ludovic Courtès)
To: Kei Kebreau <kei <at> openmailbox.org>
Cc: Christopher Allan Webber <cwebber <at> dustycloud.org>, 25908 <at> debbugs.gnu.org
Subject: Re: bug#25908: [PATCH, WIP] Add godot.
Date: Sun, 19 Mar 2017 16:51:30 +0100

Hi Kei,

Kei Kebreau <kei <at> openmailbox.org> skribis:

> Ah, the crucial clue! I replaced the above regexp with the following:
>
> (substitute* "SConstruct"
>   (("env_base = Environment\\(tools=custom_tools\\)")
>    "env_base = Environment(ENV=os.environ)"))
>
> Though I don't know what the meaning of "tools=custom_tools" is, this
> substitution let me complete the build phase without an issue.

In doubt, I’d keep “tools=custom_tools” in addition to
“ENV=os.environ”.  :-)

> Just two other things:
>
> Throughout the build I saw "-Ithirdparty/glew" as a CFLAG. Is this
> necessary for a successful build or can we use our own copy of glew?

Maybe you could (delete-file-recursively "thirdparty") from the snippet
to be sure.

> Is there a check target for this build system? If not we can just leave
> a #:tests? #f with a note in the arguments section.

AFAIK SCons does not standardize anything when it comes to running
tests.  I think ‘scons --help’ lists all the supported targets, so you
could check if there’s a “test” target or something like that.

HTH!

Ludo’.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.