guix-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: 09/09: lint: Append "/info/refs" to git-reference-url.


From: Sergey Trofimov
Subject: Re: 09/09: lint: Append "/info/refs" to git-reference-url.
Date: Thu, 30 Mar 2023 08:13:00 +0200
User-agent: mu4e 1.10.0; emacs 29.0.60


Maxim Cournoyer <maxim.cournoyer@gmail.com> writes:

Hello,

guix-commits@gnu.org writes:

    lint: Append "/info/refs" to git-reference-url.

After this fix, I'm now seeing warnings from Github, e.g.:

gnu/packages/rails.scm:675:2: ruby-activejob@7.0.4.3: scheduled Software Heritage archival gnu/packages/rails.scm:282:2: ruby-activemodel@7.0.4.3: URI https://github.com/rails/rails/info/refs not reachable: 403 ("Forbidden")


It seems that the `service=git-upload-pack` is important as it switches the protocol to "smart-http".

--8<---------------cut here---------------start------------->8---
~/devel/ext λ curl 'https://github.com/sarg/dotfiles/info/refs?service=git-upload-pack'
001e# service=git-upload-pack
Warning: Binary output can mess up your terminal. Use "--output -" to tell Warning: curl to output it to your terminal anyway, or consider "--output
Warning: <FILE>" to save to a file.
0000

~/devel/ext λ curl 'https://github.com/sarg/dotfiles/info/refs'
Please upgrade your git client.
GitHub.com no longer supports git over dumb-http: https://github.com/blog/809-git-dumb-http-transport-to-be-turned-off-in-90-days
--8<---------------cut here---------------end--------------->8---

Maybe the linter should just use `git ls-remote` instead of guessing the url?
--8<---------------cut here---------------start------------->8---
~/devel/ext λ git ls-remote https://github.com/sarg/dotfiles
7707f77e8d9c11b5435c5901eca45be1a72d2054        HEAD
7707f77e8d9c11b5435c5901eca45be1a72d2054        refs/heads/master
--8<---------------cut here---------------end--------------->8---



reply via email to

[Prev in Thread] Current Thread [Next in Thread]