Bug#898077: lintian: False positive in missing-build-dependency-for-dh-addon, python package

2018-05-12 Thread Adrian Bunk
On Sun, May 06, 2018 at 08:42:00PM +, Niels Thykier wrote:
> Chris Lamb:
> > tags 898077 + pending
> > thanks
> > 
> >> Lintian should perhaps check of there is a python package that meets the
> >> dependency requirement? Or allow e.g. "*scour"?
> > 
> > We can't do a wildcard (!)

It would also have been wrong in this case, since the Python 3 module 
package does not (and should not) have a dependency on the dh addon.

In stretch the dh addon is (wrongly) in the Python 2 module package,
that's the only reason why we ended up with the python-scour -> scour
dependency.

> > but we can also check for
> > python-scour. I've done this in Git, pending upload:
> > 
> >   
> > https://salsa.debian.org/lintian/lintian/commit/fc69686ae2d6aff415762812e805af4e5e9ca627
> > 
> >   data/debhelper/dh_addons-manual | 1 +
> >   debian/changelog| 4 
> >   2 files changed, 5 insertions(+)
> > 
> > 
> > Regards,
> > 
> 
> Hi,
> 
> I am not entirely convinced this is a good idea.  As I understand it,
> that dependency is a temporary measure to avoid breakage but the plan is
> to remove the dependency (i.e. packages will be required to build-depend
> directly on "scour" for the add-on).
>...

python-scour is Python 2, so the plan is that python-scour will be 
removed after buster.

Looking at scenarios like stretch-backports, it might be easiest to 
consider python-scour build dependencies as fine here for now and
handle them after buster with the general Python 2 removal.

> Thanks,
> ~Niels

cu
Adrian

-- 

   "Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
   "Only a promise," Lao Er said.
   Pearl S. Buck - Dragon Seed



Bug#898077: lintian: False positive in missing-build-dependency-for-dh-addon, python package

2018-05-06 Thread Niels Thykier
Chris Lamb:
> tags 898077 + pending
> thanks
> 
>> Lintian should perhaps check of there is a python package that meets the
>> dependency requirement? Or allow e.g. "*scour"?
> 
> We can't do a wildcard (!) but we can also check for
> python-scour. I've done this in Git, pending upload:
> 
>   
> https://salsa.debian.org/lintian/lintian/commit/fc69686ae2d6aff415762812e805af4e5e9ca627
> 
>   data/debhelper/dh_addons-manual | 1 +
>   debian/changelog| 4 
>   2 files changed, 5 insertions(+)
> 
> 
> Regards,
> 

Hi,

I am not entirely convinced this is a good idea.  As I understand it,
that dependency is a temporary measure to avoid breakage but the plan is
to remove the dependency (i.e. packages will be required to build-depend
directly on "scour" for the add-on).

I have taken the liberty of CC'ing Martin (maintainer of scour) to
clarify the situation.

Thanks,
~Niels



Bug#898077: lintian: False positive in missing-build-dependency-for-dh-addon, python package

2018-05-06 Thread Chris Lamb
tags 898077 + pending
thanks

> Lintian should perhaps check of there is a python package that meets the
> dependency requirement? Or allow e.g. "*scour"?

We can't do a wildcard (!) but we can also check for
python-scour. I've done this in Git, pending upload:

  
https://salsa.debian.org/lintian/lintian/commit/fc69686ae2d6aff415762812e805af4e5e9ca627

  data/debhelper/dh_addons-manual | 1 +
  debian/changelog| 4 
  2 files changed, 5 insertions(+)


Regards,

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



Bug#898077: lintian: False positive in missing-build-dependency-for-dh-addon, python package

2018-05-06 Thread Ross Gammon
Package: lintian
Version: 2.5.55
Severity: normal

Dear Maintainer,

When building laditools, the missing-build-dependency-for-dh-addon lintian
warning is received because scour is not a build dependency when the scour dh
addon is used in debian/rules.

However, python-scour is a build dependency and pulls in the scour package.

Lintian should perhaps check of there is a python package that meets the
dependency requirement? Or allow e.g. "*scour"?

Regards,

Ross



-- System Information:
Debian Release: stretch/sid
  APT prefers artful-updates
  APT policy: (500, 'artful-updates'), (500, 'artful-security'), (500, 
'artful'), (100, 'artful-backports')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 4.13.0-39-generic (SMP w/4 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8), 
LANGUAGE=en_US:en (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.29.1-4ubuntu1
ii  bzip2 1.0.6-8.1
ii  diffstat  1.61-1build1
ii  dpkg  1.18.24ubuntu1
ii  file  1:5.32-1
ii  gettext   0.19.8.1-4ubuntu1
ii  intltool-debian   0.35.0+20060710.4
ii  libapt-pkg-perl   0.1.33
ii  libarchive-zip-perl   1.59-1
ii  libclass-accessor-perl0.34-1
ii  libclone-perl 0.38-2build2
ii  libdpkg-perl  1.18.24ubuntu1
ii  libemail-valid-perl   1.202-1
ii  libfile-basedir-perl  0.07-1
ii  libipc-run-perl   0.96-1
ii  liblist-moreutils-perl0.416-1build3
ii  libparse-debianchangelog-perl 1.2.0-12
ii  libperl5.26 [libdigest-sha-perl]  5.26.0-8ubuntu1.1
ii  libtext-levenshtein-perl  0.13-1
ii  libtimedate-perl  2.3000-2
ii  liburi-perl   1.72-1
ii  libxml-simple-perl2.24-1
ii  libyaml-libyaml-perl  0.63-2build1
ii  man-db2.7.6.1-2
ii  patchutils0.3.4-2
ii  perl  5.26.0-8ubuntu1.1
ii  t1utils   1.40-2
ii  xz-utils  5.2.2-1.3

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

Versions of packages lintian suggests:
ii  binutils-multiarch 2.29.1-4ubuntu1
ii  dpkg-dev   1.18.24ubuntu1
ii  libhtml-parser-perl3.72-3build1
ii  libtext-template-perl  1.46-1

-- no debconf information