Installing only makes sense for file targets

diff --git a/source/filetarget.h b/source/filetarget.h
index 6d2480f7e75d9d64c795a4e075c2492664b32aab..2be0063c936f20bb5670a5ce16e02425e6e1d0d1 100644 (file)
--- 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 885ef4b567f00d69d9f04921cd5fca9c46c06568..8a394f3717625787dbc1c123b7bd4f61319da8da 100644 (file)
--- 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; }