* 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).