Bug#735309: libconfig-model-dpkg-perl: Please support canonical Vcs fields

[Andreas Tille]

Hi Dominique,

On Fri, Jan 17, 2014 at 06:47:12PM +0100, Dominique Dumont wrote:
> ok. I've updated Dpkg model to check/fix most vcs fields using lintian's 
> code as an example.
> 
> Andreas, could build the package and check that these modifications are 
> working for you ? 
> 
> The package is on 
> ssh://git.debian.org/git/pkg-perl/packages/libconfig-model-dpkg-perl.git

This looks good for our svn based packages which I successfully converted
to normalised Vcs fields.  However, there might (or might not?) some
enhancement for some Git based packages.  If you look for instanse at:

  ssh://git.debian.org/git/debian-med/r-cran-stringr.git

$ LC_ALL=C cme fix dpkg-control
Fixing from Dpkg::Control...
Reading package lists... Done
Building dependency tree       
Reading state information... Done
Warning in 'source Vcs-Browser' value 'http://git.debian.org/?p=debian-med/r-cran-stringr.git;a=summary': URL to debian system is not the recommended one (code is: '!defined $_ or ! /debian.org/ or m{^https?://anonscm.debian.org/(viewvc|gitweb|loggerhead)} ;')


>From my perspactive normalisation also means kicking things like

   ;a=summary

in the end.  I have no idea whether this should be discussed or whether
you might want to use your "power" as cme maintainer to deal with this.

Hmmmm, perhaps I'm wrong with my suspicion that ";a=summary" is
responsible for the problem - since after I removed it the problem above
remained.  I guess it is simply the fact that you can leave out gitweb
and apache is expanding this URL to a valid one by injecting gitweb.

I'd recommend that if you find something like

   git\.debian\.org/\?p=debian-med

by

   anonscm.debian.org/gitweb/?p=debian-med

Kind regards and many thanks for your work

      Andreas.

-- 
http://fam-tille.de