Source: seatd Version: 0.9.3-1 Usertags: pidof-without-procps Dear maintainer(s) of seatd,
it appears that seatd uses `pidof` in its testsuite, or that at least one of its binary packages uses `pidof` at runtime. Historically, `pidof` was provided by the Essential package `sysvinit-tools`, making an explicit dependency unnecessary. However `pidof` will soon be moved to `procps` and will no longer be part of the Essential set. Please add an explicit dependency on `procps`: * via the `Depends:` field of all binary packages of seatd that use `pidof` at runtime; * via the `Build-Depends:` field of seatd, if `pidof` is used in tests run at build-time; * via the `Depends:` field of `debian/control/tests`, if `pidof` is used in autopkgtests. To prevent any disruption for users of seatd, please add this dependency now, before `pidof` is moved from `sysvinit-utils` to `procps`. Alternatively, you could remove all uses of `pidof`. It is believed that seatd uses `pidof` due to the following code snippets: ``` path: seatd_0.9.3-1/debian/tests/seatd-launch # Ensure seatd-launch wrapper finds and runs seatd. SEATD_LOGLEVEL=debug seatd-launch -- sh -c 'printf "seatd PID: %d\n" $(pidof seatd)' ``` Feel free to close this issue if this is a false positive (for example if this code is in an unreachable code path). Regards, -- Gioele Barabucci
