Skip to content

Commit

Permalink
Merge #19567
Browse files Browse the repository at this point in the history
19567: tests: move build system applications to tests/build_system r=aabadie a=aabadie



Co-authored-by: Alexandre Abadie <alexandre.abadie@inria.fr>
  • Loading branch information
bors[bot] and aabadie authored May 11, 2023
2 parents d362a8d + b8001f4 commit 4af0376
Show file tree
Hide file tree
Showing 164 changed files with 19 additions and 16 deletions.
2 changes: 1 addition & 1 deletion dist/tools/buildsystem_sanity_check/check.sh
Original file line number Diff line number Diff line change
Expand Up @@ -261,7 +261,7 @@ checks_tests_application_not_defined_in_makefile() {
patterns+=(-e '^[[:space:]]*APPLICATION[[:space:]:+]=')

pathspec+=('tests/**/Makefile')
pathspec+=(':!tests/external_board_native/Makefile')
pathspec+=(':!tests/build_system/external_board_native/Makefile')

git -C "${RIOTBASE}" grep -n "${patterns[@]}" -- "${pathspec[@]}" \
| error_with_message "Don't define APPLICATION in test Makefile"
Expand Down
1 change: 1 addition & 0 deletions makefiles/app_dirs.inc.mk
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@ APPLICATION_DIRS := \
examples \
tests \
tests/bench \
tests/build_system \
tests/core \
tests/drivers \
tests/periph \
Expand Down
2 changes: 2 additions & 0 deletions tests/build_system/Makefile.build_system_common
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
RIOTBASE ?= $(CURDIR)/../../..
include $(CURDIR)/../../Makefile.tests_common
2 changes: 1 addition & 1 deletion tests/blob/Makefile → tests/build_system/blob/Makefile
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
include ../Makefile.tests_common
include ../Makefile.build_system_common

USEMODULE += fmt

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.
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
include ../Makefile.tests_common
include ../Makefile.build_system_common
CFLAGS += -DSUPER_STRING='"I love sentences with spaces"'

include $(RIOTBASE)/Makefile.include
Expand Down
File renamed without changes.
File renamed without changes.
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
BOARD ?= samr21-xpro
include ../Makefile.tests_common
include ../Makefile.build_system_common


# Normally all boards using `cortexm_common/ldscripts/cortexm.ld` linkerscript
Expand Down
File renamed without changes.
File renamed without changes.
File renamed without changes.
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
include ../Makefile.tests_common
include ../Makefile.build_system_common

FEATURES_REQUIRED += cpp libstdcpp

Expand Down
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
include ../Makefile.tests_common
include ../Makefile.build_system_common

FEATURES_REQUIRED += cpp libstdcpp

Expand Down
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
include ../Makefile.tests_common
include ../Makefile.build_system_common

# Require arch_native feature so this is not compiled for other boards in
# $(RIOTBOARD)/
Expand Down
File renamed without changes.
File renamed without changes.
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
APPLICATION = external_board
RIOTBASE ?= $(CURDIR)/../../
RIOTBASE ?= $(CURDIR)/../../../

# Only support this board
# No need for a `WHITELIST` as there is only one board in `external_boards`.
Expand Down
File renamed without changes.
File renamed without changes.
File renamed without changes.
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
include ../Makefile.tests_common
include ../Makefile.build_system_common
USEMODULE += random

USEMODULE += external_module
Expand Down
File renamed without changes.
File renamed without changes.
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
include ../Makefile.tests_common
include ../Makefile.build_system_common

EXTERNAL_PKG_DIRS += external_pkgs
USEPKG += external_pkg
Expand Down
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
include ../Makefile.tests_common
include ../Makefile.build_system_common

USEMODULE += external_module_1
USEMODULE += external_module_2
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.
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
include ../Makefile.tests_common
include ../Makefile.build_system_common

all: checks

Expand Down
File renamed without changes.
File renamed without changes.
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
DEVELHELP = 0
include ../Makefile.tests_common
include ../Makefile.build_system_common

USEMODULE += shell

Expand Down
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
BOARD_WHITELIST = native

include ../Makefile.tests_common
include ../Makefile.build_system_common
include $(RIOTBASE)/Makefile.include


Expand Down
File renamed without changes.
File renamed without changes.
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
BOARD ?= stm32f4discovery
include ../Makefile.tests_common
include ../Makefile.build_system_common

# The stm32f4discovery is the only board that provides known conflicting features,
# so using this compile test on other boards will not provide the expected warning.
Expand Down
File renamed without changes.
File renamed without changes.
File renamed without changes.
2 changes: 1 addition & 1 deletion tests/xfa/Makefile → tests/build_system/xfa/Makefile
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
include ../Makefile.tests_common
include ../Makefile.build_system_common

include $(RIOTBASE)/Makefile.include

Expand Down
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.

0 comments on commit 4af0376

Please sign in to comment.