severity 464989 minor
user [EMAIL PROTECTED]
usertags 464989 -goal-dash
Thanks

For anyone wanting to apply the patch and upload the package, please beware
that this is not really a bashism that would affect Debian.

At the beginning of the block before the applied patch, there is a:

    # Improvise mktemp in case we need it too!
        if ! command -v mktemp > /dev/null 2>&1 ; then

So, this is block is only executed in the case that mktemp is not present,
and mktemp is "Essential: yes" in Debian, so the block should never be
executed.

Also, beware that even if "command -v mktemp" works both in dash and bash,
it's not really POSIX, so it's not a really portable solution, the portable
solution would be:

<Clint> if ! which mktemp

Finally, the $RANDOM number could be obtained with only two external
commands with the following line:

<seanius> well, you could get the equivalent with two commands anyway
<seanius> tr -c -d '0-9' < /dev/urandom| dd bs=1 count=5 2>/dev/null

Since this is not a real bug, and the script itself is too damn ugly, I'm
downgrading and untagging the bug.

-- 
 Bezos,     (o.
     Marga. (/)_

Attachment: signature.asc
Description: Digital signature

Reply via email to