Big rewrite for a more tool-centric approach

[builder.git] / source / tar.cpp

diff --git a/source/tar.cpp b/source/tar.cpp
index d3301a0dbcb1ae1e3d0bae93689d9400c8c9c772..82330caa9762f34531be65b56fcffcdd2b2eb3f6 100644 (file)
--- a/source/tar.cpp
+++ b/source/tar.cpp
@@ -11,33 +11,44 @@
 using namespace std;
 using namespace Msp;
 
-Tar::Tar(Builder &b, const TarBall &t):
-       InternalAction(b),
-       tarball(t)
+Tar::Tar(Builder &b):
+       Tool(b, "TAR")
+{ }
+
+Target *Tar::create_target(const list<Target *> &sources, const string &arg) const
 {
-       string basename = FS::basename(tarball.get_path());
-       announce(tarball.get_package()->get_name(), "TAR ", basename);
-       if(builder.get_verbose()>=2)
-               IO::print("Create %s\n", basename);
+       if(!sources.front()->get_package())
+               throw invalid_argument("Tar::create_target");
+       const SourcePackage &src_pkg = dynamic_cast<const SourcePackage &>(*sources.front()->get_package());
+
+       TarBall *tarball = new TarBall(builder, src_pkg, arg);
+       for(list<Target *>::const_iterator i=sources.begin(); i!=sources.end(); ++i)
+               tarball->add_depend(*i);
 
-       if(!builder.get_dry_run())
-               worker = new Worker(*this);
+       return tarball;
+}
+
+Task *Tar::run(const Target &target) const
+{
+       const TarBall &tarball = dynamic_cast<const TarBall &>(target);
+       Worker *worker = new Worker(tarball);
+       return new InternalTask(worker);
 }
 
 
-Tar::Worker::Worker(Tar &t):
-       tar(t)
+Tar::Worker::Worker(const TarBall &tb):
+       tarball(tb)
 {
        launch();
 }
 
 void Tar::Worker::main()
 {
-       const FS::Path &pkg_src = tar.tarball.get_package()->get_source();
-       FS::Path basedir = FS::basepart(FS::basename(tar.tarball.get_path()));
+       const FS::Path &pkg_src = tarball.get_package()->get_source();
+       FS::Path basedir = FS::basepart(FS::basename(tarball.get_path()));
 
-       IO::File out(tar.tarball.get_path().str(), IO::M_WRITE);
-       const TargetList &deps = tar.tarball.get_depends();
+       IO::File out(tarball.get_path().str(), IO::M_WRITE);
+       const TargetList &deps = tarball.get_depends();
        for(TargetList::const_iterator i=deps.begin(); i!=deps.end(); ++i)
        {
                FileTarget *ft = dynamic_cast<FileTarget *>(*i);
@@ -51,8 +62,8 @@ void Tar::Worker::main()
                if(rel_path.size()>99)
                {
                        IO::print("Can't store %s in tar archive - too long name\n", rel_path);
-                       error = true;
-                       break;
+                       status = Task::ERROR;
+                       return;
                }
 
                memcpy(buf, rel_path.data(), rel_path.size());
@@ -82,7 +93,7 @@ void Tar::Worker::main()
                }
        }
 
-       done = true;
+       status = Task::SUCCESS;
 }
 
 void Tar::Worker::store_number(char *buf, unsigned value, unsigned length)