[Debian-med-packaging] Vcs-Fields: [Was: r7070 - in trunk/packages/bioperl/tags: . 1.6.901-1/debian]

[Charles Plessy]

Le Fri, Jun 17, 2011 at 02:42:30PM +0200, Andreas Tille a écrit :
> On Fri, Jun 17, 2011 at 08:37:37PM +0900, Charles Plessy wrote:
> > 
> > Vcs-Svn, Vcs-Git: old URLs work, but I do not know if it is only a transition.
> > having /svn or /git in is buggy: these are only needed for the svn+ssh or
> > git+ssh URLs, which are not the anonymous URLs expected for these fields.
> 
> However - these "buggy" URLs just drastically simplify the change to '+ssh'
> because you just need to add one single string and not two.  Or would '+ssh'
> work as well without the /svn resp. /git?

Hi Andreas,

debcheckout does it for us:

$ debcheckout -d bioperl
type	svn
url	svn://svn.debian.org/debian-med/trunk/packages/bioperl/trunk/

$ debcheckout -a -d bioperl
type	svn
url	svn+ssh://svn.debian.org/svn/debian-med/trunk/packages/bioperl/trunk/

$ debcheckout -a -d git://git.debian.org/debian-med/foo.git
type	git
url	git+ssh://git.debian.org/git/debian-med/foo.git


> > Vcs-Browser: currently the old URLs are supported through redirection, and again
> > I do not know if this will last or if they will be deprecated some day.  As an
> > exception, the “?rev=0&sc=0” tails makes the redirection fail, and I think it
> > would be good to remove them when there is opportunity.
> 
> So it might be a good idea to let a script remove all those “?rev=0&sc=0”
> strings in trunk/debian/control files, right?

I agree.

Have a nice week-end,

-- 
Charles