From 72f8db43888f9aee3ce3b7def816c29b16270d07 Mon Sep 17 00:00:00 2001 From: Ralf Ertzinger Date: Sun, 16 Aug 2009 12:47:03 +0200 Subject: [PATCH] Integrate missing patches from C. Garbs --- bugs/issue-9ce9df1f86abe9075c1464b6bdf20128335d9c6e.yaml | 2 +- videosite/AsyncFileGetter.pm | 7 +++++-- 2 files changed, 6 insertions(+), 3 deletions(-) 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); -- 1.8.3.1