From 1a75d7e01d0704f83633f696a312744de272b4c4 Mon Sep 17 00:00:00 2001 From: Mikko Rasa Date: Tue, 10 Jul 2012 12:25:52 +0300 Subject: [PATCH] Installing only makes sense for file targets --- source/filetarget.h | 4 ++++ source/target.h | 3 --- 2 files changed, 4 insertions(+), 3 deletions(-) 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; } -- 2.45.2