Source: tecnoballz
Source-Version: 0.93.1-1
Severity: important
Tags: patch

Hi!

This package is using obsolete absolute paths when stating or calling
dpkg-statoverride. This has been a compatibility symlinks for a while
and got removed in dpkg 1.17.0, I'll reintroduce it in dpkg 1.17.22 but
will be removing it again in 1.18.x.

Please stop using absolute paths in maintainer scripts, as they will
suffer this kind of problem. For more info please see:

<https://lintian.debian.org/tags/command-with-path-in-maintainer-script.html>

although that lintian check does not currently trigger for this issue,
but it should.

Attached a patch fixing this, by always assuming that dpkg-statoverride
is present, which has been true for a very long time now.

Thanks,
Guillem
diff --git a/debian/tecnoballz.postinst b/debian/tecnoballz.postinst
index 72b5c48..6862ad7 100644
--- a/debian/tecnoballz.postinst
+++ b/debian/tecnoballz.postinst
@@ -9,8 +9,7 @@ case "$1" in
     configure)
 
     # tecnoballz is setuid "root:games".
-    if ! [ -x /usr/sbin/dpkg-statoverride ] || \
-        ! dpkg-statoverride --list "$PROG" >/dev/null ; then
+    if ! dpkg-statoverride --list "$PROG" >/dev/null ; then
       chown root:games "$PROG"
       chmod 2755 "$PROG"
     fi

Reply via email to