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

Previous Next

Full log

View this message in rfc822 format

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: bug#25908: [PATCH, WIP] Add godot.
Date: Sat, 25 Mar 2017 18:47:46 +0100

Hi Kei,

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

> ludo <at> gnu.org (Ludovic Courtès) writes:

[...]

>>> 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.
>>
>
> It seems like these libraries will be quite difficult to unbundle. The
> build system has targets that depend on files specifically found in the
> "thirdparty" directory.

As a first step, you could remove (in the snippet) those that can easily
be avoided, and document work that remains to be done in a comment.
Though yeah, we’ll have to find out how to modify ‘SConscript’ & co. to
remove the dependencies on bundled libs.

>>> 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.
>>
>
> There doesn't seem to be tests in the Godot's build system.

Simpler!

Ludo’.

Previous Next

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