fuchsia-china/third_party.pigweed.src
1
0
Fork 0
mirror of https://fuchsia.googlesource.com/third_party/pigweed.googlesource.com/pigweed/pigweed synced 2024-08-02 06:46:04 +00:00
Code Issues Actions 4 Packages Projects Releases Wiki Activity

bazel: Remove testonly_freertos platform

Browse Source 
This platform is obsolete. We originally added it because we had no
"real platform" (corresponding to an actual physical device) in upstream
that used FreeRTOS (see http://pwrev.dev/130956). But today
`--config=stm32f429i_freertos` is that platform, and we perform a
wildcard build for it as part of presubmit.

Change-Id: Ifaa9fab6f0e929b690b314198818c1bb79850d22
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/216181
Lint: Lint 🤖 <android-build-ayeaye@system.gserviceaccount.com>
Pigweed-Auto-Submit: Ted Pudlik <tpudlik@google.com>
Reviewed-by: Taylor Cramer <cramertj@google.com>
Presubmit-Verified: CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com>
Commit-Queue: Auto-Submit <auto-submit@pigweed-service-accounts.iam.gserviceaccount.com>
This commit is contained in:
Ted Pudlik 2024-06-14 19:16:36 +00:00 committed by CQ Bot Account
parent a200b716cd
commit 71bc6ad022
3 changed files with 2 additions and 53 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
pw_build/BUILD.bazel
View File

@ -100,8 +100,8 @@ pw_cc_test(
# run manually via,
#
# bazel build \
# --config=stm32f429i_freertos \
# --//pw_thread_freertos:config_override=//pw_build:test_module_config \
# --platforms=//pw_build/platforms:testonly_freertos \
# //pw_build:module_config_test
tags = ["manual"],
deps = ["//pw_thread:thread"],

33
pw_build/platforms/BUILD.bazel
View File

@ -76,39 +76,6 @@ platform(
parents = [":nrf52833"],
)
# ----------------------
# --- Test Platforms ---
# ----------------------
# This is a platform for compilation testing of freertos backends. This is not
# a complete specification of any real target platform.
platform(
name = "testonly_freertos",
constraint_values = [
# Use FreeRTOS backends.
"//pw_build/constraints/rtos:freertos",
# Use the FreeRTOS config file for stm32f429i_disc1_stm32cube.
"//targets/stm32f429i_disc1_stm32cube:freertos_config_cv",
# Use the ARM_CM4F port of FreeRTOS.
"@freertos//:port_ARM_CM4F",
# Use the baremetal pw_sys_io backend (because the default
# pw_sys_io_stdio backend is not compatible with FreeRTOS).
"//pw_sys_io_baremetal_stm32f429:backend",
# Select cortex-m backends
"//pw_cpu_exception:basic_handler_backend",
"//pw_cpu_exception_cortex_m:entry_backend",
"//pw_cpu_exception_cortex_m:support_backend",
"//pw_interrupt_cortex_m:backend",
"//pw_malloc_freertos:backend",
# os:none means, we're not building for any host platform (Windows,
# Linux, or Mac). The pw_sys_io_baremetal_stm32f429 backend is only
# compatible with os:none.
"@platforms//os:none",
# For toolchain resolution.
"@pw_toolchain//constraints/arm_mcpu:cortex-m4",
],
)
# ----------------------
# --- Misc Platforms ---
# ----------------------

20
pw_presubmit/py/pw_presubmit/pigweed_presubmit.py
View File

@ -859,29 +859,11 @@ def bazel_build(ctx: PresubmitContext) -> None:
*targets,
)
# Provide some coverage of the FreeRTOS build.
#
# This is just a minimal presubmit intended to ensure we don't break what
# support we have.
#
# TODO: b/271465588 - Eventually just build the entire repo for this
# platform.
build_bazel(
ctx,
'build',
'--platforms=//pw_build/platforms:testonly_freertos',
'//pw_sync/...',
'//pw_thread/...',
'//pw_thread_freertos/...',
'//pw_interrupt/...',
'//pw_cpu_exception/...',
)
build_bazel(
ctx,
'build',
'--config=stm32f429i_freertos',
'--//pw_thread_freertos:config_override=//pw_build:test_module_config',
'--platforms=//pw_build/platforms:testonly_freertos',
'//pw_build:module_config_test',
)