Hmm. Interestingly, it already uses File::Temp::tempdir -- but explicitly
sets a base DIR for some reason.


On Fri, Apr 19, 2013 at 1:47 PM, Mark Smith <notifications@github.com>wrote:

> In cgi-bin/DW/EmailPost/Entry.pm:
>
> > +# This file is a refactoring of "cgi-bin/LJ/Emailpost.pm"
> > +# from the original LiveJournal repository
> > +
> > +package DW::EmailPost::Entry;
> > +
> > +use base qw(DW::EmailPost::Base);
> > +use strict;
> > +
> > +use LJ::Protocol;
> > +
> > +use Date::Parse;
> > +use HTML::Entities;
> > +use IO::Handle;
> > +use XML::Simple;
> > +
> > +my $workdir = "/tmp";
>
> It doesn't have to be fixed, but there exists File::Temp which is even a
> core module, don't have to install it, and is the recommended way of
> allocating tempfiles.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/dreamwidth/dw-free/pull/346/files#r3868209>
> .
>