Merge branch 'bugfix/build_deps_assembler' into 'master'

build system: Generate dependency make rules for assembler source

Was previously only C, C++.

See merge request !304
This commit is contained in:
Angus Gratton 2016-12-15 06:48:15 +08:00
commit 953b12b4c0
2 changed files with 13 additions and 3 deletions

View file

@ -172,7 +172,7 @@ LDFLAGS ?= -nostdlib \
# CPPFLAGS used by C preprocessor
# If any flags are defined in application Makefile, add them at the end.
CPPFLAGS := -DESP_PLATFORM $(CPPFLAGS) $(EXTRA_CPPFLAGS)
CPPFLAGS := -DESP_PLATFORM -MMD -MP $(CPPFLAGS) $(EXTRA_CPPFLAGS)
# Warnings-related flags relevant both for C and C++
COMMON_WARNING_FLAGS = -Wall -Werror=all \
@ -188,8 +188,7 @@ COMMON_FLAGS = \
-ffunction-sections -fdata-sections \
-fstrict-volatile-bitfields \
-mlongcalls \
-nostdlib \
-MMD -MP
-nostdlib
# Optimization flags are set based on menuconfig choice
ifneq ("$(CONFIG_OPTIMIZATION_LEVEL_RELEASE)","")

View file

@ -155,6 +155,17 @@ function run_tests()
assert_rebuilt ${APP_BINS}
assert_not_rebuilt ${BOOTLOADER_BINS}
print_status "sdkconfig update triggers recompiles"
make
take_build_snapshot
touch sdkconfig
make
# pick one each of .c, .cpp, .S that #includes sdkconfig.h
# and therefore should rebuild
assert_rebuilt newlib/syscall_table.o
assert_rebuilt nvs_flash/src/nvs_api.o
assert_rebuilt freertos/xtensa_vectors.o
print_status "All tests completed"
if [ -n "${FAILURES}" ]; then
echo "Some failures were detected:"