Your message dated Fri, 17 Jul 2026 17:48:54 +0200
with message-id 
<tbhi3oalzin4ma7xywc5qqricrbsnxgqgmazescyut3xzcutkz@tcg5hgdfidqr>
and subject line Re: Bug#1136609: weechat-scripts uses pidof but does not 
depend on procps
has caused the Debian Bug report #1136609,
regarding weechat-scripts uses pidof but does not depend on procps
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact [email protected]
immediately.)


-- 
1136609: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1136609
Debian Bug Tracking System
Contact [email protected] with problems
--- Begin Message ---
Source: weechat-scripts
Version: 20260507-1
Usertags: pidof-without-procps

Dear maintainer(s) of weechat-scripts,

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

```
path: weechat-scripts_20260123-1/python/i3lock_away.py
def check():
    """Check for existance of process and set away if it isn't there"""
    pidof = subprocess.Popen("pidof i3lock",
                             shell=True, stdout=subprocess.PIPE)
    pidof.wait()
```

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

--- End Message ---
--- Begin Message ---
Hi Gioele,

On Thu, May 14, 2026 at 01:33:50AM +0200, Gioele Barabucci wrote:
[...]

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

I don't think it's necessary to add a hard dependency for this
particular script, as the benefit seems rather limited.

I'm therefore closing this issue.

Please feel free to reopen it if you think it deserves it.

Regards,

-- 
Emmanuel Bouthenot
  kolter@{openics,debian}.org   kolter@{libera,oftc}
  0x4B077723929D42C3/4096R

--- End Message ---

Reply via email to