Ian Jackson <[email protected]> writes:

> Sean Whitton writes ("Re: Include git commit id and git tree id in
> *.changes files when uploading? [and 1 more messages] [and 1 more
> messages]"):
>> Simon Josefsson [12/Jan  7:55pm +01] wrote:
>> > I suppose the PTS watch tracker performs a heavy 'git clone' somehow,
>> > and this is the problem?  I got complaints for >512MB git repositories
>> > (or some similar limit), so this seems likely.
>> >
>> > Fetching all tags or the latest git commit on a particular branch is
>> > inexpensive.  It doesn't have to do a full clone.
>> 
>> Right.  But so long as that's the implementation, the discouragement is
>> apt.
>
> I just experimented.  I took the "bubblewrap" package in sid as a test
> case.  The existing watch file uses an https pattern.  I replaced it
> with this:
>
>   version=4
>   opts="mode=git" https://github.com/bundlewrap/bundlewrap refs/tags/v?(\d\S*)

I found the PTS complaint I remembered:

https://tracker.debian.org/pkg/lib25519
https://qa.debian.org/cgi-bin/vcswatch?package=lib25519

"Error: Repository size 530227200 exceeds 500 MiB, blocking it"

That means it is not 'uscan' that is inefficient, but vcswatch.

Presumably vcswatch can be rewritten in a more efficient way and avoid
download the entire git repository

I agree with your conclusion though.  Let's stop recommending against
mode=git.  For many packages, it is the best option.

/Simon

Attachment: signature.asc
Description: PGP signature

Reply via email to