GNU bug report logs - #26062
Spurious "environment variable may be needed" output

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 26062 in the body.
You can then email your comments to 26062 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

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

From: taylanbayirli <at> gmail.com (Taylan Ulrich Bayırlı/Kammer)
To: bug-guix <at> gnu.org
Subject: Spurious "environment variable may be needed" output
Date: Sat, 11 Mar 2017 19:32:12 +0100

'guix package' commands keep giving me this output:

The following environment variable definitions may be needed:
   export GUILE_LOAD_COMPILED_PATH="/home/taylan/.guix-profile/lib/guile/2.0/site-ccache:/home/taylan/.guix-profile/share/guile/site/2.0${GUILE_LOAD_COMPILED_PATH:+:}$GUILE_LOAD_COMPILED_PATH"

I already have said paths in my $GUILE_LOAD_COMPILED_PATH.  In fact, I
get the output even if I define it to have *solely* those two paths.

taylan <at> T420:~$ guix package -r nano
[... snip ...]
75 packages in profile
The following environment variable definitions may be needed:
   export GUILE_LOAD_COMPILED_PATH="/home/taylan/.guix-profile/lib/guile/2.0/site-ccache:/home/taylan/.guix-profile/share/guile/site/2.0${GUILE_LOAD_COMPILED_PATH:+:}$GUILE_LOAD_COMPILED_PATH"
taylan <at> T420:~$ echo $GUILE_LOAD_COMPILED_PATH
/home/taylan/.guix-profile/lib/guile/2.0/site-ccache:/home/taylan/.guix-profile/share/guile/site/2.0
taylan <at> T420:~$ 

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

From: ludo <at> gnu.org (Ludovic Courtès)
To: taylanbayirli <at> gmail.com (Taylan Ulrich "Bayırlı/Kammer")
Cc: 26062 <at> debbugs.gnu.org
Subject: Re: bug#26062: Spurious "environment variable may be needed" output
Date: Fri, 17 Mar 2017 10:51:14 +0100

Hello Taylan,

taylanbayirli <at> gmail.com (Taylan Ulrich "Bayırlı/Kammer") skribis:

> taylan <at> T420:~$ guix package -r nano
> [... snip ...]
> 75 packages in profile
> The following environment variable definitions may be needed:
>    export GUILE_LOAD_COMPILED_PATH="/home/taylan/.guix-profile/lib/guile/2.0/site-ccache:/home/taylan/.guix-profile/share/guile/site/2.0${GUILE_LOAD_COMPILED_PATH:+:}$GUILE_LOAD_COMPILED_PATH"
> taylan <at> T420:~$ echo $GUILE_LOAD_COMPILED_PATH
> /home/taylan/.guix-profile/lib/guile/2.0/site-ccache:/home/taylan/.guix-profile/share/guile/site/2.0
> taylan <at> T420:~$ 

Weird, I don’t seem to have this problem.

Could you add a few ‘pk’ in ‘search-path-environment-variables’ in (guix
scripts package) to see what’s happening?

Thanks,
Ludo’.

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

From: taylanbayirli <at> gmail.com (Taylan Ulrich Bayırlı/Kammer)
To: ludo <at> gnu.org (Ludovic Courtès)
Cc: 26062 <at> debbugs.gnu.org
Subject: Re: bug#26062: Spurious "environment variable may be needed" output
Date: Fri, 17 Mar 2017 21:34:54 +0100

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

> Hello Taylan,
>
> taylanbayirli <at> gmail.com (Taylan Ulrich "Bayırlı/Kammer") skribis:
>
>> taylan <at> T420:~$ guix package -r nano
>> [... snip ...]
>> 75 packages in profile
>> The following environment variable definitions may be needed:
>>    export GUILE_LOAD_COMPILED_PATH="/home/taylan/.guix-profile/lib/guile/2.0/site-ccache:/home/taylan/.guix-profile/share/guile/site/2.0${GUILE_LOAD_COMPILED_PATH:+:}$GUILE_LOAD_COMPILED_PATH"
>> taylan <at> T420:~$ echo $GUILE_LOAD_COMPILED_PATH
>> /home/taylan/.guix-profile/lib/guile/2.0/site-ccache:/home/taylan/.guix-profile/share/guile/site/2.0
>> taylan <at> T420:~$ 
>
> Weird, I don’t seem to have this problem.
>
> Could you add a few ‘pk’ in ‘search-path-environment-variables’ in (guix
> scripts package) to see what’s happening?

After some poking around, I found the issue; it's the following in the
guix-0.12.0 recipe:

    (wrap-program (string-append out "/bin/guix")
      `("GUILE_LOAD_PATH" ":" prefix (,path))
      `("GUILE_LOAD_COMPILED_PATH" ":" = (,path)))

As you see, it sets GUILE_LOAD_COMPILED_PATH to an absolute value rather
than prefixing it.  So whenever I execute 'guix package ...' from the
shell, the installed wrapper overrides my correct value of
GUILE_LOAD_COMPILED_PATH with something insufficient.

FWIW, it overrides it with the following (split for readability):

