X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;f=source%2Ftar.cpp;h=bfb4f5f9761f245dfa83cd8279a0b7a01c05bab0;hb=699493ee7346c578537f4e310702c57012686c22;hp=0b0c38df775696fcf1e98d8333a3357df68af305;hpb=555190f7eafcf3a67750fc63872b23e17371aa98;p=builder.git

diff --git a/source/tar.cpp b/source/tar.cpp
index 0b0c38d..bfb4f5f 100644
--- a/source/tar.cpp
+++ b/source/tar.cpp
@@ -1,14 +1,17 @@
 /* $Id$
 
 This file is part of builder
-Copyright © 2007 Mikko Rasa, Mikkosoft Productions
+Copyright © 2007-2009  Mikko Rasa, Mikkosoft Productions
 Distributed under the LGPL
 */
 
-#include <iostream>
-#include <msp/path/utils.h>
+#include <cstring>
+#include <msp/fs/stat.h>
+#include <msp/fs/utils.h>
+#include <msp/io/file.h>
+#include <msp/io/print.h>
 #include "builder.h"
-#include "package.h"
+#include "sourcepackage.h"
 #include "tar.h"
 #include "tarball.h"
 
@@ -16,108 +19,84 @@ using namespace std;
 using namespace Msp;
 
 Tar::Tar(Builder &b, const TarBall &t):
-	Action(b),
-	tarball(t),
-	worker(0)
+	InternalAction(b),
+	tarball(t)
 {
-	string basename=tarball.get_name().substr(tarball.get_name().rfind('/')+1);
+	string basename = FS::basename(tarball.get_path());
 	announce(tarball.get_package()->get_name(), "TAR ", basename);
 	if(builder.get_verbose()>=2)
-		cout<<"Create "<<basename<<'\n';
-	
-	if(!builder.get_dry_run())
-		worker=new Worker(*this);
-}
-
-Tar::~Tar()
-{
-	delete worker;
-}
+		IO::print("Create %s\n", basename);
 
-int Tar::check()
-{
-	if(!worker)  // True for dry run
-	{
-		signal_done.emit();
-		return 0;
-	}
-	
-	if(worker->get_done())
-	{
-		signal_done.emit();
-		worker->join();
-		return worker->get_error()?1:0;
-	}
-	
-	return -1;
+	if(!builder.get_dry_run())
+		worker = new Worker(*this);
 }
 
 
 Tar::Worker::Worker(Tar &t):
-	tar(t),
-	done(false),
-	error(false)
+	tar(t)
 {
 	launch();
 }
 
 void Tar::Worker::main()
 {
-	const Path::Path &pkg_src=tar.tarball.get_package()->get_source();
-	Path::Path basedir=Path::splitext(Path::basename(tar.tarball.get_name())).base;
+	const FS::Path &pkg_src = tar.tarball.get_package()->get_source();
+	FS::Path basedir = FS::basepart(FS::basename(tar.tarball.get_path()));
 
-	ofstream out(tar.tarball.get_name().c_str());
-	const TargetList &deps=tar.tarball.get_depends();
+	IO::File out(tar.tarball.get_path().str(), IO::M_WRITE);
+	const TargetList &deps = tar.tarball.get_depends();
 	for(TargetList::const_iterator i=deps.begin(); i!=deps.end(); ++i)
 	{
+		FileTarget *ft = dynamic_cast<FileTarget *>(*i);
+		if(!ft)
+			continue;
+
 		char buf[4096];
 		memset(buf, 0, 512);
 
-		string rel_path=(basedir/relative((*i)->get_name(), pkg_src)).str();
+		string rel_path = (basedir/relative(ft->get_path(), pkg_src)).str();
 		if(rel_path.size()>99)
 		{
-			cout<<"Can't store "<<rel_path<<" in tar archive - too long name\n";
-			error=true;
+			IO::print("Can't store %s in tar archive - too long name\n", rel_path);
+			error = true;
 			break;
 		}
 
 		memcpy(buf, rel_path.data(), rel_path.size());
 
-		struct stat st;
-		Path::stat((*i)->get_name(), st);
+		struct stat st = FS::stat(ft->get_path());
 		store_number(buf+100, st.st_mode, 7);
 		store_number(buf+108, st.st_uid, 7);
 		store_number(buf+116, st.st_gid, 7);
 		store_number(buf+124, st.st_size, 11);
 		store_number(buf+136, st.st_mtime, 11);
-		buf[156]='0';
+		buf[156] = '0';
 
 		memset(buf+148, ' ', 8);
-		unsigned chk=0;
+		unsigned chk = 0;
 		for(unsigned j=0; j<512; ++j)
-			chk+=static_cast<unsigned char>(buf[j]);
+			chk += static_cast<unsigned char>(buf[j]);
 		store_number(buf+148, chk, 7);
-		buf[155]=0;
+		buf[155] = 0;
 
 		out.write(buf, 512);
-		ifstream in((*i)->get_name().c_str());
+		IO::File in(ft->get_path().str());
 		for(int j=0; j<st.st_size; j+=4096)
 		{
-			in.read(buf, 4096);
-			unsigned len=in.gcount();
-			len+=((~len)+1)&0777;
+			unsigned len = in.read(buf, 4096);
+			len += ((~len)+1)&0777;
 			out.write(buf, len);
 		}
 	}
 
-	done=true;
+	done = true;
 }
 
 void Tar::Worker::store_number(char *buf, unsigned value, unsigned length)
 {
 	for(unsigned i=length; i--;)
 	{
-		buf[i]='0'+value%8;
-		value/=8;
+		buf[i] = '0'+value%8;
+		value /= 8;
 	}
 }