stack: Refactor stack into its own library

Move nop controller and simulator plugin to models where all
the other Nop classes are.

Author: cassava

CMakeLists.txt

@@ -32,6 +32,7 @@ add_subdirectory(runtime)
 add_subdirectory(models)
 add_subdirectory(osi)
 add_subdirectory(oak)
+add_subdirectory(stack)
 add_subdirectory(engine)
 add_subdirectory(plugins)
 

Makefile.all

@@ -50,6 +50,7 @@ ALL_PKGS := \
 	fable \
 	runtime \
 	models \
+	stack \
 	osi \
 	oak \
 	engine \
@@ -77,7 +78,8 @@ runtime: fable
 models: runtime
 osi: runtime models
 oak: runtime
-engine: models oak
+stack: runtime models
+engine: models oak stack
 $(PLUGIN_PKGS): runtime models
 plugins/esmini: osi
 

engine/CMakeLists.txt

@@ -7,6 +7,7 @@ if(CLOE_FIND_PACKAGES)
     find_package(fable REQUIRED QUIET)
     find_package(cloe-runtime REQUIRED QUIET)
     find_package(cloe-models REQUIRED QUIET)
+    find_package(cloe-stack REQUIRED QUIET)
 endif()
 find_package(Boost REQUIRED QUIET)
 find_package(CLI11 REQUIRED QUIET)
@@ -22,67 +23,6 @@ string(TIMESTAMP CLOE_ENGINE_TIMESTAMP "%Y-%m-%d")
 set(CLOE_ENGINE_VERSION ${CLOE_PROJECT_VERSION})
 set(PROJECT_GIT_REF "unknown")
 
