[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#49034] [PATCH] profiles: Add 'load-profile'.
From: |
Ludovic Courtès |
Subject: |
[bug#49034] [PATCH] profiles: Add 'load-profile'. |
Date: |
Fri, 18 Jun 2021 12:45:48 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.2 (gnu/linux) |
Hi,
Maxime Devos <maximedevos@telenet.be> skribis:
> Ludovic Courtès schreef op di 15-06-2021 om 10:13 [+0200]:
>> +(define %precious-variables
>> + ;; Environment variables in the default 'load-profile' white list.
>> + '("HOME" "USER" "LOGNAME" "DISPLAY" "TERM" "TZ" "PAGER"))
> [...]
>> -;; Protect some env vars from purification. Borrowed from nix-shell.
>> -(define %precious-variables
>> - '("HOME" "USER" "LOGNAME" "DISPLAY" "TERM" "TZ" "PAGER"))
>
> This is really a separate issue, but I don't understand why "XAUTHORITY"
> isn't in the list. Without preserving XAUTHORITY, the following won't work
> (at least on my desktop environment):
>
> guix environment --pure --ad-hoc atril -- atril stuff.pdf
> # No protocol specified
> # Cannot parse arguments: Cannot open display:
>
> But the following does start atril:
> guix environment --preserve=XAUTHORITY --pure --ad-hoc atril -- atril
> stuff.pdf
> # A window titled ‘Atril Document Viewer’
>
> Could "XAUTHORITY" be added to the list?
> Preserving "DISPLAY" but not "XAUTHORITY" seems rather useless to me,
> at least on my desktop environment.
Agreed, I’ll do that.
Thanks,
Ludo’.