Integrate missing patches from C. Garbs

diff --git a/bugs/issue-9ce9df1f86abe9075c1464b6bdf20128335d9c6e.yaml b/bugs/issue-9ce9df1f86abe9075c1464b6bdf20128335d9c6e.yaml
index c83b792..ed54e42 100644 (file)
--- a/bugs/issue-9ce9df1f86abe9075c1464b6bdf20128335d9c6e.yaml
+++ b/bugs/issue-9ce9df1f86abe9075c1464b6bdf20128335d9c6e.yaml
@@ -9,7 +9,7 @@ component: videosite
 release: 
 reporter: Christian Garbs <mitch@cgarbs.de>
 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 (file)
--- a/videosite/AsyncFileGetter.pm
+++ b/videosite/AsyncFileGetter.pm
@@ -1,5 +1,5 @@
 # (c) 2007 by Ralf Ertzinger <ralf@camperquake.de>
-#     2008 by Christian Garbs <mitch@cgarbs.de>
+#     2008-2009 by Christian Garbs <mitch@cgarbs.de>
 #
 # 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);