rusefi
/
rusefi-full
mirror of https://github.com/rusefi/rusefi-full.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Reenable sanitize (#4384) 

* turn on sanitize, fix overrun

* allow sanitize override, disable it for shared lib

* probably fix android build
This commit is contained in:
Matthew Kennedy 2022-07-26 23:22:22 -07:00 committed by GitHub
parent d9710a7389
commit 4643dc3c79
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
4 changed files with 11 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
.github/workflows/build-android.yaml vendored
View File

@ -24,7 +24,7 @@ jobs:
- name: Build Native Unit Tests as shared library - name: Build Native Unit Tests as shared library
working-directory: ./unit_tests/ working-directory: ./unit_tests/
# we have a unit test of JNI thus we need to build shared library # we have a unit test of JNI thus we need to build shared library
run: make -j4 run: make -j4 SANITIZE=no build/lib_rusefi_test
- name: Build Android - name: Build Android
env: env:

4
.github/workflows/build-rusEFI-console.yaml vendored
View File

@ -31,8 +31,8 @@ jobs:
- name: Build Native Unit Tests as shared library - name: Build Native Unit Tests as shared library
working-directory: ./unit_tests/ working-directory: ./unit_tests/
# we have a unit test of JNI thus we need to build shared library # we have a unit test of JNI thus we need to build shared library
run: make -j4 run: make -j4 SANITIZE=no build/lib_rusefi_test
- name: Test console - name: Test console
# at the moment 'jar' task does not depend on tests?! maybe because tests take some time? # at the moment 'jar' task does not depend on tests?! maybe because tests take some time?

2
unit_tests/rules.mk
View File

@ -89,7 +89,7 @@ VPATH = $(SRCPATHS)
# Makefile rules # Makefile rules
# #
all: $(OBJS) $(BINARY_OUTPUT) $(SHARED_OUTPUT) MAKE_ALL_RULE_HOOK all: $(OBJS) $(BINARY_OUTPUT) MAKE_ALL_RULE_HOOK
MAKE_ALL_RULE_HOOK: MAKE_ALL_RULE_HOOK:

12
unit_tests/unit_test_rules.mk
View File

@ -14,11 +14,13 @@ PCHSUB = unit_tests
include $(PROJECT_DIR)/rusefi_rules.mk include $(PROJECT_DIR)/rusefi_rules.mk
ifneq ($(OS),Windows_NT) # User may want to pass in a forced value for SANITIZE
# at the moment lib asan breaks JNI library ifeq ($(SANITIZE),)
SANITIZE = no ifneq ($(OS),Windows_NT)
else SANITIZE = yes
SANITIZE = no else
SANITIZE = no
endif
endif endif
IS_MAC = no IS_MAC = no