ppmrainbow(1) insists on optional -t tmpdir being given

[Ralph Corderoy]

Binary package hint: netpbm

Ubuntu 9.04 and 9.10, package netpbm 2:10.0-12 and 2:10.0-12ubuntu1.
ppmrainbow(1) fails unless the optional -t option is given specifying a
temporary directory. 9.04:

    $ ppmrainbow -w 9 -h 1 red cyan | pnmnoraw
    Use of uninitialized value $tmpdir in sprintf at /usr/bin/ppmrainbow line 58.
    sh: cannot create /000.ppm: Permission denied
    pnmnoraw: EOF / read error reading magic number
    $

Line 58 is

    push @outlist, my $outfile = sprintf "%s/%03u.ppm", $tmpdir, $n;

and $tmpdir hasn't been defined by that point unless the -t option has been
given. $tempdir, note the `e', has been defined earlier, using File::Temp's
tempdir, but that doesn't make use of the $tmpdir set by the command line.

ppmrainbow seems confused. The -t option probably shouldn't exist; $TMPDIR in
the environment is sufficient. And it doesn't work as currently documented, -t
has to be given.

[Ralph Corderoy (ralph-inputplus)]

Still present in 9.10's netpbm 2:10.0-12ubuntu1.1. Can someone *please* confirm this bug. It's trivial to do the above test.

[Dan Fandrich (dan-coneharvesters)]

Still present in 10.04 (using netpbm 2:10.0-12.1ubuntu1)

[Ralph Corderoy (ralph-inputplus)]

Thanks Dan. Could you change the status from New to Confirmed then please. It may make something happen in getting it fixed.

[Dan Fandrich (dan-coneharvesters)]

Confirmed! FWIW, it looks like an upstream problem. The Ubuntu diff doesn't touch ppmrainbow. netpbm-10.35.76 from upstream doesn't suffer from this problem.