guix-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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


From: Ludovic Courtès
Subject: bug#25908: [PATCH, WIP] Add godot.
Date: Tue, 14 Mar 2017 10:28:13 +0100
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux)

Hi Chris!

Christopher Allan Webber <address@hidden> 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’.





reply via email to

[Prev in Thread] Current Thread [Next in Thread]