GNU bug report logs - #75981
[PATCH (WIP) v1 0/4] Add 'guix fork'.

Previous Next

Full log

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

From: Ludovic Courtès <ludo <at> gnu.org>
To: 45mg <45mg.writes <at> gmail.com>
Cc: Nicolas Graves <ngraves <at> ngraves.fr>, Tomas Volf <~@wolfsden.cz>,
 Liliana Marie Prikler <liliana.prikler <at> gmail.com>, 75981 <at> debbugs.gnu.org,
 Ricardo Wurmus <rekado <at> elephly.net>, Attila Lendvai <attila <at> lendvai.name>
Subject: Re: bug#75981: [PATCH (WIP) v1 0/4] Add 'guix fork'.
Date: Sat, 15 Feb 2025 00:51:45 +0100

Hello,

One quick comment…

45mg <45mg.writes <at> gmail.com> skribis:

> +      (info (G_ "Adding key to keyring branch...~%"))
> +      (invoke-git "switch" "keyring")
> +      (invoke "gpg"
> +              "--armor" "--export"
> +              "-o" (string-append directory "/" key-file-name)
> +              signing-key)
> +      (invoke-git "add" "--" key-file-name)
> +      (invoke-git "commit" "-m" "Add key for fork introduction.")
> +
> +      (info (G_ "Setting up fork branch...~%"))
> +      (invoke-git "switch" "--create" "fork" "master")
> +      (when channel-url
> +        (update-channel-url (string-append directory "/.guix-channel")
> +                            channel-url))
> +      (rewrite-authorizations (string-append directory "/.guix-authorizations")
> +                              introduction-name signing-key)
> +      (invoke-git "add" "--"
> +                  (string-append directory "/.guix-authorizations")
> +                  (string-append directory "/.guix-channel"))
> +      (invoke-git "commit"
> +                  (string-append "--gpg-sign=" signing-key)
> +                  "-m"
> +                  (string-append
> +                   "Initial fork commit.\n\n"
> +                   ".guix-authorizations: Allow only " introduction-name "'s key."
> +                   (if channel-url
> +                       "\n.guix-channels: Update channel URL."
> +                       "")))

Apologies for not following the initial discussions that led to this
design.  To make sure I understand, what this does is create
‘.guix-authorizations’ with a single key and then makes that commit the
introduction of the fork, right?

The idea being that one would keep rebasing their fork and ‘guix fork’
would take care of updating the introduction in ‘.git/config’ so you can
keep authenticating it, right?

This looks interesting and a much needed improvement.  There’s quite a
bit to discuss about the implementation; in particular, it would be cool
if we could avoid duplicating most of (guix scripts git authenticate)
and if we could avoid shelling out to various commands, as in the
snippet above, to the extent possible (for instance using Guile-Git).

Thanks for your work!

Ludo’.

Previous Next

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