Processed: Re: lintian: check if Vcs-* fields other than Vcs-Git link to known git repository hosting locations

2018-07-13 Thread Debian Bug Tracking System
Processing commands for cont...@bugs.debian.org:

> tags 903690 + pending
Bug #903690 [lintian] lintian: check if Vcs-* fields other than Vcs-Git link to 
known git repository hosting locations
Added tag(s) pending.
> thanks
Stopping processing here.

Please contact me if you need assistance.
-- 
903690: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=903690
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems



Bug#903690: lintian: check if Vcs-* fields other than Vcs-Git link to known git repository hosting locations

2018-07-13 Thread Chris Lamb
tags 903690 + pending
thanks

Fixed in Git, pending upload:

  
https://salsa.debian.org/lintian/lintian/commit/0ef5fa79db384ea97cf68a39c9bf116353a6189b

  checks/fields.desc  | 10 ++
  checks/fields.pm| 18 ++
  data/fields/{vcs-browsers => vcs-hosters}   |  2 +-
  debian/changelog|  2 ++
  .../debian/debian/control.in| 21 +
  t/tests/fields-vcs-field-mismatch/desc  |  5 +
  t/tests/fields-vcs-field-mismatch/tags  |  3 +++
  7 files changed, 56 insertions(+), 5 deletions(-)


Regards,

