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

[Maxim Cournoyer]

Hi Sergey,

Sergey Trofimov <sarg@sarg.org.ru> writes:

> 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".
>
> ~/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
>
>
> Maybe the linter should just use `git ls-remote` instead of 
> guessing the url?
>
> ~/devel/ext λ git ls-remote https://github.com/sarg/dotfiles
> 7707f77e8d9c11b5435c5901eca45be1a72d2054      HEAD
> 7707f77e8d9c11b5435c5901eca45be1a72d2054      refs/heads/master

That sounds like a better way to me!

-- 
Thanks,
Maxim