#1136552 php-defaults uses pidof but does not depend on procps

#1136552#5
Date:
2026-05-13 23:30:35 UTC
From:
To:
Dear maintainer(s) of php-defaults,

it appears that php-defaults 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 php-defaults
  that use `pidof` at runtime;
* via the `Build-Depends:` field of php-defaults, 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 php-defaults, 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 php-defaults uses `pidof` due to the following
code snippets:

```
path: php-defaults_99/sessionclean
done
# first find all open session files and touch them (hope it's not massive amount of files)
for pid in $(pidof $proc_names); do
    find "/proc/$pid/fd" -ignore_readdir_race -lname "$save_path/sess_*" -exec touch -c {} \; 2>/dev/null
done ) | \
```

Feel free to close this issue if this is a false positive (for example
if this code is in an unreachable code path).

Regards,

#1136552#10
Date:
2026-10-16 09:38:23 UTC
From:
To:
Control: tags -1 patch
In fact `sessionclean` is shipped as part of `php-common` and called
from a systemd service, so `php-common` must depend on `pidof`.

You can find a MR with a fix for this issue at
<https://salsa.debian.org/php-team/php-defaults/-/merge_requests/6>.

Regards,

#1136552#19
Date:
2026-10-18 12:50:47 UTC
From:
To:
We believe that the bug you reported is fixed in the latest version of
php-defaults, which is due to be installed in the Debian FTP archive.

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to 1136552@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Ondřej Surý <ondrej@debian.org> (supplier of updated php-defaults package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing ftpmaster@ftp-master.debian.org)
Format: 1.8
Date: Fri, 16 Oct 2026 12:17:48 +0200
Source: php-defaults
Architecture: source
Version: 101
Distribution: unstable
Urgency: medium
Maintainer: Debian PHP Maintainers <team+pkg-php@tracker.debian.org>
Changed-By: Ondřej Surý <ondrej@debian.org>
Closes: 1136552
Changes:
 php-defaults (101) unstable; urgency=medium
 .
   [ Gioele Barabucci ]
   * d/control.in: Add procps dependency to php-common (Closes: #1136552)
Checksums-Sha1:
 9850c528915e93f911596f20337bde4cd516010b 3297 php-defaults_101.dsc
 81effbecc889a7e4b395c80cb6c3fdd309fb9a18 17852 php-defaults_101.tar.xz
 c5e7bab856c21df1b226ffa76e739b9172b03c0c 13410 php-defaults_101_amd64.buildinfo
Checksums-Sha256:
 e54e0ee7f83a1683156f8f9ee61545fad4b5b2ff21f12943b0be5bccdaff8042 3297 php-defaults_101.dsc
 4bb69a3c5260b1118d47e303c312bc179193924c840958c5c1293ddc0ecb5235 17852 php-defaults_101.tar.xz
 bf391bcc63429877bf5fcbdb07f5cc4f6e02cc1b2ee53f9f70e8bedf03147cd6 13410 php-defaults_101_amd64.buildinfo
Files:
 09f427171efb9cf2b6c9399630d9bfd5 3297 php optional php-defaults_101.dsc
 69c8340f7cdd7ed306ceffecca4efc93 17852 php optional php-defaults_101.tar.xz
 736ca4ec4dadb47d318f2526a55774d6 13410 php optional php-defaults_101_amd64.buildinfo
-----BEGIN PGP SIGNATURE-----

iQKTBAEBCgB9FiEEw2Gx4wKVQ+vGJel9g3Kkd++uWcIFAmrUvchfFIAAAAAALgAo
aXNzdWVyLWZwckBub3RhdGlvbnMub3BlbnBncC5maWZ0aGhvcnNlbWFuLm5ldEMz
NjFCMUUzMDI5NTQzRUJDNjI1RTk3RDgzNzJBNDc3RUZBRTU5QzIACgkQg3Kkd++u
WcIUPw/+KWzt8yL3MAI/IqcQTDSmkMc1YjruxJ9jfMZkGdx2r3GxIlQ29SK75wy3
oqa7Pas6c5UVxxth1Vbdgjloqyf41/g/KAF+BKe3TfhIfRsYCoiBXR5FAeE6ObqN
7fWnJOp1BnHugxTIxRO1WwDAxAEJdVxJF9SAsxWLYBFshLvEnUK8xcor1PtDno2e
Y5V7wRC2df67gCgWoRLmRugrYlgt6xz12VYH18OLmKyQrmA0gOTMqEBO8GOZHoiW
7r3o8VjPaIcfHvd4scuIeIXWZkqRnLbRlM/i4/Ai7gSdSGVduVSGoqDq6w2tnOI6
HjjjGWo5ZUWSSNIhe9FyjF2bqxgDLeuUb/gVwDNpCpK8kayzhROIx101RK6aJfqq
w/RkfHOhJ0LkrkXGDmGHQnUj7z2iD81Kx2/jbzGQg0SuxH4K/bYyh+QpsubMGoFH
4mvUZN9heCD3+KmSB+L8llj0GvxLeSSBIATNc3dPta8hPwruxuSuKcavwCoLRwlP
wv7mGEp41is2g52+el1OMzJGvfbZPT0FubjLcOMMWLHrgrWKeic3JoOOzAJn216y
ONgukvp5K1wlnJwIotBn04ZERyf6MrcPBKEtjnyTYZ3qswOjesPLx0y4F2lDxZ5w
tQ+vhOpQoKhraXeHCaGA9UVPxM4GoXFhEXvVQbVbrwwEIZfv65U=
=OY7U
-----END PGP SIGNATURE-----