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

Previous Next

Full log

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

From: 45mg <45mg.writes <at> gmail.com>
To: Liliana Marie Prikler <liliana.prikler <at> gmail.com>, 45mg
 <45mg.writes <at> gmail.com>, 75981 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>,
 Maxim Cournoyer <maxim.cournoyer <at> gmail.com>,
 Simon Tournier <zimon.toutoune <at> gmail.com>, Tomas Volf <~@wolfsden.cz>,
 Ludovic Courtès <ludo <at> gnu.org>,
 Ricardo Wurmus <rekado <at> elephly.net>, Attila Lendvai <attila <at> lendvai.name>,
 Simon Streit <simon <at> netpanic.org>
Subject: Re: [PATCH v2 2/2] scripts: Add `guix git authenticate --branch`.
Date: Mon, 24 Feb 2025 12:45:23 +0000

Liliana Marie Prikler <liliana.prikler <at> gmail.com> writes:

> Am Sonntag, dem 23.02.2025 um 18:50 +0530 schrieb 45mg:
>>  (define* (record-configuration repository
>> +                               #:optional branch
>>                                 #:key commit signer keyring-
>> reference)
>>    "Record COMMIT, SIGNER, and KEYRING-REFERENCE in the 'config' file
>> of
>> -REPOSITORY."
>> +REPOSITORY, under [guix \"authentication\"].
>> +If BRANCH is given, record them under [guix \"authentication-
>> BRANCH\"]."
>>    (define config
>>      (repository-config repository))
>> +  (define section
>> +    (if branch
>> +        (string-append "guix.authentication-" branch)
>> +        "guix.authentication"))
>>  
>>    ;; Guile-Git < 0.7.0 lacks 'set-config-string'.
>>    (if (module-defined? (resolve-interface '(git)) 'set-config-
>> string)
>>        (begin
>> -        (set-config-string config "guix.authentication.introduction-
>> commit"
>> +        (set-config-string config (string-append section
>> ".introduction-commit")
>>                             commit)
>> -        (set-config-string config "guix.authentication.introduction-
>> signer"
>> +        (set-config-string config (string-append section
>> ".introduction-signer")
>>                             signer)
>> -        (set-config-string config "guix.authentication.keyring"
>> +        (set-config-string config (string-append section ".keyring")
>>                             keyring-reference)
>>          (info (G_ "introduction and keyring recorded \
>>  in repository configuration file~%")))
> I would use guix.authentication[.BRANCH].KEY for consistency.  Yes, you
> can use periods in Git config subsection names as per [1]:
>> Subsection names are case sensitive and can contain any characters
>> except newline and the null byte. 

This would indeed be cleaner. But as you can see in the diffs for
`config-value` and guix.texi, 'guix.authentication-BRANCH' was already
chosen, and I didn't want to break anyone's workflow by changing it.

> Cheers
>
> [1] https://git-scm.com/docs/git-config

Previous Next

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