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

Bug#747169: RFS: socksjs-twisted/1.2.1-1 [ITP]



On Thu, Sep 11, 2014 at 11:09:34PM +0200, Alexandre Rossi wrote:
> Thanks for your time.
> 
> >>>    - Please, create a VCS to control your debian/ versions. You can
> >>> use github or other. So, add the Vcs-Browser and Vcs-{Git|Svn|Cvs} to
> >>> d/control. You can see an example here[1].
> >>
> >> Done.
> >
> > You need use 'git://' to Vcs-Git, intead of 'https://'.
> 
> That's not what recommended by github and https:// works as a git
> clone url. I'm not against changing this though.

FWIW, IMHO this is correct: I think it's fine for Vcs-* to specify ways
to check out a read-only copy of the latest source and/or its history.
Even for very commonly-used VCS hosting sites such as Gitorious, GitHub
and Alioth, it's quite common for somebody to either not have an account
there yet or simply to not have access to this particular project, not
to mention the fact that one *may* have an account there, but access to
it may depend on, say, an SSH private key that is not currently loaded
and is not accessible from this particular machine.  So, in short, IMHO
it's fine for Vcs-Git to contain an https:// URL; for people with actual
commit access it's a matter of 'git remote set origin git://...' to
change it after a debcheckout, or, of course, 'debcheckout -a' in the
first place for the well-known repositories.

And this discussion taught me something new - I didn't know about
'debcheckout -a' until now, I just found it in a "hey, wouldn't it be
nice if debcheckout could do that... I wonder how hard it would be to
implement it... wait a minute, let me check the manual page...  oh wait,
it does that already!" moment :)

G'luck,
Peter

-- 
Peter Pentchev  roam@ringlet.net roam@FreeBSD.org p.penchev@storpool.com
PGP key:        http://people.FreeBSD.org/~roam/roam.key.asc
Key fingerprint 2EE7 A7A5 17FC 124C F115  C354 651E EFB0 2527 DF13

Attachment: signature.asc
Description: Digital signature


Reply to: