X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;f=source%2Fobjectfile.cpp;h=cd870592530f74b0220b6e5ed9692c090db16e8b;hb=409a427f912f9c203b102beed0816b53b250931f;hp=6790082861875c087fee7f34d953e3e4445f8bf5;hpb=be8a901dfc026f61db46d5d64a41cecc619bc97d;p=builder.git

diff --git a/source/objectfile.cpp b/source/objectfile.cpp
index 6790082..cd87059 100644
--- a/source/objectfile.cpp
+++ b/source/objectfile.cpp
@@ -1,11 +1,12 @@
 /* $Id$
 
 This file is part of builder
-Copyright © 2006-2007 Mikko Rasa, Mikkosoft Productions
+Copyright © 2006-2010  Mikko Rasa, Mikkosoft Productions
 Distributed under the LGPL
 */
 
-#include <msp/path/utils.h>
+#include <algorithm>
+#include <msp/fs/utils.h>
 #include "builder.h"
 #include "compile.h"
 #include "component.h"
@@ -17,70 +18,52 @@ Distributed under the LGPL
 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::relative(s.get_path(), c.get_package().get_source()).str())),
+	comp(c),
+	source(s)	
 {
-	buildable=true;
-	add_depend(&src);
+	buildable = true;
+	add_depend(&source);
 }
 
-/**
-Processes as many new dependences as possible.  Some may be left unprocessed
-if their own dependencies are not ready, requiring another call to this
-function.  Use the get_deps_ready() function to determine whether this is the
-case.
-*/
 void ObjectFile::find_depends()
 {
 	for(TargetList::iterator i=new_deps.begin(); i!=new_deps.end();)
 	{
-		Target *tgt=*i;
+		Target *tgt = *i;
 		if(tgt->get_depends_ready())
 		{
-			i=new_deps.erase(i);
+			i = new_deps.erase(i);
 			find_depends(tgt);
 		}
 		else
 			++i;
 	}
 
-	deps_ready=new_deps.empty();
+	deps_ready = new_deps.empty();
 }
 
 
-/**
-Recursively looks for header targets and adds them as dependencies.
-*/
 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(!src)
+	SourceFile *src = dynamic_cast<SourceFile *>(tgt->get_real_target());
+	FileTarget *file = dynamic_cast<FileTarget *>(tgt);
+	if(!src || !file)
 		return;
 
-	const StringList &incpath=comp.get_build_info().incpath;
+	FS::Path spath = FS::dirname(file->get_path());
+	const StringList &incpath = comp.get_build_info().incpath;
 
-	const list<string> &includes=src->get_includes();
+	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, incpath);
-		if(hdr2 && find(depends.begin(), depends.end(), hdr2)==depends.end())
-			add_depend(hdr2);
+		Target *hdr = builder.get_header(*i, spath, incpath);
+		if(hdr && find(depends.begin(), depends.end(), hdr)==depends.end())
+			add_depend(hdr);
 	}
 }
 
-/**
-Adds a target to the dependency list as well as the new dependencies list.
-*/
 void ObjectFile::add_depend(Target *tgt)
 {
 	Target::add_depend(tgt);
@@ -92,8 +75,14 @@ Action *ObjectFile::create_action()
 	return new Compile(builder, *this);
 }
 
-string ObjectFile::generate_target_name(const Component &comp, const string &src)
+FS::Path ObjectFile::generate_target_path(const Component &comp, const string &src)
 {
-	const SourcePackage &pkg=comp.get_package();
-	return (pkg.get_temp_dir()/comp.get_name()/(splitext(basename(src)).base+".o")).str();
+	const SourcePackage &pkg = comp.get_package();
+	string fn = FS::basepart(src)+".o";
+	if(!fn.compare(0, 2, "./"))
+		fn.erase(0, 2);
+	for(string::iterator i=fn.begin(); i!=fn.end(); ++i)
+		if(*i=='/')
+			*i = '_';
+	return pkg.get_temp_dir()/comp.get_name()/fn;
 }