From: Ralf Ertzinger Date: Sun, 16 Aug 2009 10:47:03 +0000 (+0200) Subject: Integrate missing patches from C. Garbs X-Git-Url: https://git.camperquake.de/gitweb.cgi?p=videosite.git;a=commitdiff_plain;h=72f8db43888f9aee3ce3b7def816c29b16270d07 Integrate missing patches from C. Garbs --- diff --git a/bugs/issue-9ce9df1f86abe9075c1464b6bdf20128335d9c6e.yaml b/bugs/issue-9ce9df1f86abe9075c1464b6bdf20128335d9c6e.yaml index c83b792..ed54e42 100644 --- a/bugs/issue-9ce9df1f86abe9075c1464b6bdf20128335d9c6e.yaml +++ b/bugs/issue-9ce9df1f86abe9075c1464b6bdf20128335d9c6e.yaml @@ -9,7 +9,7 @@ component: videosite release: reporter: Christian Garbs status: :unstarted -disposition: +disposition: creation_time: 2008-07-06 19:19:44.496583 Z references: [] diff --git a/videosite/AsyncFileGetter.pm b/videosite/AsyncFileGetter.pm index 679f280..56aa835 100644 --- a/videosite/AsyncFileGetter.pm +++ b/videosite/AsyncFileGetter.pm @@ -1,5 +1,5 @@ # (c) 2007 by Ralf Ertzinger -# 2008 by Christian Garbs +# 2008-2009 by Christian Garbs # # licensed under GNU GPL v2 # @@ -15,6 +15,7 @@ use videosite::FileGetter; use strict; use LWP::Simple qw(!get); use File::Basename; +use File::Temp qw(tempfile); use MIME::Base64; sub new { @@ -50,7 +51,9 @@ sub get { $self->debug('Going to download %s to %s', $video->{'DLURL'}, $dlfile); - my $cmdline = "GET \"$video->{'DLURL'}\" > \"$dlfile\" &"; + my (undef, $tmpfile) = tempfile('videosite.tmp.XXXXXXXXXXXX', DIR => $dirname); + + my $cmdline = "( GET \"$video->{'DLURL'}\" > \"$tmpfile\" && mv \"$tmpfile\" \"$dlfile\" && chmod =rw \"$dlfile\" || rm -f \"$tmpfile\" ) &"; $self->debug(encode_base64($cmdline)); system($cmdline);