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

---
 CMakeLists.txt                              | 8 ++++----
 fsw/{src => inc}/hs_events.h                | 0
 fsw/{src => inc}/hs_msg.h                   | 0
 fsw/{src => inc}/hs_msgdefs.h               | 0
 fsw/{platform_inc => inc}/hs_msgids.h       | 0
 fsw/{mission_inc => inc}/hs_perfids.h       | 0
 fsw/{platform_inc => inc}/hs_platform_cfg.h | 0
 fsw/{src => inc}/hs_tbldefs.h               | 0
 unit-test/CMakeLists.txt                    | 4 +++-
 9 files changed, 7 insertions(+), 5 deletions(-)
 rename fsw/{src => inc}/hs_events.h (100%)
 rename fsw/{src => inc}/hs_msg.h (100%)
 rename fsw/{src => inc}/hs_msgdefs.h (100%)
 rename fsw/{platform_inc => inc}/hs_msgids.h (100%)
 rename fsw/{mission_inc => inc}/hs_perfids.h (100%)
 rename fsw/{platform_inc => inc}/hs_platform_cfg.h (100%)
 rename fsw/{src => inc}/hs_tbldefs.h (100%)

diff --git a/CMakeLists.txt b/CMakeLists.txt
index c186dc7..8f141df 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -1,9 +1,5 @@
 project(CFS_HS C)
 
-include_directories(fsw/src)
-include_directories(fsw/mission_inc)
-include_directories(fsw/platform_inc)
-
 set(APP_SRC_FILES
   fsw/src/hs_monitors.c
   fsw/src/hs_utils.c
@@ -15,6 +11,10 @@ set(APP_SRC_FILES
 # Create the app module
 add_cfe_app(hs ${APP_SRC_FILES})
 
+# This permits direct access to public headers in the fsw/inc and fsw/src directories
+target_include_directories(hs PUBLIC fsw/inc)
+target_include_directories(hs PUBLIC fsw/src)
+
 set(APP_TABLE_FILES
   fsw/tables/hs_mat.c
   fsw/tables/hs_xct.c
diff --git a/fsw/src/hs_events.h b/fsw/inc/hs_events.h
similarity index 100%
rename from fsw/src/hs_events.h
rename to fsw/inc/hs_events.h
diff --git a/fsw/src/hs_msg.h b/fsw/inc/hs_msg.h
similarity index 100%
rename from fsw/src/hs_msg.h
rename to fsw/inc/hs_msg.h
diff --git a/fsw/src/hs_msgdefs.h b/fsw/inc/hs_msgdefs.h
similarity index 100%
rename from fsw/src/hs_msgdefs.h
rename to fsw/inc/hs_msgdefs.h
diff --git a/fsw/platform_inc/hs_msgids.h b/fsw/inc/hs_msgids.h
similarity index 100%
rename from fsw/platform_inc/hs_msgids.h
rename to fsw/inc/hs_msgids.h
diff --git a/fsw/mission_inc/hs_perfids.h b/fsw/inc/hs_perfids.h
similarity index 100%
rename from fsw/mission_inc/hs_perfids.h
rename to fsw/inc/hs_perfids.h
diff --git a/fsw/platform_inc/hs_platform_cfg.h b/fsw/inc/hs_platform_cfg.h
similarity index 100%
rename from fsw/platform_inc/hs_platform_cfg.h
rename to fsw/inc/hs_platform_cfg.h
diff --git a/fsw/src/hs_tbldefs.h b/fsw/inc/hs_tbldefs.h
similarity index 100%
rename from fsw/src/hs_tbldefs.h
rename to fsw/inc/hs_tbldefs.h
diff --git a/unit-test/CMakeLists.txt b/unit-test/CMakeLists.txt
index 15408fe..8ac80b2 100644
--- a/unit-test/CMakeLists.txt
+++ b/unit-test/CMakeLists.txt
@@ -19,8 +19,10 @@ add_cfe_coverage_stubs("hs_internal"
 # Link with the cfe core stubs and unit test assert libs
 target_link_libraries(coverage-hs_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-hs_internal-stubs PUBLIC utilities)
+target_include_directories(coverage-hs_internal-stubs PUBLIC ../fsw/inc)
+target_include_directories(coverage-hs_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