Drop dependency on source repository hash

target_factory uses a hash of the top commit of the repository as
library id. As this id depends on the repository / branch it is not
stable; a version number should be used to represent library API
version if required.

As the library id is only used in the test, drop this function from
the library interface.

Gbp-Pq: Name 0001-Drop-dependency-on-source-repository-hash.patch

diff --git a/include/vitis/ai/target_factory.hpp b/include/vitis/ai/target_factory.hpp
index 60ad4e67d03ae22b9528d997b48f3123298d9a83..24918ac00f26c73e71dddaa0d433f2d88801d0ea 100644 (file)
--- a/include/vitis/ai/target_factory.hpp
+++ b/include/vitis/ai/target_factory.hpp
@@ -40,7 +40,6 @@ class TargetFactory {
   virtual void dump(const Target& target, const std::string& file) const = 0;
 
   static const std::string get_lib_name();
-  static const std::string get_lib_id();
 
  public:
   virtual ~TargetFactory() = default;

diff --git a/src/target_factory.cpp b/src/target_factory.cpp
index 5f3312321e112cdcf021ce1d8f36cb1ae0574913..621008c439fbb4a44f2a33904c5ecb3174edaefa 100644 (file)
--- a/src/target_factory.cpp
+++ b/src/target_factory.cpp
@@ -227,10 +227,5 @@ const std::string TargetFactory::get_lib_name() {
   return ret;
 }
 
-const std::string TargetFactory::get_lib_id() {
-  const auto ret = std::string{PROJECT_GIT_COMMIT_ID};
-  return ret;
-}
-
 }  // namespace ai
 }  // namespace vitis

diff --git a/test/test_lib_info.cpp b/test/test_lib_info.cpp
index 8b78591b34642fad2fd52e93226dfa195cea9cd3..2802fe24658203e1bea8dea93a0a00e970dd83e6 100644 (file)
--- a/test/test_lib_info.cpp
+++ b/test/test_lib_info.cpp
@@ -24,7 +24,6 @@ int main(int, char* argv[]) {
                   UNI_LOG_STD_LEVEL_INFO);
 
   UNI_LOG_INFO << "lib_name = " << TargetFactory::get_lib_name();
-  UNI_LOG_INFO << "lib_id = " << TargetFactory::get_lib_id();
 
   return 0;
 }