Hi Thomas,

On Tuesday 26 August 2008 08:17, Thomas Goirand wrote:
> Thijs Kinkhorst wrote:
> > First, I think it's always a good idea not to enable DEBUG by default.
>
> Sure, it's a mistake, I perfectly understand this.
>
> > Second, I don't think that it requires a "rewrite of the entire file" to
> > fix it. Using PHP's tempnam() function to get the filenames instead of
> > the hardcoded path names with PID is a change of just a few lines.
>
> Did you mean Perl mktemp()? It's not a PHP script!!! :)

Sorry, I confused this bug with another one, there's so many of them all of a 
sudden :-)

> That makes me 
> think that I might have missed some Perl dependencies. The script uses
> the following:
>
> use strict;
> use MIME::Tools;
> use MIME::Parser;
> use File::MkTemp;
>
> does any of you knows what it corresponds in terms of Debian deps?

The file search on packages.debian.org is very helpful for this (search e.g. 
for "Tools.pm" or "Parser.pm").

With respect to the File::MkTemp, I think you need to use File::Temp as the 
MkTemp one is not available in Debian. This module also provides a mktemp 
function.

> Last, would a patch like the attached one would do? I'm all but good in
> Perl, so I might need help on that one.

That would work indeed if you change the included module (and verify that that 
indeed also works, of course).


cheers,
Thijs

Attachment: pgp5zRzGVxkPJ.pgp
Description: PGP signature

Reply via email to