"/gnu/store/f4pw2jr2ial53awa34rjj3wb4b4k3nag-guile-json-0.6.0/share/guile/site/2.0:
/gnu/store/y9k9l62cf8hhr5m3np3756swrbxws0dl-guile-ssh-0.10.2/share/guile/site/2.0:
/gnu/store/iyvx2rpyndi3k7dzbl4jnv7k6ddkx88a-gnutls-3.5.8/share/guile/site/2.0"

Should the 'wrap-program' call simply use 'prefix' instead of '='?  I'm
not sure if there's a reason it uses '='.

Taylan

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

From: ludo <at> gnu.org (Ludovic Courtès)
To: taylanbayirli <at> gmail.com (Taylan Ulrich "Bayırlı/Kammer")
Cc: 26062 <at> debbugs.gnu.org
Subject: Re: bug#26062: Spurious "environment variable may be needed" output
Date: Sat, 18 Mar 2017 11:33:33 +0100

Hi!

taylanbayirli <at> gmail.com (Taylan Ulrich "Bayırlı/Kammer") skribis:

> After some poking around, I found the issue; it's the following in the
> guix-0.12.0 recipe:
>
>     (wrap-program (string-append out "/bin/guix")
>       `("GUILE_LOAD_PATH" ":" prefix (,path))
>       `("GUILE_LOAD_COMPILED_PATH" ":" = (,path)))
>
> As you see, it sets GUILE_LOAD_COMPILED_PATH to an absolute value rather
> than prefixing it.  So whenever I execute 'guix package ...' from the
> shell, the installed wrapper overrides my correct value of
> GUILE_LOAD_COMPILED_PATH with something insufficient.

Oh, good catch!

> FWIW, it overrides it with the following (split for readability):
>
> "/gnu/store/f4pw2jr2ial53awa34rjj3wb4b4k3nag-guile-json-0.6.0/share/guile/site/2.0:
> /gnu/store/y9k9l62cf8hhr5m3np3756swrbxws0dl-guile-ssh-0.10.2/share/guile/site/2.0:
> /gnu/store/iyvx2rpyndi3k7dzbl4jnv7k6ddkx88a-gnutls-3.5.8/share/guile/site/2.0"
>
> Should the 'wrap-program' call simply use 'prefix' instead of '='?  I'm
> not sure if there's a reason it uses '='.

Above the ‘wrap-program’ call, there’s this comment:

        ;; Ignore user settings so that a bogus
        ;; GUILE_LOAD_COMPILED_PATH does not prevent use of
        ;; 'guix', notably when it contains entries pointing to
        ;; incompatible .go files as reported at
        ;; <https://lists.gnu.org/archive/html/guix-devel/2016-03/msg01261.html>.

Since 2.0.12, Guile skips invalid .go files, so this problem is gone.
So I think you can go ahead and change '= to 'prefix there.

Thanks!

Ludo’.

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

From: taylanbayirli <at> gmail.com (Taylan Ulrich Bayırlı/Kammer)
To: ludo <at> gnu.org (Ludovic Courtès)
Cc: 26062 <at> debbugs.gnu.org
Subject: Re: bug#26062: Spurious "environment variable may be needed" output
Date: Sat, 18 Mar 2017 18:27:45 +0100

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

> Above the ‘wrap-program’ call, there’s this comment:
>
>         ;; Ignore user settings so that a bogus
>         ;; GUILE_LOAD_COMPILED_PATH does not prevent use of
>         ;; 'guix', notably when it contains entries pointing to
>         ;; incompatible .go files as reported at
>         ;; <https://lists.gnu.org/archive/html/guix-devel/2016-03/msg01261.html>.

Duh, missed that. :-)

> Since 2.0.12, Guile skips invalid .go files, so this problem is gone.
> So I think you can go ahead and change '= to 'prefix there.

Done and pushed.

Thanks for the assistance!

Taylan

Message #22 received at 26062-done <at> debbugs.gnu.org (full text, mbox):

From: ludo <at> gnu.org (Ludovic Courtès)
To: taylanbayirli <at> gmail.com (Taylan Ulrich "Bayırlı/Kammer")
Cc: 26062-done <at> debbugs.gnu.org
Subject: Re: bug#26062: Spurious "environment variable may be needed" output
Date: Sun, 19 Mar 2017 16:51:54 +0100

taylanbayirli <at> gmail.com (Taylan Ulrich "Bayırlı/Kammer") skribis:

> ludo <at> gnu.org (Ludovic Courtès) writes:
>
>> Above the ‘wrap-program’ call, there’s this comment:
>>
>>         ;; Ignore user settings so that a bogus
>>         ;; GUILE_LOAD_COMPILED_PATH does not prevent use of
>>         ;; 'guix', notably when it contains entries pointing to
>>         ;; incompatible .go files as reported at
>>         ;; <https://lists.gnu.org/archive/html/guix-devel/2016-03/msg01261.html>.
>
> Duh, missed that. :-)
>
>> Since 2.0.12, Guile skips invalid .go files, so this problem is gone.
>> So I think you can go ahead and change '= to 'prefix there.
>
> Done and pushed.

Thanks, closing the bug.

Ludo'.

Previous Next

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