-- 
  ,''`.
 : :'  : Chris Lamb
 `. `'`  la...@debian.org / chris-lamb.co.uk
   `-



Bug#889042: lintian: Do not use dot as a separator in build profile names

2018-07-13 Thread Chris Lamb
Dear Javier,

> I can see through your polite answer.
> 
> I do my job. Your position regarding humankind is established already.

Sorry, I don't understand your response - can you elaborate?


Best wishes,

-- 
  ,''`.
 : :'  : Chris Lamb
 `. `'`  la...@debian.org / chris-lamb.co.uk
   `-



Bug#903690: lintian: check if Vcs-* fields other than Vcs-Git link to known git repository hosting locations

2018-07-13 Thread Paul Wise
On Fri, 2018-07-13 at 09:10 +0100, Chris Lamb wrote:

> Does the package get a "vcs-deprecated-in-debian-infra" warning?

Doesn't look like it:

https://lintian.debian.org/full/pkg-games-de...@lists.alioth.debian.org.html#chromium-bsu

-- 
bye,
pabs

https://wiki.debian.org/PaulWise


signature.asc
Description: This is a digitally signed message part


Bug#889042: lintian: Do not use dot as a separator in build profile names

2018-07-13 Thread Chris Lamb
Hi Javier,

> Control: tags -1 patch
> 
> Fix works on 2.5.50.4. This patch is for 2.5.92.

Thanks for the patch! :)  However, I think we would like to see some
resolution to #757760 (or similar) as Mattia outlines in:

  https://bugs.debian.org/889042#25
  
.. before we merge it. :)


Regards,

-- 
  ,''`.
 : :'  : Chris Lamb
 `. `'`  la...@debian.org / chris-lamb.co.uk
   `-



Bug#903690: lintian: check if Vcs-* fields other than Vcs-Git link to known git repository hosting locations

2018-07-13 Thread Chris Lamb
Hi Paul,

> The chromium-bsu source package uses a link to a salsa git repository
> in a Vcs-Svn field

Does the package get a "vcs-deprecated-in-debian-infra" warning? Might
be simplier/easier to use that than create a new tag.


Regards,

-- 
  ,''`.
 : :'  : Chris Lamb
 `. `'`  la...@debian.org / chris-lamb.co.uk
   `-



Bug#903669: lintian -- false positive on using-first-person-in-description

2018-07-13 Thread Chris Lamb
Hi Thorsten,

> >> The description of pyicloud contains the name of a software called "Find
> >> My iPhone".
> >
> > I think this is a case for an override rather than a change in Lintian
> > as it will never be able to reliably know that this is a false positive
> > without aintaining a list of project names like "Find My iPhone."
> 
> aren't there lists for other lintian tags as well? For example where Rouge 
> is the correct spelling that should not be replaced by Rogue, as mentioned 
> in #891794?

So, as it happens, the fix for #891794 was actually to /remove/ that
correction rather than add an exception for that correction.. :) But you
are right that Lintian does have some exception lists.

However, this one just "feels" too esoteric and package-specific.
Please override :)


Best wishes,

-- 
  ,''`.
 : :'  : Chris Lamb
 `. `'`  la...@debian.org / chris-lamb.co.uk
   `-



Bug#903690: lintian: check if Vcs-* fields other than Vcs-Git link to known git repository hosting locations

2018-07-13 Thread Paul Wise
Package: lintian
Version: 2.5.92
Severity: wishlist

The chromium-bsu source package uses a link to a salsa git repository
in a Vcs-Svn field. Consequently debcheckout fails to clone the
repository and DUCK warns about the issue. vcswatch does not warn about
the issue because the last uploader edited the field on vcswatch. There
are two other packages affected by a similar issue but the amount could
increase as people continue to migrate packages from alioth to salsa.

http://duck.debian.net/static/sp/c/chromium-bsu.html
https://qa.debian.org/cgi-bin/vcswatch?package=chromium-bsu

$ apt-cache showsrc chromium-bsu | grep -i git | grep -i svn
Vcs-Svn: https://salsa.debian.org/games-team/chromium-bsu.git

$ debcheckout chromium-bsu
declared svn repository at https://salsa.debian.org/games-team/chromium-bsu.git
svn co https://salsa.debian.org/games-team/chromium-bsu.git chromium-bsu ...
svn: E170013: Unable to connect to a repository at URL 
'https://salsa.debian.org/games-team/chromium-bsu.git'
svn: E160013: '/games-team/chromium-bsu.git' path not found
checkout failed (the command above returned a non-zero exit code)

$ grep -hi ^vcs- /var/lib/apt/lists/*Sources | grep -vi ^vcs-browser | grep -vi 
^Vcs-Git | grep salsa | sort -u
Vcs-Svn: https://salsa.debian.org/dmn/bgoffice-dict-downloader.git
Vcs-Svn: https://salsa.debian.org/games-team/chromium-bsu.git
Vcs-Svn: https://salsa.debian.org/georgesk/qspeakers.git

-- System Information:
Debian Release: buster/sid
  APT prefers testing-debug
  APT policy: (900, 'testing-debug'), (900, 'testing'), (800, 
'unstable-debug'), (800, 'unstable'), (790, 'buildd-unstable'), (700, 
'experimental-debug'), (700, 'experimental'), (690, 'buildd-experimental')
Architecture: amd64 (x86_64)

Kernel: Linux 4.17.0-1-amd64 (SMP w/4 CPU cores)
Locale: LANG=en_AU.utf8, LC_CTYPE=en_AU.utf8 (charmap=UTF-8), 
LANGUAGE=en_AU.utf8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)

Versions of packages lintian depends on:
ii  binutils   2.30.90.20180710-1
ii  bzip2  1.0.6-8.1
ii  diffstat   1.61-1+b1
ii  dpkg   1.19.0.5+b1
ii  file   1:5.33-3
ii  gettext0.19.8.1-6+b1
ii  intltool-debian0.35.0+20060710.4
ii  libapt-pkg-perl0.1.34
ii  libarchive-zip-perl1.60-1
ii  libclass-accessor-perl 0.51-1
ii  libclone-perl  0.39-1
ii  libdpkg-perl   1.19.0.5
ii  libemail-valid-perl1.202-1
ii  libfile-basedir-perl   0.08-1
ii  libipc-run-perl20180523.0-1
ii  liblist-moreutils-perl 0.416-1+b3
ii  libparse-debianchangelog-perl  1.2.0-12
ii  libtext-levenshtein-perl   0.13-1
ii  libtimedate-perl   2.3000-2
ii  liburi-perl1.74-1
ii  libxml-simple-perl 2.25-1
ii  libyaml-libyaml-perl   0.69+repack-1
ii  man-db 2.8.3-2
ii  patchutils 0.3.4-2
ii  perl [libdigest-sha-perl]  5.26.2-6
ii  t1utils1.41-2
ii  xz-utils   5.2.2-1.3

Versions of packages lintian recommends:
ii  libperlio-gzip-perl  0.19-1+b4

Versions of packages lintian suggests:
ii  binutils-multiarch 2.30.90.20180710-1
ii  dpkg-dev   1.19.0.5
ii  libhtml-parser-perl3.72-3+b2
ii  libtext-template-perl  1.53-1

-- no debconf information

-- 
bye,
pabs

https://wiki.debian.org/PaulWise


signature.asc
Description: This is a digitally signed message part


Bug#886036: Improve changelog version parsing

2018-07-13 Thread Chris Lamb
Hi Felix,

> Thank you for following up. I have more patches. I am just not sure if the
> version parsing should really be duplicated in Lintian. Maybe there is a
> benefit to checks and balances, but at least some of the work might well go
> into 'dpkg/scripts/Dpkg/Version.pm', which we could then use in Lintian.

That sounds look a good idea although it might affect our backporting
policy? I'm afraid I can't speak to this, alas. Niels? :)


Regards,

-- 
  ,''`.
 : :'  : Chris Lamb
 `. `'`  la...@debian.org / chris-lamb.co.uk
   `-



Bug#886036: Improve changelog version parsing

2018-07-13 Thread Felix Lechner
Hi Chris,

Thank you for following up. I have more patches. I am just not sure if the
version parsing should really be duplicated in Lintian. Maybe there is a
benefit to checks and balances, but at least some of the work might well go
into 'dpkg/scripts/Dpkg/Version.pm', which we could then use in Lintian.

What do you think, please? Thank you for your guidance!

Best regards,
Felix

On Sun, Apr 15, 2018 at 1:21 AM, Chris Lamb  wrote:
> Chris Lamb wrote:
>
>> Can you update us on the status of this issue? Did we resolve it already
>> or do you have more patches ready to roll soon? :)
>
> Gentle ping on this? I do remember you had some patches queued up,
> I just hope we/I haven't neglected them somewhere...
>
>
> Regards,
>
> --
>   ,''`.
>  : :'  : Chris Lamb
>  `. `'`  la...@debian.org / chris-lamb.co.uk
>`-