Re: SVN texlive (new) commit: r2653 - in texlive-new/trunk: . all/debian

[Frank Küster <frank@debian.org>]

Norbert Preining <preining@alioth.debian.org> wrote:

> fixes:
> - wrong directives in the config file
> - bad definition of check_texmfcnf in rules.in, needs a || true

- test -z $$DEBIAN_NOCHECK_TEXMFCNF && sh debian/compare-texmf.cnf.sh
+ (test -z $$DEBIAN_NOCHECK_TEXMFCNF && sh debian/compare-texmf.cnf.sh) || true

I don't think this is right.  If the check is done
(i.e. DEBIAN_NOCHECK_TEXMFCNF is not set), then compare-texmf.cnf.sh's
return status should not be discarded; instead it should fail if
texmf.cnf isn't clean.

The right line would rather look like this:

 test -n $$DEBIAN_NOCHECK_TEXMFCNF || sh debian/compare-texmf.cnf.sh

or what am I missing?

Regards, Frank
-- 
Dr. Frank Küster
Single Molecule Spectroscopy, Protein Folding @ Inst. f. Biochemie, Univ. Zürich
Debian Developer (teTeX/TeXLive)