commit 9930137b234144a244e953cb1c2aae7d51bd67a3
author Lei YU <mine260309@gmail.com> Sun Sep 29 16:27:12 2019 +0800
committer Lei YU <mine260309@gmail.com> Wed Oct 23 06:53:42 2019 +0000
tree c8282b08d8ac4ddebb8b21633ea02aacfe767b81
parent a5c47bb3fd28c9178f3573fc49d6db60c43a0480

Refactor: Make Activation inherit FilePath interface

Previously Version inherits FilePath interface to provide the
information of the file path property of a software, and Activation was
using pre-defined location to find the image path (IMG_DIR / versionId).

Now the code is going to support pre-built image and stored image during
update, the Activation class needs to know the file path to perform the
update.

So this commit "moves" the FilePath from Version to Activation.

Tested: Verify the "Path" property on FilePath still exists on the DBus
        object after uploading a tarball.

Signed-off-by: Lei YU <mine260309@gmail.com>
Change-Id: I37b41eaa93cd239ab7732f6fac9400410995ca71

src/version.hpp

diff --git a/src/version.hpp b/src/version.hpp
index d4106e6..89db260 100644
--- a/src/version.hpp
+++ b/src/version.hpp
@@ -3,7 +3,6 @@
 #include "config.h"
 
 #include <sdbusplus/bus.hpp>
-#include <xyz/openbmc_project/Common/FilePath/server.hpp>
 #include <xyz/openbmc_project/Object/Delete/server.hpp>
 #include <xyz/openbmc_project/Software/Version/server.hpp>
 
@@ -17,8 +16,7 @@
 using eraseFunc = std::function<void(std::string)>;
 
 using VersionInherit = sdbusplus::server::object::object<
-    sdbusplus::xyz::openbmc_project::Software::server::Version,
-    sdbusplus::xyz::openbmc_project::Common::server::FilePath>;
+    sdbusplus::xyz::openbmc_project::Software::server::Version>;
 using DeleteInherit = sdbusplus::server::object::object<
     sdbusplus::xyz::openbmc_project::Object::server::Delete>;
 
@@ -86,13 +84,11 @@
      * @param[in] versionId      - The version Id
      * @param[in] versionString  - The version string
      * @param[in] versionPurpose - The version purpose
-     * @param[in] filePath       - The image filesystem path
      * @param[in] callback       - The eraseFunc callback
      */
     Version(sdbusplus::bus::bus& bus, const std::string& objPath,
             const std::string& versionId, const std::string& versionString,
-            VersionPurpose versionPurpose, const std::string& filePath,
-            eraseFunc callback) :
+            VersionPurpose versionPurpose, eraseFunc callback) :
         VersionInherit(bus, (objPath).c_str(), true),
         eraseCallback(callback), bus(bus), objPath(objPath),
         versionId(versionId), versionStr(versionString)
@@ -100,7 +96,6 @@
         // Set properties.
         purpose(versionPurpose);
         version(versionString);
-        path(filePath);
 
         deleteObject = std::make_unique<Delete>(bus, objPath, *this);