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

Bug#920314: lintian: vcs-field-has-unexpected-spaces has false positives



On Wed, Jan 23, 2019 at 08:55:34PM -0500, Jeremy Bicha wrote:
> I think all of these are false positives except for rust-num-cpus.
> 
> https://lintian.debian.org/tags/vcs-field-has-unexpected-spaces.html

Really none of them are:

* vcs-git git://git.debian.org/mirrorer/inoticoming.git --branch debian
    -> s/--branch/-b/
* vcs-git -b dpkg https://anonscm.debian.org/git/pkg-request-tracker/libmodule-install-rtx-perl.git
    -> move the `-b dpkg` to the end, policy says it needs to be there
* vcs-hg http://youpibouh.thefreecat.org/loadlin/loadlin.hg/ -b debian
    -> -b is not valid for vcs-hg according to policy
* vcs-git https://salsa.debian.org/rust-team/debcargo-conf.git src/num-cpus
    -> this one probably wanted to wrap "src/nom-cpus" in square
    brakets, as that's a syntax understood by vcswatch to say "inside
    this subdirectory".  Not sure if lintian already knows of that, and
    last I checked policy had yet to integrate that change.


So I think the only reasonable things to do here are only:
 * if you think -b is needed in vcs-hg, open a bug in policy
 * check if the "[subdirectory]" syntax is understood by lintian and
   check the progress status in policy

-- 
regards,
                        Mattia Rizzolo

GPG Key: 66AE 2B4A FCCF 3F52 DA18  4D18 4B04 3FCD B944 4540      .''`.
more about me:  https://mapreri.org                             : :'  :
Launchpad user: https://launchpad.net/~mapreri                  `. `'`
Debian QA page: https://qa.debian.org/developer.php?login=mattia  `-

Attachment: signature.asc
Description: PGP signature


Reply to: