[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#41764: `make authenticate` fails to find the keyring branch
From: |
Miguel Ángel Arruga Vivas |
Subject: |
bug#41764: `make authenticate` fails to find the keyring branch |
Date: |
Sat, 24 Oct 2020 14:49:43 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.1 (gnu/linux) |
Hi,
Ludovic Courtès <ludo@gnu.org> writes:
> Hi,
>
> Leo Famulari <leo@famulari.name> skribis:
>
>> I just tried pushing for the first time since installing the new
>> pre-push hook that runs `make authenticate`.
>>
>> This failed with the following error:
>>
>> Git error: cannot locate remote-tracking branch 'keyring'
>>
>> However, `git branch --all` includes "remotes/origin/keyring".
>>
>> After I did `git checkout origin/keyring`, it worked.
>
> Right, since commit 512b9e2da26968ebafdd47f701edd8fc3936d3e8, you have
> to have a local ‘keyring’ branch.
I've hit this too...
>> Let's update the manual section Commit Access with the recommended way
>> to make this branch accessible to `make authenticate`. Maybe it should
>> even do it automatically?
>
> I don’t think it can do it automatically because it cannot guess what
> the remote is called (Tobias reported an issue earlier because
> “origin/keyring” was hard-coded and Tobias didn’t have an “origin”
> remote.)
I'd say this isn't really a solution: hard-coding the origin is bad but
not the branch name? Someone who wants to have a local branch called
keyring for whatever reason will have the exact same problem.
From my POV, "origin/keyring" should be the *default*, as only people
modifying its contents should have a local branch pointing to it, but
the moment you name it hard- instead of normal-code you can clearly say
there's a configuration need. :-)
The attached patch exposes a variable for make called GUIX_GIT_KEYRING
to provide the keyring reference to guix git authenticate, including an
example in the manual, as everything else is already there. WDYT?
Happy hacking!
Miguel
0001-build-Add-GUIX_GIT_KEYRING-variable-for-make-authent.patch
Description: 0001-build-Add-GUIX_GIT_KEYRING-variable-for-make-authent.patch
signature.asc
Description: PGP signature
- bug#41764: `make authenticate` fails to find the keyring branch,
Miguel Ángel Arruga Vivas <=