X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;ds=sidebyside;f=source%2Fcopy.cpp;h=70f05398fbac5ae5dc885d7c91c8c33eb95da4ae;hb=555190f7eafcf3a67750fc63872b23e17371aa98;hp=1f764dc038045575bb526c44939fcad4ac72add5;hpb=f1c967215e6b08095bdf07518472beca3067ec37;p=builder.git

diff --git a/source/copy.cpp b/source/copy.cpp
index 1f764dc..70f0539 100644
--- a/source/copy.cpp
+++ b/source/copy.cpp
@@ -1,5 +1,13 @@
+/* $Id$
+
+This file is part of builder
+Copyright © 2006-2007 Mikko Rasa, Mikkosoft Productions
+Distributed under the LGPL
+*/
+
 #include <errno.h>
 #include <fstream>
+#include <iostream>
 #include <msp/path/utils.h>
 #include "builder.h"
 #include "copy.h"
@@ -24,8 +32,11 @@ Copy::Copy(Builder &b, const Package &pkg, const Path::Path &s, const Path::Path
 
 int Copy::check()
 {
-	if(!worker)
+	if(!worker)  // True for dry run
+	{
+		signal_done.emit();
 		return 0;
+	}
 	
 	if(worker->get_done())
 	{
@@ -46,6 +57,7 @@ void Copy::Worker::main()
 {
 	Path::mkpath(copy.dest.subpath(0, copy.dest.size()-1), 0755);
 	
+	// Remove old file.  Not doing this would cause Bad Stuff when installing libraries.
 	if(unlink(copy.dest.str().c_str())<0 && errno!=ENOENT)
 	{
 		int err=errno;
@@ -70,6 +82,7 @@ void Copy::Worker::main()
 		return;
 	}
 
+	// Actual transfer loop
 	char buf[16384];
 	while(!in.eof())
 	{
@@ -77,6 +90,7 @@ void Copy::Worker::main()
 		out.write(buf, in.gcount());
 	}
 
+	// Preserve file permissions
 	struct stat st;
 	Path::stat(copy.src, st);
 	chmod(copy.dest.str().c_str(), st.st_mode&0777);