Skip to content

Commit

Permalink
Fix #57, Moves interface definition files to inc
Browse files Browse the repository at this point in the history
  • Loading branch information
chillfig committed Dec 22, 2022
1 parent 16c93f4 commit 8bd0240
Show file tree
Hide file tree
Showing 9 changed files with 6 additions and 5 deletions.
7 changes: 3 additions & 4 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -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
Expand All @@ -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
Expand Down
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
4 changes: 3 additions & 1 deletion unit-test/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit 8bd0240

Please sign in to comment.