From: Mikko Rasa <tdb@tdb.fi>
Date: Tue, 10 Jul 2012 09:25:52 +0000 (+0300)
Subject: Installing only makes sense for file targets
X-Git-Url: http://git.tdb.fi/?a=commitdiff_plain;h=1a75d7e01d0704f83633f696a312744de272b4c4;p=builder.git

Installing only makes sense for file targets
---

diff --git a/source/filetarget.h b/source/filetarget.h
index 6d2480f..2be0063 100644
--- a/source/filetarget.h
+++ b/source/filetarget.h
@@ -14,6 +14,7 @@ protected:
 	Msp::FS::Path path;
 	Msp::Time::TimeStamp mtime;
 	unsigned size;
+	Msp::FS::Path install_location;
 
 	FileTarget(Builder &, const Msp::FS::Path &);
 	FileTarget(Builder &, const SourcePackage &, const Msp::FS::Path &);
@@ -26,6 +27,9 @@ public:
 	const Msp::Time::TimeStamp &get_mtime() const { return mtime; }
 	unsigned get_size() const { return size; }
 
+	bool is_installable() const { return !install_location.empty(); }
+	const Msp::FS::Path &get_install_location() const { return install_location; }
+
 	/**
 	Changes the mtime of the target to the current time.
 	*/
diff --git a/source/target.h b/source/target.h
index 885ef4b..8a394f3 100644
--- a/source/target.h
+++ b/source/target.h
@@ -45,7 +45,6 @@ protected:
 	const Tool *tool;
 	State state;
 	std::string rebuild_reason;
-	Msp::FS::Path install_location;
 
 	Dependencies depends;
 
@@ -87,8 +86,6 @@ public:
 	/** Forces rebuild of the target. */
 	void force_rebuild();
 
-	bool is_installable() const { return !install_location.empty(); }
-	const Msp::FS::Path &get_install_location() const { return install_location; }
 	void add_depend(Target *);
 	const Dependencies &get_depends() const { return depends; }