all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* tracking /etc/profile.d/*guix.sh in downstream distributions
@ 2023-05-19 22:10 Vagrant Cascadian
  2023-05-20  3:37 ` Philip McGrath
  0 siblings, 1 reply; 3+ messages in thread
From: Vagrant Cascadian @ 2023-05-19 22:10 UTC (permalink / raw)
  To: guix-devel; +Cc: Philip McGrath

[-- Attachment #1: Type: text/plain, Size: 1094 bytes --]

Philip McGrath recently pointed out that the /etc/profile.d/guix.sh
snippet had not been updating in the Guix packaging for Debian:

  https://bugs.debian.org/1036304

(and as bonus complication, was also renamed to zzz-guix.sh)

The reason I never noticed before is because it is actually a manual
process, with this script embedded in the sys_create_init_profile()
function of etc/guix-install.sh ... and thus easy to miss when updating
packaging.

What would be the impact of separating the /etc/profile.d/zzz-guix.sh
into a separate file upstream? Would guix-install.sh need to be adjusted
to guix-install.sh.in so as to embed the contents of this file?

Alternately, maybe guix-install.sh could be updated to optionally output
or generate the zzz-guix.sh file either by passing commandline arguments
or some other conditional mechanism?

For the short term, I will try to update this manually, but would love
to see a longer term option for future releases!

I am somewhat hoping updating /etc/profile.d/zzz-guix.sh would also fix:

  https://bugs.debian.org/988260


live well,
  vagrant

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 227 bytes --]

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

* Re: tracking /etc/profile.d/*guix.sh in downstream distributions
  2023-05-19 22:10 tracking /etc/profile.d/*guix.sh in downstream distributions Vagrant Cascadian
@ 2023-05-20  3:37 ` Philip McGrath
  2023-05-25  0:45   ` Maxim Cournoyer
  0 siblings, 1 reply; 3+ messages in thread
From: Philip McGrath @ 2023-05-20  3:37 UTC (permalink / raw)
  To: guix-devel, Vagrant Cascadian, ludo

[-- Attachment #1: Type: text/plain, Size: 1709 bytes --]

On Friday, May 19, 2023 6:10:53 PM EDT Vagrant Cascadian wrote:
> Philip McGrath recently pointed out that the /etc/profile.d/guix.sh
> snippet had not been updating in the Guix packaging for Debian:
> 
>   https://bugs.debian.org/1036304
> 
> (and as bonus complication, was also renamed to zzz-guix.sh)
> 
> The reason I never noticed before is because it is actually a manual
> process, with this script embedded in the sys_create_init_profile()
> function of etc/guix-install.sh ... and thus easy to miss when updating
> packaging.
> 
> What would be the impact of separating the /etc/profile.d/zzz-guix.sh
> into a separate file upstream? Would guix-install.sh need to be adjusted
> to guix-install.sh.in so as to embed the contents of this file?
> 

I think this would also make it easier for Guix contributors to edit the file. 
When I sent a patch for it, Ludo’ and I both got to learn about the obscure 
feature of Bash here documents it uses: https://issues.guix.gnu.org/56050#11
Ludo’s informal poll results suggest that others would be confused, too:
https://toot.aquilenet.fr/@civodul/108590952454715930

> Alternately, maybe guix-install.sh could be updated to optionally output
> or generate the zzz-guix.sh file either by passing commandline arguments
> or some other conditional mechanism?
> 

One reason I use Vagrant's packaging for Debian is that otherwise, Guix has no 
mechanism at all for updating the /etc/profile.d/ script, the default 
authorized substitute keys, etc. That's not exactly the same problem, but a 
solution that works well for downstream packagers seems like it make updates 
for other users an easier problem to address.

Philip

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

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

* Re: tracking /etc/profile.d/*guix.sh in downstream distributions
  2023-05-20  3:37 ` Philip McGrath
@ 2023-05-25  0:45   ` Maxim Cournoyer
  0 siblings, 0 replies; 3+ messages in thread
From: Maxim Cournoyer @ 2023-05-25  0:45 UTC (permalink / raw)
  To: Philip McGrath; +Cc: guix-devel, ludo

Hi,

Philip McGrath <philip@philipmcgrath.com> writes:

> On Friday, May 19, 2023 6:10:53 PM EDT Vagrant Cascadian wrote:
>> Philip McGrath recently pointed out that the /etc/profile.d/guix.sh
>> snippet had not been updating in the Guix packaging for Debian:
>> 
>>   https://bugs.debian.org/1036304
>> 
>> (and as bonus complication, was also renamed to zzz-guix.sh)
>> 
>> The reason I never noticed before is because it is actually a manual
>> process, with this script embedded in the sys_create_init_profile()
>> function of etc/guix-install.sh ... and thus easy to miss when updating
>> packaging.
>> 
>> What would be the impact of separating the /etc/profile.d/zzz-guix.sh
>> into a separate file upstream? Would guix-install.sh need to be adjusted
>> to guix-install.sh.in so as to embed the contents of this file?
>> 
>
> I think this would also make it easier for Guix contributors to edit the file. 
> When I sent a patch for it, Ludo’ and I both got to learn about the obscure 
> feature of Bash here documents it uses: https://issues.guix.gnu.org/56050#11
> Ludo’s informal poll results suggest that others would be confused, too:
> https://toot.aquilenet.fr/@civodul/108590952454715930
>
>> Alternately, maybe guix-install.sh could be updated to optionally output
>> or generate the zzz-guix.sh file either by passing commandline arguments
>> or some other conditional mechanism?

One problem I see with extracting that from the script (and relying on
the build system to stitch the final script together), is that we then
can't simply link to the repo as we currently do for
https://guix.gnu.org/install.sh or https://guix-install.sh (they simply
point to
https://git.savannah.gnu.org/cgit/guix.git/plain/etc/guix-install.sh).

-- 
Thanks,
Maxim


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

end of thread, other threads:[~2023-05-25  0:46 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-05-19 22:10 tracking /etc/profile.d/*guix.sh in downstream distributions Vagrant Cascadian
2023-05-20  3:37 ` Philip McGrath
2023-05-25  0:45   ` Maxim Cournoyer

Code repositories for project(s) associated with this external index

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

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.