X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;f=source%2Ffiletarget.cpp;h=b42580a099a0a95eb411bc34fa65345cd72a2b03;hb=766dad2d4f7ca73dc182f22e306ba0def6fd8722;hp=7de9b797761717ce968649931e29ea6a32b8920c;hpb=c8dfe708b5551818dad3f6cf3e86cc72fe684572;p=builder.git

diff --git a/source/filetarget.cpp b/source/filetarget.cpp
index 7de9b79..b42580a 100644
--- a/source/filetarget.cpp
+++ b/source/filetarget.cpp
@@ -1,10 +1,13 @@
 #include <msp/fs/stat.h>
 #include <msp/fs/utils.h>
 #include <msp/strings/format.h>
+#include <msp/strings/utils.h>
 #include <msp/time/utils.h>
 #include "builder.h"
 #include "filetarget.h"
 #include "sourcepackage.h"
+#include "task.h"
+#include "tool.h"
 
 using namespace std;
 using namespace Msp;
@@ -52,7 +55,6 @@ string FileTarget::generate_name(Builder &builder, const SourcePackage *pkg, con
 	else if(FS::descendant_depth(path, builder.get_prefix())>=0)
 	{
 		FS::Path relpath = FS::relative(path, builder.get_prefix());
-		builder.get_logger().log("debug", format("%s %s %s", path, builder.get_prefix(), relpath));
 		return "<prefix>"+relpath.str().substr(1);
 	}
 
@@ -62,6 +64,7 @@ string FileTarget::generate_name(Builder &builder, const SourcePackage *pkg, con
 void FileTarget::touch()
 {
 	mtime = Time::now();
+	modified();
 	signal_bubble_rebuild.emit();
 }
 
@@ -84,22 +87,73 @@ void FileTarget::check_rebuild()
 		}
 	}
 
-	if(!needs_rebuild() && package && package->get_config().get_mtime()>mtime)
-		mark_rebuild("Package options changed");
+	if(!needs_rebuild())
+	{
+		for(Dependencies::iterator i=side_effects.begin(); (i!=side_effects.end() && !needs_rebuild()); ++i)
+		{
+			FileTarget *ft = dynamic_cast<FileTarget *>(*i);
+			if(ft && !ft->get_mtime())
+				mark_rebuild((*i)->get_name()+" does not exist");
+		}
+	}
+
+	if(!needs_rebuild() && package)
+	{
+		if(package->get_config().get_mtime()>mtime)
+			mark_rebuild("Package options changed");
+
+		if(tool->get_executable())
+		{
+			string build_sig = create_build_signature();
+			if(package->get_cache().has_key(this, "build_sig"))
+			{
+				if(package->get_cache().get_value(this, "build_sig")!=build_sig)
+					mark_rebuild("Build signature changed");
+			}
+		}
+	}
 }
 
-Task *FileTarget::build()
+string FileTarget::create_build_signature() const
 {
-	if(tool && !builder.get_dry_run() && mtime)
-		FS::unlink(path);
+	if(!package)
+		return string();
 
-	return Target::build();
+	const BuildInfo &binfo = (component ? component->get_build_info() : package->get_build_info());
+	return tool->create_build_signature(binfo);
+}
+
+Task *FileTarget::build()
+{
+	Task *task = Target::build();
+	task->set_file(path);
+	task->set_unlink(true);
+	return task;
 }
 
 void FileTarget::build_finished(bool success)
 {
 	if(success)
+	{
 		stat();
+		if(package)
+		{
+			string build_sig = create_build_signature();
+			if(!build_sig.empty())
+				package->get_cache().set_value(this, "build_sig", build_sig);
+		}
+	}
 
 	Target::build_finished(success);
 }
+
+void FileTarget::clean()
+{
+	if(mtime)
+	{
+		FS::unlink(path);
+		mtime = Time::TimeStamp();
+		size = 0;
+		check_rebuild();
+	}
+}