X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;f=source%2Fcopy.cpp;h=31e62100d9751d991a0bf59529dd7699973e8ac5;hb=d1eb133ab529cdae131be7b150209f03189248f3;hp=30ab60e446b6ae8353112a5e707a8fb21ddf7355;hpb=4a46970b4a036e432d0bf7af5d5960ddf59082d3;p=builder.git

diff --git a/source/copy.cpp b/source/copy.cpp
index 30ab60e..31e6210 100644
--- a/source/copy.cpp
+++ b/source/copy.cpp
@@ -1,5 +1,7 @@
+#ifndef _WIN32
 #include <unistd.h>
 #include <sys/stat.h>
+#endif
 #include <msp/fs/dir.h>
 #include <msp/fs/stat.h>
 #include <msp/fs/utils.h>
@@ -8,19 +10,15 @@
 #include "builder.h"
 #include "copy.h"
 #include "installedfile.h"
+#include "internaltask.h"
 
 using namespace std;
 using namespace Msp;
 
-Copy::Copy(Builder &b):
-	Tool(b, "CP")
-{ }
-
-Target *Copy::create_target(const list<Target *> &sources, const string &arg) const
+Target *Copy::create_target(const vector<Target *> &sources, const string &arg)
 {
 	FileTarget &file_tgt = dynamic_cast<FileTarget &>(*sources.front());
-	const SourcePackage &pkg = dynamic_cast<const SourcePackage &>(*file_tgt.get_package());
-	InstalledFile *inst = new InstalledFile(builder, pkg, file_tgt, arg);
+	InstalledFile *inst = new InstalledFile(builder, *file_tgt.get_package(), file_tgt, arg);
 	inst->set_tool(*this);
 	return inst;
 }
@@ -28,38 +26,17 @@ Target *Copy::create_target(const list<Target *> &sources, const string &arg) co
 Task *Copy::run(const Target &target) const
 {
 	const InstalledFile &install = dynamic_cast<const InstalledFile &>(target);
-	Worker *worker = new Worker(install);
-	return new InternalTask(worker);
-}
-
-
-Copy::Worker::Worker(const InstalledFile &t):
-	target(t)
-{
-	launch();
+	InternalTask *task = new InternalTask([&install]{ return _run(install); });
+	task->add_file(install.get_path());
+	task->set_unlink();
+	return task;
 }
 
-void Copy::Worker::main()
+bool Copy::_run(const InstalledFile &install)
 {
-	const FileTarget &source = target.get_source();
+	const FileTarget &source = install.get_source();
 	const FS::Path &src_path = source.get_path();
-	const FS::Path &dst_path = target.get_path();
-	FS::mkpath(FS::dirname(dst_path), 0755);
-
-	// Remove old file.  Not doing this would cause Bad Stuff when installing libraries.
-	if(FS::exists(dst_path))
-	{
-		try
-		{
-			unlink(dst_path);
-		}
-		catch(const exception &e)
-		{
-			IO::print(IO::cerr, "%s\n", e.what());
-			status = Task::ERROR;
-			return;
-		}
-	}
+	const FS::Path &dst_path = install.get_path();
 
 	try
 	{
@@ -77,21 +54,24 @@ void Copy::Worker::main()
 	catch(const exception &e)
 	{
 		IO::print(IO::cerr, "%s\n", e.what());
-		status = Task::ERROR;
-		return;
+		return false;
 	}
 
+#ifndef _WIN32
 	// Preserve file permissions
 	struct stat st;
 	if(stat(src_path.str().c_str(), &st)==0)
 		chmod(dst_path.str().c_str(), st.st_mode&0777);
 
-	const FS::Path &link = target.get_symlink();
+	const FS::Path &link = install.get_symlink();
 	if(!link.empty())
 	{
 		FS::Path relpath = FS::relative(dst_path, FS::dirname(link));
+		if(FS::exists(link))
+			FS::unlink(link);
 		symlink(relpath.str().c_str(), link.str().c_str());
 	}
+#endif
 
-	status = Task::SUCCESS;
+	return true;
 }