-# Library libstack ---------------------------------------------------
-message(STATUS "Building cloe-stacklib library.")
-add_library(cloe-stacklib STATIC
-    src/config.hpp
-    src/stack.hpp
-    src/stack.cpp
-    src/stack_factory.hpp
-    src/stack_factory.cpp
-    src/plugin.hpp
-    src/plugin.cpp
-
-    # Built-in plugins:
-    src/plugins/nop_controller.cpp
-    src/plugins/nop_controller.hpp
-    src/plugins/nop_simulator.cpp
-    src/plugins/nop_simulator.hpp
-)
-add_library(cloe::stacklib ALIAS cloe-stacklib)
-set_target_properties(cloe-stacklib PROPERTIES
-    CXX_STANDARD 17
-    CXX_STANDARD_REQUIRED ON
-    OUTPUT_NAME stack
-)
-target_include_directories(cloe-stacklib
-  PRIVATE
-    src
-)
-target_link_libraries(cloe-stacklib
-  PUBLIC
-    cloe::runtime
-    cloe::models
-    fable::fable
-    Boost::headers
-    Threads::Threads
-    ${CMAKE_DL_LIBS}
-)
-
-include(CTest)
-if(BUILD_TESTING)
-    find_package(GTest REQUIRED QUIET)
-    include(GoogleTest)
-
-    message(STATUS "Building test-stacklib executable.")
-    add_executable(test-stacklib
-        src/stack_test.cpp
-        src/stack_component_test.cpp
-    )
-    set_target_properties(test-stacklib PROPERTIES
-        CXX_STANDARD 17
-        CXX_STANDARD_REQUIRED ON
-    )
-    target_link_libraries(test-stacklib
-        GTest::gtest
-        GTest::gtest_main
-        Boost::boost
-        cloe::models
-        cloe::stacklib
-    )
-    gtest_add_tests(TARGET test-stacklib)
-endif()
-
 # Library libengine ----------------------------------------------
 message(STATUS "Building cloe-enginelib library.")
 add_library(cloe-enginelib STATIC
@@ -157,7 +97,7 @@ target_include_directories(cloe-enginelib
 )
 target_link_libraries(cloe-enginelib
   PUBLIC
-    cloe::stacklib
+    cloe::stack
     cloe::models
     cloe::runtime
     fable::fable
@@ -189,8 +129,11 @@ else()
   target_compile_definitions(cloe-enginelib PUBLIC CLOE_ENGINE_WITH_LRDB=0)
 endif()
 
+include(CTest)
 if(BUILD_TESTING)
     message(STATUS "Building test-enginelib executable.")
+    find_package(GTest REQUIRED QUIET)
+    include(GoogleTest)
     add_executable(test-enginelib
         src/lua_stack_test.cpp
         src/lua_setup_test.cpp
@@ -204,11 +147,12 @@ if(BUILD_TESTING)
         CXX_STANDARD_REQUIRED ON
     )
     target_link_libraries(test-enginelib
+      PRIVATE
         GTest::gtest
         GTest::gtest_main
         Boost::boost
         cloe::models
-        cloe::stacklib
+        cloe::stack
         cloe::enginelib
     )
     gtest_add_tests(TARGET test-enginelib)
@@ -245,7 +189,7 @@ target_include_directories(cloe-engine
 )
 target_link_libraries(cloe-engine
   PRIVATE
-    cloe::stacklib
+    cloe::stack
     cloe::enginelib
     CLI11::CLI11
     linenoise::linenoise

engine/conanfile.py

@@ -57,6 +57,7 @@ def set_version(self):
     def requirements(self):
         self.requires(f"cloe-runtime/{self.version}@cloe/develop")
         self.requires(f"cloe-models/{self.version}@cloe/develop")
+        self.requires(f"cloe-stack/{self.version}@cloe/develop")
         self.requires("cli11/2.3.2", private=True)
         self.requires("sol2/3.3.1")
         if self.options.server:

engine/src/lua_setup.cpp

@@ -28,12 +28,12 @@
 
 #include <cloe/utility/std_extensions.hpp>  // for split_string
 
+#include <cloe/stack.hpp>
 #include <fable/utility/sol.hpp>     // for Json(sol::object)
 #include <fable/utility/string.hpp>  // for join_vector
 
 #include "error_handler.hpp"  // for format_cloe_error
 #include "lua_api.hpp"
-#include "stack.hpp"
 #include "utility/command.hpp"  // for CommandExecuter, CommandResult
 
 // This variable is set from CMakeLists.txt, but in case it isn't,

engine/src/lua_setup_stack.cpp

@@ -17,10 +17,10 @@
  */
 
 #include <fable/utility/sol.hpp>
+#include <cloe/stack.hpp>
 
 #include "lua_api.hpp"
 #include "lua_setup.hpp"
-#include "stack.hpp"
 
 namespace cloe {
 

engine/src/lua_setup_test.cpp

@@ -23,11 +23,11 @@
 #include <fstream>
 #include <string_view>
 
+#include <cloe/stack.hpp>  // for Stack
 #include <sol/state.hpp>
 
 #include "lua_api.hpp"    // for lua_safe_script_file
 #include "lua_setup.hpp"  // for setup_lua
-#include "stack.hpp"      // for Stack
 using namespace cloe;     // NOLINT(build/namespaces)
 
 #ifndef CLOE_LUA_PATH
@@ -87,7 +87,8 @@ TEST_F(cloe_lua_setup, cloe_engine_is_available) {
 
 TEST_F(cloe_lua_setup, describe_cloe) {
   setup_lua(lua, opt, stack);
-  ASSERT_EQ(lua.script("local cloe = require('cloe'); return cloe.inspect(cloe.LogLevel.CRITICAL)").get<std::string>(),
+  ASSERT_EQ(lua.script("local cloe = require('cloe'); return cloe.inspect(cloe.LogLevel.CRITICAL)")
+                .get<std::string>(),
             std::string("\"critical\""));
 }
 

engine/src/lua_stack_test.cpp

@@ -22,13 +22,13 @@
 
 #include <sol/state.hpp>
 
-#include <cloe/core.hpp>  // for Json
+#include <cloe/core.hpp>   // for Json
+#include <cloe/stack.hpp>  // for Stack
 #include <fable/utility.hpp>
 #include <fable/utility/gtest.hpp>  // for assert_from_conf
 #include <fable/utility/sol.hpp>
 
 #include "lua_setup.hpp"
-#include "stack.hpp"   // for Stack
 using namespace cloe;  // NOLINT(build/namespaces)
 
 TEST(cloe_lua_stack, deserialize_vehicle_conf) {

engine/src/main.cpp

@@ -24,8 +24,8 @@
 
 #include <cloe/core/error.hpp>
 #include <cloe/core/logger.hpp>
+#include <cloe/stack_config.hpp>
 
-#include "config.hpp"
 #include "main_commands.hpp"
 
 int main(int argc, char** argv) {

engine/src/main_check.cpp

@@ -21,9 +21,9 @@
 #include <vector>    // for vector<>
 
 #include <cloe/core/fable.hpp>
+#include <cloe/stack.hpp>
 
 #include "main_commands.hpp"
-#include "stack.hpp"
 
 namespace engine {