Subject: Re: bug#25908: [PATCH, WIP] Add godot.

From: [email-masked] (Ludovic Courtès)
Date: Tue, 14 Mar 2017 10:28:13 +0100

Hi Chris!

Christopher Allan Webber <[email-masked]> skribis:

> This is still WIP, and unfortunately it doesn't work.  Here's the error
> I got:
>
>                  from ./drivers/pulseaudio/audio_driver_pulseaudio.h:36,
>                  from platform/x11/os_x11.h:46,
>                  from platform/x11/godot_x11.cpp:30:
>   /gnu/store/iwgi9001dmmihrjg4rqhd6pa6788prjw-glibc-2.24/include/bits/local_lim.h:38:26: fatal error: linux/limits.h: No such file or directory
>   #include <linux/limits.h>

This is because, by default, SCons unsets environment variables (!), in
this case C_INCLUDE_PATH.

However:

> +             ;; By design, SCons does not, by default, propagate
> +             ;; environment variables to subprocesses.  See:
> +             ;; <http://comments.gmane.org/gmane.linux.distributions.nixos/4969>
> +             ;; Here, we modify the SConstruct file to arrange for
> +             ;; environment variables to be propagated.
> +             (substitute* "SConstruct"
> +               (("env = Environment\\(\\)")
> +                "env = Environment(ENV=os.environ)")

this should definitely take care of that.

Could it be that the regexp doesn’t match or something?

HTH!

Ludo’.