I know this code is just a prototype, but somebody might read the bug log and then want to copy it to their package, so I feel obliged to point out some mistakes:

* Jari Aalto <jari.aa...@cante.net>, 2009-12-22, 04:08:
   LC_ALL= LANG=C pod2man $POD2MAN_FLAGS \

This is not a correct way to sanitize localization variables. You should set LC_ALL=C (and then you don't need to touch LANG).

            --center="$PODCENTER" \
            --name="$PACKAGE" \
            --section="$MANSECT" \
            $(MANPOD) \
   | sed "s,[Pp]erl v[0-9.]*,$PACKAGE," \

The use of sed here is rather brittle, as "Perl v3.14" could occur naturally in the POD text. You could use the --release option instead.

--
Jakub Wilk


--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to