[Python-modules-team] [Python-modules-commits] r24400 - in packages/pyparsing/trunk/debian (5 files)

[Jakub Wilk]

>-	find $(CURDIR)/debian/python{,3}-pyparsing \
>-	    \( -name '*.html' -o -name '*.css' -o -name '*.py' \
>-	                      -o -name 'README' \) \
>-	    -type f -print0 \
>-	| xargs -0 --no-run-if-empty fromdos -p --

Was dropping this intentional? The change is not documented in the 
changelog, and tofrodos is still in Build-Depends-Indep (even though 
it's no longer used AFAICS).

OTOH the original code didn't work anyway if /bin/sh was dash:

find: `/build/pyparsing-gronMK/pyparsing-1.5.6+dfsg1/debian/python{,3}-pyparsing': No such file or directory

>+	for i in `pyversions -s` ; do \
>+		$$i setup.py install --install-layout=deb \
>+	    --root=$(CURDIR)/debian/python-pyparsing ; \
>+	done

"set -e" is missing here (see Policy §4.6).

-- 
Jakub Wilk