pidof removal progress ($621) · Snippets

nginx: https://paste.debian.net/1254794/

popa3d: patch in https://bugs.debian.org/1020707 (made RC partly to check if package is alive, no activity since 2015).
sslh: patch in https://bugs.debian.org/1020711
dns-flood-detector: NOP! postinst is only a comment, prerm is only relevant when upgrading from <= "1.12-1" but even old-old-stable has 1.20. (So remove both occurances?)

These could all be fixed by using invoke-rc.d $NAME status instead:

kopano-webapp-nginx
madwimax (udevadm trigger, same as nut)
nghttp2-proxy (NAME=nghttpx)
tgt
yadifa

TODO:

xdm needs investigation. (Checks for active session (children) on top of invoke-rc.d xdm status.)

dbus-broker does not need changes, as pidof is only used when not running with systemd (and sysvinit will have a dep on the package), and only as a fallback, and behind a command -v check.

@ah unfortunately all the paste.debian.net links are dead :-( are there copies of those patches on salsa or elsewhere by any chance?

openvpn: https://salsa.debian.org/debian/openvpn/-/merge_requests/18

fixed in unstable (et.al.)

portsentry - no mention of pidof in unstable/sid version.

ok as is?!

xdm - uses pidof ... || true so ok as is?
wdm - copy of xdm prerm, see xdm.
rygel - uses pidof ... || true by design (originally written by ah@d.o)

only used in dead code or comments

ps-watcher - only when upgrading from <= "1.06-6", old-old-stable (bullseye) has 1.08-10.

removed from unstable/archive

gconf2
snort
maradns

add dep on pidof package?

zeitgeist-core - needs pidof to exist or will fail. Either rewrite or depend.

needs further investigation

laptop-mode-tools - all uses are guarded except install.sh which runs in build env.
gnumeric - ...
gnome-activity-journal - code copy from zeitgeist-core ?

nut: https://salsa.debian.org/debian/nut/-/merge_requests/10

gnome-activity-journal: https://salsa.debian.org/sudip/gnome-activity-journal/-/merge_requests/1

gnumeric: only uses pidof on upgrades, and already depends on procps, so nothing to do, everything will work out of the box once pidof moves to procps

zeitgeist-core: false positive, it's dead code, shell function in maintainer script that is never called

laptop-mode-tools: https://salsa.debian.org/debian/laptop-mode-tools/-/merge_requests/1

yadifa: https://salsa.debian.org/dns-team/yadifa/-/merge_requests/2

tgt: https://salsa.debian.org/debian/tgt/-/merge_requests/5

nghttp2-proxy: https://salsa.debian.org/debian/nghttp2/-/merge_requests/9

Thanks @ah, thanks @bluca! We are now just 10 packages away from removing Essential: yes from sysvinit-utils.

dbus: https://salsa.debian.org/utopia-team/dbus/-/merge_requests/11

laptop-mode-tools approved and uploaded

dbus is merged https://salsa.debian.org/utopia-team/dbus/-/merge_requests/13

nghttp2 is merged

dbus is uploaded

gnome-activity-journal is merged