unofficial mirror of bug-guix@gnu.org 
 help / color / mirror / code / Atom feed
* bug#26062: Spurious "environment variable may be needed" output
@ 2017-03-11 18:32 Taylan Ulrich Bayırlı/Kammer
  2017-03-17  9:51 ` Ludovic Courtès
  0 siblings, 1 reply; 6+ messages in thread
From: Taylan Ulrich Bayırlı/Kammer @ 2017-03-11 18:32 UTC (permalink / raw)
  To: 26062

'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@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@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@T420:~$ 

^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#26062: Spurious "environment variable may be needed" output
  2017-03-11 18:32 bug#26062: Spurious "environment variable may be needed" output Taylan Ulrich Bayırlı/Kammer
@ 2017-03-17  9:51 ` Ludovic Courtès
  2017-03-17 20:34   ` Taylan Ulrich Bayırlı/Kammer
  0 siblings, 1 reply; 6+ messages in thread
From: Ludovic Courtès @ 2017-03-17  9:51 UTC (permalink / raw)
  To: Taylan Ulrich "Bayırlı/Kammer"; +Cc: 26062

Hello Taylan,

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

> taylan@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@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@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’.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#26062: Spurious "environment variable may be needed" output
  2017-03-17  9:51 ` Ludovic Courtès
@ 2017-03-17 20:34   ` Taylan Ulrich Bayırlı/Kammer
  2017-03-18 10:33     ` Ludovic Courtès
  0 siblings, 1 reply; 6+ messages in thread
From: Taylan Ulrich Bayırlı/Kammer @ 2017-03-17 20:34 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: 26062

ludo@gnu.org (Ludovic Courtès) writes:

> Hello Taylan,
>
> taylanbayirli@gmail.com (Taylan Ulrich "Bayırlı/Kammer") skribis:
>
>> taylan@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@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@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

^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#26062: Spurious "environment variable may be needed" output
  2017-03-17 20:34   ` Taylan Ulrich Bayırlı/Kammer
@ 2017-03-18 10:33     ` Ludovic Courtès
  2017-03-18 17:27       ` Taylan Ulrich Bayırlı/Kammer
  0 siblings, 1 reply; 6+ messages in thread
From: Ludovic Courtès @ 2017-03-18 10:33 UTC (permalink / raw)
  To: Taylan Ulrich "Bayırlı/Kammer"; +Cc: 26062

Hi!

taylanbayirli@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’.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#26062: Spurious "environment variable may be needed" output
  2017-03-18 10:33     ` Ludovic Courtès
@ 2017-03-18 17:27       ` Taylan Ulrich Bayırlı/Kammer
  2017-03-19 15:51         ` Ludovic Courtès
  0 siblings, 1 reply; 6+ messages in thread
From: Taylan Ulrich Bayırlı/Kammer @ 2017-03-18 17:27 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: 26062

ludo@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

^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#26062: Spurious "environment variable may be needed" output
  2017-03-18 17:27       ` Taylan Ulrich Bayırlı/Kammer
@ 2017-03-19 15:51         ` Ludovic Courtès
  0 siblings, 0 replies; 6+ messages in thread
From: Ludovic Courtès @ 2017-03-19 15:51 UTC (permalink / raw)
  To: Taylan Ulrich "Bayırlı/Kammer"; +Cc: 26062-done

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

> ludo@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'.

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2017-03-19 15:53 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-03-11 18:32 bug#26062: Spurious "environment variable may be needed" output Taylan Ulrich Bayırlı/Kammer
2017-03-17  9:51 ` Ludovic Courtès
2017-03-17 20:34   ` Taylan Ulrich Bayırlı/Kammer
2017-03-18 10:33     ` Ludovic Courtès
2017-03-18 17:27       ` Taylan Ulrich Bayırlı/Kammer
2017-03-19 15:51         ` Ludovic Courtès

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).