Bug#931847: Boggus package-supports-alternative-init-but-no-init.d-script test?

2019-07-18 Thread Chris Lamb
Michael Biebl wrote:

> Fwiw, I've only seen 100s of false positives so far. Claiming it's
> certainty to be "certain" is overstating it "a little".

Due to prioritisation of effort and energy the Lintian maintainers put
very little time and effort into curating these "certainty" levels.

I would thus not read anything into them whatsoever and certainly not
be too irked if they do not match reality.


Regards,

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



Bug#931847: Boggus package-supports-alternative-init-but-no-init.d-script test?

2019-07-18 Thread Michael Biebl
On 18.07.19 21:43, Michael Biebl wrote:
> Hm, why the moreinfo tag?
> This lintian check is clearly way too broad to be useful as-is.
> At its current state, please demote it to pedantic (or reverting it
> completely) until it actually is useful.

Fwiw, I've only seen 100s of false positives so far. Claiming it's
certainty to be "certain" is overstating it "a little".


-- 
Why is it that all of the instruments seeking intelligent life in the
universe are pointed away from Earth?



signature.asc
Description: OpenPGP digital signature


Bug#931847: Boggus package-supports-alternative-init-but-no-init.d-script test?

2019-07-18 Thread Michael Biebl
On Thu, 11 Jul 2019 11:39:28 -0300 "Chris Lamb"  wrote:
> block 931847 by 911165
> tags 931847 + moreinfo
> thanks
> 
> Ansgar Burchardt wrote:
> 
> > > My understanding of the policy is that, if a package supports an
> > > alternative init (other than systemd) it must also support sysvinit.
> > > 
> > > Also note that if the check is actually correct, this will create false
> > > positive for all the systemd .service files not started at boot
> > > (scheduled jobs, dbus activated,...).
> > 
> > The current policy requirement is that everything would need to provide
> > a sysvinit script, see https://bugs.debian.org/911165
> > 
> > Sadly the process to change this is stuck.
> 
> (I'm just applying some routine bug triage here; not a comment on the
> bug's merits.)


Hm, why the moreinfo tag?
This lintian check is clearly way too broad to be useful as-is.
At its current state, please demote it to pedantic (or reverting it
completely) until it actually is useful.

Thanks,
Michael

-- 
Why is it that all of the instruments seeking intelligent life in the
universe are pointed away from Earth?



signature.asc
Description: OpenPGP digital signature


Bug#931847: Boggus package-supports-alternative-init-but-no-init.d-script test?

2019-07-15 Thread Laurent Bigonville
On Thu, 11 Jul 2019 13:26:34 +0200 Laurent Bigonville  
wrote:

[...]
>
> Also note that if the check is actually correct, this will create false
> positive for all the systemd .service files not started at boot
> (scheduled jobs, dbus activated,...).
>

It's also creating false positives with symlinked .services:

For example:

E: policycoreutils: 
package-supports-alternative-init-but-no-init.d-script 
lib/systemd/system/basic.target.wants/selinux-autorelabel-mark.service
E: policycoreutils: 
package-supports-alternative-init-but-no-init.d-script 
lib/systemd/system/selinux-autorelabel-mark.service


/lib/systemd/system/basic.target.wants/selinux-autorelabel-mark.service 
is a symlink to /lib/systemd/system/selinux-autorelabel-mark.service


Not sure whether only files in /lib/systemd/system/ be looked at or 
should symlinks be completely ignored(?)




Bug#931847: Boggus package-supports-alternative-init-but-no-init.d-script test?

2019-07-11 Thread Chris Lamb
block 931847 by 911165
tags 931847 + moreinfo
thanks

Ansgar Burchardt wrote:

> > My understanding of the policy is that, if a package supports an
> > alternative init (other than systemd) it must also support sysvinit.
> > 
> > Also note that if the check is actually correct, this will create false
> > positive for all the systemd .service files not started at boot
> > (scheduled jobs, dbus activated,...).
> 
> The current policy requirement is that everything would need to provide
> a sysvinit script, see https://bugs.debian.org/911165
> 
> Sadly the process to change this is stuck.

(I'm just applying some routine bug triage here; not a comment on the
bug's merits.)


Regards,

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



Bug#931847: Boggus package-supports-alternative-init-but-no-init.d-script test?

2019-07-11 Thread Ansgar Burchardt
On Thu, 11 Jul 2019 13:26:34 +0200 Laurent Bigonville wrote:
> My understanding of the policy is that, if a package supports an
> alternative init (other than systemd) it must also support sysvinit.
> 
> Also note that if the check is actually correct, this will create false
> positive for all the systemd .service files not started at boot
> (scheduled jobs, dbus activated,...).

The current policy requirement is that everything would need to provide
a sysvinit script, see https://bugs.debian.org/911165

Sadly the process to change this is stuck.

Ansgar



Bug#931847: Boggus package-supports-alternative-init-but-no-init.d-script test?

2019-07-11 Thread Laurent Bigonville
Package: lintian
Version: 2.16.0
Severity: important

Hi,

Isn't the new package-supports-alternative-init-but-no-init.d-script
test boggus (added by: #926471)?

The test is the following:

if (   $file =~ m,etc/sv/([^/]+)/run$,
or $file =~ m,lib/systemd/system/(.*)\.service,) {

Shouldn't that this be an "and"?

My understanding of the policy is that, if a package supports an
alternative init (other than systemd) it must also support sysvinit.

Also note that if the check is actually correct, this will create false
positive for all the systemd .service files not started at boot
(scheduled jobs, dbus activated,...).

Kind regards,

Laurent Bigonville

-- System Information:
Debian Release: bullseye/sid
  APT prefers unstable-debug
  APT policy: (500, 'unstable-debug'), (500, 'unstable'), (1, 
'experimental-debug'), (1, 'experimental')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 4.19.0-5-amd64 (SMP w/8 CPU cores)
Kernel taint flags: TAINT_FIRMWARE_WORKAROUND
Locale: LANG=fr_BE.UTF-8, LC_CTYPE=fr_BE.UTF-8 (charmap=UTF-8), 
LANGUAGE=fr_BE.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /usr/bin/dash
Init: systemd (via /run/systemd/system)
LSM: SELinux: enabled - Mode: Permissive - Policy name: refpolicy

Versions of packages lintian depends on:
ii  binutils   2.32.51.20190707-1
ii  bzip2  1.0.6-9.2
ii  diffstat   1.62-1
ii  dpkg   1.19.7
ii  dpkg-dev   1.19.7
ii  file   1:5.35-4
ii  gettext0.19.8.1-9
ii  gpg2.2.13-2
ii  intltool-debian0.35.0+20060710.5
ii  libapt-pkg-perl0.1.36+b1
ii  libarchive-zip-perl1.64-1
ii  libcapture-tiny-perl   0.48-1
ii  libcgi-pm-perl 4.40-1
ii  libclass-accessor-perl 0.51-1
ii  libclone-perl  0.41-1+b1
pn  libdigest-sha-perl 
ii  libdpkg-perl   1.19.7
ii  libemail-valid-perl1.202-1
ii  libfile-basedir-perl   0.08-1
ii  libio-async-perl   0.72-1
ii  libipc-run-perl20180523.0-1
ii  liblist-compare-perl   0.53-1
ii  liblist-moreutils-perl 0.416-1+b4
ii  libmoo-perl2.003004-2
ii  libparse-debianchangelog-perl  1.2.0-13
ii  libpath-tiny-perl  0.108-1
ii  libtext-levenshtein-perl   0.13-1
ii  libtimedate-perl   2.3000-2
ii  libtry-tiny-perl   0.30-1
ii  libtype-tiny-perl  1.004004-1
ii  liburi-perl1.76-1
ii  libxml-simple-perl 2.25-1
ii  libyaml-libyaml-perl   0.76+repack-1
ii  man-db 2.8.5-2
ii  patchutils 0.3.4-2
ii  perl   5.28.1-6
ii  t1utils1.41-3
ii  xz-utils   5.2.4-1

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

Versions of packages lintian suggests:
pn  binutils-multiarch 
ii  libhtml-parser-perl3.72-3+b3
ii  libtext-template-perl  1.55-1

-- no debconf information