From 8bd024089667359b043afbf3bfcf9c82f034fd5d Mon Sep 17 00:00:00 2001
From: Justin Figueroa <justin.d.figueroa@nasa.gov>
Date: Tue, 6 Dec 2022 11:42:22 -0500
Subject: [PATCH] Fix #57, Moves interface definition files to inc

---
 CMakeLists.txt                              | 7 +++----
 fsw/{src => inc}/mm_events.h                | 0
 fsw/{mission_inc => inc}/mm_mission_cfg.h   | 0
 fsw/{src => inc}/mm_msg.h                   | 0
 fsw/{src => inc}/mm_msgdefs.h               | 0
 fsw/{platform_inc => inc}/mm_msgids.h       | 0
 fsw/{mission_inc => inc}/mm_perfids.h       | 0
 fsw/{platform_inc => inc}/mm_platform_cfg.h | 0
 unit-test/CMakeLists.txt                    | 4 +++-
 9 files changed, 6 insertions(+), 5 deletions(-)
 rename fsw/{src => inc}/mm_events.h (100%)
 rename fsw/{mission_inc => inc}/mm_mission_cfg.h (100%)
 rename fsw/{src => inc}/mm_msg.h (100%)
 rename fsw/{src => inc}/mm_msgdefs.h (100%)
 rename fsw/{platform_inc => inc}/mm_msgids.h (100%)
 rename fsw/{mission_inc => inc}/mm_perfids.h (100%)
 rename fsw/{platform_inc => inc}/mm_platform_cfg.h (100%)

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 79ef04b..8a01cbb 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -1,9 +1,5 @@
 project(CFS_MM C)
 
-include_directories(fsw/src)
-include_directories(fsw/mission_inc)
-include_directories(fsw/platform_inc)
-
 set(APP_SRC_FILES
   fsw/src/mm_mem32.c
   fsw/src/mm_app.c
@@ -17,6 +13,9 @@ set(APP_SRC_FILES
 # Create the app module
 add_cfe_app(mm ${APP_SRC_FILES})
 
+# This permits direct access to public headers in the fsw/inc directory
+target_include_directories(mm PUBLIC fsw/inc)
+
 # If UT is enabled, then add the tests from the subdirectory
 # Note that this is an app, and therefore does not provide
 # stub functions, as other entities would not typically make
diff --git a/fsw/src/mm_events.h b/fsw/inc/mm_events.h
similarity index 100%
rename from fsw/src/mm_events.h
rename to fsw/inc/mm_events.h
diff --git a/fsw/mission_inc/mm_mission_cfg.h b/fsw/inc/mm_mission_cfg.h
similarity index 100%
rename from fsw/mission_inc/mm_mission_cfg.h
rename to fsw/inc/mm_mission_cfg.h
diff --git a/fsw/src/mm_msg.h b/fsw/inc/mm_msg.h
similarity index 100%
rename from fsw/src/mm_msg.h
rename to fsw/inc/mm_msg.h
diff --git a/fsw/src/mm_msgdefs.h b/fsw/inc/mm_msgdefs.h
similarity index 100%
rename from fsw/src/mm_msgdefs.h
rename to fsw/inc/mm_msgdefs.h
diff --git a/fsw/platform_inc/mm_msgids.h b/fsw/inc/mm_msgids.h
similarity index 100%
rename from fsw/platform_inc/mm_msgids.h
rename to fsw/inc/mm_msgids.h
diff --git a/fsw/mission_inc/mm_perfids.h b/fsw/inc/mm_perfids.h
similarity index 100%
rename from fsw/mission_inc/mm_perfids.h
rename to fsw/inc/mm_perfids.h
diff --git a/fsw/platform_inc/mm_platform_cfg.h b/fsw/inc/mm_platform_cfg.h
similarity index 100%
rename from fsw/platform_inc/mm_platform_cfg.h
rename to fsw/inc/mm_platform_cfg.h
diff --git a/unit-test/CMakeLists.txt b/unit-test/CMakeLists.txt
index 2991065..954e092 100644
--- a/unit-test/CMakeLists.txt
+++ b/unit-test/CMakeLists.txt
@@ -21,8 +21,10 @@ add_cfe_coverage_stubs("mm_internal"
 # Link with the cfe core stubs and unit test assert libs
 target_link_libraries(coverage-mm_internal-stubs ut_core_api_stubs ut_assert)
 
-# Include and expose unit test utilities includes
+# Include and expose unit test utilities, fsw/inc, and fsw/src includes
 target_include_directories(coverage-mm_internal-stubs PUBLIC utilities)
+target_include_directories(coverage-mm_internal-stubs PUBLIC ../fsw/inc)
+target_include_directories(coverage-mm_internal-stubs PUBLIC ../fsw/src)
 
 # Generate a dedicated "testrunner" executable for each test file
 # Accomplish this by cycling through all the app's source files, there must be