[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#1001051: texlive-base postinst uses which



Package: texlive-base
Version: 2021.20211127-1
Severity: normal
Tags: patch
X-Debbugs-Cc: jspricke@debian.org

Hi,

installing textlive-base gives:

$ sudo apt install texlive-base
[..]
Setting up texlive-base (2021.20211127-1) ...
/usr/bin/which: this version of `which' is deprecated; use `command -v' in scripts instead.
/usr/bin/which: this version of `which' is deprecated; use `command -v' in scripts instead.
/usr/bin/which: this version of `which' is deprecated; use `command -v' in scripts instead.
/usr/bin/which: this version of `which' is deprecated; use `command -v' in scripts instead.

The attached patch fixes this.
You can also just press merge here:

https://github.com/debian-tex/texlive-nonbin/pull/3

Answering your question from there:

> According to the CTTE decision the "which" command will continue to exist. Should we nevertheless prepare for the removal of it.

The CTTE stated that "For the Debian 12 release, we expect which(1) to
be in either an Essential package or a transitively Essential package"
this does not mean that which(1) will be there indefinitely. Contrary to
which, command -v is specified by POSIX and meant to stay. So this patch
is to help for a future transition.
>From eebe6baf1b311c6f4fd01e788f8f4769712de071 Mon Sep 17 00:00:00 2001
From: Jochen Sprickerhof <git@jochen.sprickerhof.de>
Date: Wed, 1 Dec 2021 23:35:27 +0100
Subject: [PATCH] Use command -v instead of which

---
 texlive-base/debian/texlive-base.postinst.pre | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/texlive-base/debian/texlive-base.postinst.pre b/texlive-base/debian/texlive-base.postinst.pre
index 640e11c..05fbc13 100644
--- a/texlive-base/debian/texlive-base.postinst.pre
+++ b/texlive-base/debian/texlive-base.postinst.pre
@@ -34,7 +34,7 @@ case "$1" in
     rm -f $file.ucf-new
     rm -f $file.ucf-dist
     ucf --purge $file
-    if test -x "`which ucfr`" ; then
+    if command -v ucfr ; then
       ucfr --purge texlive-base $file
     fi
   done

Reply to: