Skip to content

Commit

Permalink
[#407] Add coverage flags to build-test make command
Browse files Browse the repository at this point in the history
  • Loading branch information
Donnie-Ice committed Dec 9, 2024
1 parent 48f7b18 commit 776744b
Showing 1 changed file with 1 addition and 5 deletions.
6 changes: 1 addition & 5 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -11,9 +11,6 @@ export CFS_APP_PATH = ../components
export MISSION_DEFS = ../cfg/build/
export MISSIONCONFIG = ../cfg/build/nos3

# Enable coverage flags
CFLAGS += -g -O0 --coverage
LDFLAGS += --coverage

# The "prep" step requires extra options that are specified via enviroment variables.
# Certain special ones should be passed via cache (-D) options to CMake.
Expand All @@ -32,7 +29,6 @@ ifneq ($(BUILDTYPE),)
PREP_OPTS += -DCMAKE_BUILD_TYPE=$(BUILDTYPE)
endif

PREP_OPTS += $(CFLAGS)

# The "LOCALTGTS" defines the top-level targets that are implemented in this makefile
# Any other target may also be given, in that case it will simply be passed through.
Expand Down Expand Up @@ -77,7 +73,7 @@ ifeq ($(FLIGHT_SOFTWARE), fprime)
else
mkdir -p $(FSWBUILDDIR)
cd $(FSWBUILDDIR) && cmake $(PREP_OPTS) -DENABLE_UNIT_TESTS=true ../cfe
$(MAKE) --no-print-directory -C $(FSWBUILDDIR) mission-install
$(MAKE) --no-print-directory CFLAGS="-Wall -Wextra -g --coverage" -C $(FSWBUILDDIR) mission-install
endif

checkout:
Expand Down

0 comments on commit 776744b

Please sign in to comment.