It should have always been die. >From gentoo-historical-2.git,
``` commit 39b01d59be512eb9ed10a66db19513e6b53130f4 Author: Tomas Chvatal <[email protected]> AuthorDate: Tue Apr 24 08:49:09 2012 Commit: Tomas Chvatal <[email protected]> CommitDate: Tue Apr 24 08:49:09 2012 Add myspell-r2.eclass which is just eclassdoced myspell.eclass with removed no-longer required stuff. ``` I have many questions. However, the one which is relevant here is: why did this new eclass *add* a new src_unpack, which called a single simple command, `unzip`, and then uses assert to catch pipe errors in it. assert is banned in EAPI 9, so it has to go anyway. Take this opportunity to perform a 14 year old cleanliness fix. Signed-off-by: Eli Schwartz <[email protected]> --- eclass/myspell-r2.eclass | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/eclass/myspell-r2.eclass b/eclass/myspell-r2.eclass index 05d48dc97dee..a8655314fd0b 100644 --- a/eclass/myspell-r2.eclass +++ b/eclass/myspell-r2.eclass @@ -1,4 +1,4 @@ -# Copyright 1999-2023 Gentoo Authors +# Copyright 1999-2026 Gentoo Authors # Distributed under the terms of the GNU General Public License v2 # @ECLASS: myspell-r2.eclass @@ -61,8 +61,7 @@ myspell-r2_src_unpack() { case ${f} in *.oxt) echo ">>> Unpacking "${DISTDIR}/${f}" to ${PWD}" - unzip -qoj "${DISTDIR}"/${f} - assert "failed unpacking ${DISTDIR}/${f}" + unzip -qoj "${DISTDIR}"/${f} || die "failed unpacking ${DISTDIR}/${f}" ;; *) unpack ${f} ;; esac -- 2.52.0
