commit:     58a6197c998321bce1c47b38d930847039af64c9
Author:     Florian Schmaus <flow <AT> gentoo <DOT> org>
AuthorDate: Thu Feb 29 13:39:18 2024 +0000
Commit:     Florian Schmaus <flow <AT> gentoo <DOT> org>
CommitDate: Thu Feb 29 13:39:18 2024 +0000
URL:        https://gitweb.gentoo.org/proj/tex-overlay.git/commit/?id=58a6197c

*/*: drop --ignore-errors, use nonftal

Signed-off-by: Florian Schmaus <flow <AT> gentoo.org>

 app-text/texlive-core/texlive-core-2023.ebuild |  6 +++---
 eclass/texlive-common.eclass                   | 17 ++++-------------
 2 files changed, 7 insertions(+), 16 deletions(-)

diff --git a/app-text/texlive-core/texlive-core-2023.ebuild 
b/app-text/texlive-core/texlive-core-2023.ebuild
index 6205862..0e13faf 100644
--- a/app-text/texlive-core/texlive-core-2023.ebuild
+++ b/app-text/texlive-core/texlive-core-2023.ebuild
@@ -460,9 +460,9 @@ pkg_postinst() {
        # texlive-core.
 
        # TODO: Research the rationale of calling etexmf-update and
-       # eftmutil-sys here and the reasons why it fails.
-       etexmf-update --ignore-errors
-       efmtutil-sys --ignore-errors
+       # eftmutil-sys here and the reasons why it sometimes fails.
+       nonfatal etexmf-update
+       nonfatal efmtutil-sys
 
        elog
        elog "If you have configuration files in ${EPREFIX}/etc/texmf to merge,"

diff --git a/eclass/texlive-common.eclass b/eclass/texlive-common.eclass
index adfb7da..85cdb8f 100644
--- a/eclass/texlive-common.eclass
+++ b/eclass/texlive-common.eclass
@@ -168,13 +168,10 @@ dobin_texmf_scripts() {
 }
 
 # @FUNCTION: etexmf-update
-# @USAGE: [--ignore-errors]
 # @DESCRIPTION:
 # Runs texmf-update if it is available and prints a warning otherwise. This
 # function helps in factorizing some code.  Useful in ebuilds' pkg_postinst and
 # pkg_postrm phases.
-# If --ignore-errors is provided, then a non-zero exit status of texmf-update 
does not
-# cause die to be invoked.
 # Called by app-text/dvipsk, app-text/texlive-core, dev-libs/kpathsea, and
 # texlive-module.eclass.
 etexmf-update() {
@@ -182,8 +179,8 @@ etexmf-update() {
                if [[ -z ${ROOT} && -x "${EPREFIX}"/usr/sbin/texmf-update ]] ; 
then
                        "${EPREFIX}"/usr/sbin/texmf-update
                        local res="${?}"
-                       if [[ "${?}" -ne 0 && "${#}" -ne 1 && "${1}" != 
"--ignore-errors" ]] && ver_test -ge 2023; then
-                               die "texmf-update returned non-zero exit status 
${res}"
+                       if [[ "${?}" -ne 0 ]] && ver_test -ge 2023; then
+                               die -n "texmf-update returned non-zero exit 
status ${res}"
                        fi
                else
                        ewarn "Cannot run texmf-update for some reason."
@@ -194,22 +191,16 @@ etexmf-update() {
 }
 
 # @FUNCTION: efmtutil-sys
-# @USAGE: [--ignore-errors]
 # @DESCRIPTION:
 # Runs fmtutil-sys if it is available and prints a warning otherwise. This
 # function helps in factorizing some code. Used in ebuilds' pkg_postinst to
 # force a rebuild of TeX formats.
-# If --ignore-errors is provided, then a non-zero exit status of fmtutil-sys 
does not
-# cause die to be invoked.
 efmtutil-sys() {
        if has_version 'app-text/texlive-core' ; then
                if [[ -z ${ROOT} && -x "${EPREFIX}"/usr/bin/fmtutil-sys ]] ; 
then
                        einfo "Rebuilding formats"
-                       "${EPREFIX}"/usr/bin/fmtutil-sys --all &> /dev/null
-                       local res="${?}"
-                       if [[ "${?}" -ne 0 && "${#}" -ne 1 && "${1}" != 
"--ignore-errors" ]]; then
-                               die "fmtutil-sys returned non-zero exit status 
${res}"
-                       fi
+                       "${EPREFIX}"/usr/bin/fmtutil-sys --all &> /dev/null \
+                               || die -n "fmtutil-sys returned non-zero exit 
status ${res}"
                else
                        ewarn "Cannot run fmtutil-sys for some reason."
                        ewarn "Your formats might be inconsistent with your 
installed ${PN} version"

Reply via email to