X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;f=source%2Ftarget.cpp;h=c9c23ad9116165297d602978b9be04c13f902530;hb=555190f7eafcf3a67750fc63872b23e17371aa98;hp=1f4ace9d92c1d7b79122129a99b9781814e0f0d5;hpb=74266a6e650f019063cdcd1c9a7bd26d8f99041b;p=builder.git

diff --git a/source/target.cpp b/source/target.cpp
index 1f4ace9..c9c23ad 100644
--- a/source/target.cpp
+++ b/source/target.cpp
@@ -1,3 +1,10 @@
+/* $Id$
+
+This file is part of builder
+Copyright © 2006-2007 Mikko Rasa, Mikkosoft Productions
+Distributed under the LGPL
+*/
+
 #include <msp/path/utils.h>
 #include <msp/time/utils.h>
 #include "action.h"
@@ -15,7 +22,10 @@ ready to be built (maybe because they are being built right now), returns 0.
 */
 Target *Target::get_buildable_target()
 {
-	bool self_ok=true;
+	if(!rebuild)
+		return 0;
+
+	bool self_ok=!building;
 	for(TargetList::iterator i=depends.begin(); i!=depends.end(); ++i)
 	{
 		Target *tgt=(*i)->get_buildable_target();
@@ -25,7 +35,7 @@ Target *Target::get_buildable_target()
 			self_ok=false;
 	}
 
-	if(self_ok && rebuild && !building)
+	if(self_ok)
 		return this;
 
 	return 0;
@@ -33,6 +43,8 @@ Target *Target::get_buildable_target()
 
 void Target::add_depend(Target *dep)
 {
+	if(dep==this)
+		throw InvalidParameterValue("A target can't depend on itself");
 	depends.push_back(dep);
 	dep->rdepends.push_back(this);
 }
@@ -90,6 +102,8 @@ Target::Target(Builder &b, const Package *p, const string &n):
 	prepared(false),
 	counted(false)
 {
+	builder.add_target(this);
+
 	struct stat st;
 	if(!Path::stat(name, st))
 		mtime=Time::TimeStamp::from_unixtime(st.st_mtime);