Rework the Target class hierarchy

[builder.git] / source / objectfile.cpp

diff --git a/source/objectfile.cpp b/source/objectfile.cpp
index bba701d23bc48ca87d03f42790a78f4640a5f907..fd1c9758d8f498e04f227bd752cb80f866e4fd30 100644 (file)
--- a/source/objectfile.cpp
+++ b/source/objectfile.cpp
@@ -1,27 +1,35 @@
-#include <msp/algo.h>
-#include <msp/path/utils.h>
+/* $Id$
+
+This file is part of builder
+Copyright © 2006-2009  Mikko Rasa, Mikkosoft Productions
+Distributed under the LGPL
+*/
+
+#include <algorithm>
+#include <msp/fs/utils.h>
 #include "builder.h"
 #include "compile.h"
 #include "component.h"
 #include "install.h"
 #include "objectfile.h"
-#include "package.h"
 #include "sourcefile.h"
+#include "sourcepackage.h"
 
 using namespace std;
 using namespace Msp;
 
-ObjectFile::ObjectFile(Builder &b, const Component &c, SourceFile &src):
-       Target(b, &c.get_package(), generate_target_name(c, src.get_name())),
-       comp(c)
+ObjectFile::ObjectFile(Builder &b, const Component &c, SourceFile &s):
+       FileTarget(b, &c.get_package(), generate_target_path(c, FS::basename(s.get_path()))),
+       comp(c),
+       source(s)       
 {
        buildable=true;
-       add_depend(&src);
+       add_depend(&source);
 }
 
 void ObjectFile::find_depends()
 {
-       for(list<Target *>::iterator i=new_deps.begin(); i!=new_deps.end();)
+       for(TargetList::iterator i=new_deps.begin(); i!=new_deps.end();)
        {
                Target *tgt=*i;
                if(tgt->get_depends_ready())
@@ -36,31 +44,26 @@ void ObjectFile::find_depends()
        deps_ready=new_deps.empty();
 }
 
-Action *ObjectFile::build()
-{
-       return Target::build(new Compile(builder, depends.front()->get_name(), name, comp));
-}
 
 void ObjectFile::find_depends(Target *tgt)
 {
-       const string &tname=tgt->get_name();
-       string path=tname.substr(0, tname.rfind('/'));
-
        SourceFile *src=dynamic_cast<SourceFile *>(tgt);
        if(!src)
        {
-               Install *inst=dynamic_cast<Install *>(tgt);
-               if(inst)
-                       src=dynamic_cast<SourceFile *>(inst->get_depends().front());
+               if(Install *inst=dynamic_cast<Install *>(tgt))
+                       src=dynamic_cast<SourceFile *>(&inst->get_source());
        }
        if(!src)
                return;
 
+       FS::Path spath=src->get_path();
+       const StringList &incpath=comp.get_build_info().incpath;
+
        const list<string> &includes=src->get_includes();
        for(list<string>::const_iterator i=includes.begin(); i!=includes.end(); ++i)
        {
-               Target *hdr2=builder.get_header(*i, path, package->get_build_info().incpath);
-               if(hdr2 && !contains(depends, hdr2))
+               Target *hdr2=builder.get_header(*i, spath, incpath);
+               if(hdr2 && find(depends.begin(), depends.end(), hdr2)==depends.end())
                        add_depend(hdr2);
        }
 }
@@ -71,7 +74,13 @@ void ObjectFile::add_depend(Target *tgt)
        new_deps.push_back(tgt);
 }
 
-string ObjectFile::generate_target_name(const Component &comp, const string &src)
+Action *ObjectFile::create_action()
+{
+       return new Compile(builder, *this);
+}
+
+FS::Path ObjectFile::generate_target_path(const Component &comp, const string &src)
 {
-       return (comp.get_package().get_source()/"temp"/comp.get_name()/(Path::splitext(src.substr(src.rfind('/')+1)).base+".o")).str();
+       const SourcePackage &pkg=comp.get_package();
+       return pkg.get_temp_dir()/comp.get_name()/(FS::basepart(src)+".o");
 }