-
Notifications
You must be signed in to change notification settings - Fork 8.4k
Labels
area: Base OSBase OS Library (lib/os)Base OS Library (lib/os)bugThe issue is a bug, or the PR is fixing a bugThe issue is a bug, or the PR is fixing a bug
Description
Describe the bug
This define seemingly is set for all builds, irrespective if the feature is enabled or not, it should only be provided if the feature Kconfig is enabled for it
Regression
- This is a regression.
Steps to reproduce
Build hello_world for nrf52840dk/nrf52840 and examine build flags for main.c in build.ninja file
Relevant log output
#############################################
# Order-only phony target for app
build cmake_object_order_depends_target_app: phony || zephyr/device_api_ld_target zephyr/driver_validation_h_target zephyr/kobj_type>
build CMakeFiles/app.dir/src/main.c.obj: C_COMPILER__app_unscanned_ /tmp/aa/zephyr/samples/hello_world/src/main.c || cmake_object_or>
DEFINES =
-DKERNEL -DK_HEAP_MEM_POOL_SIZE=0 -DPICOLIBC_LONG_LONG_PRINTF_SCANF -DZVFS_OPEN_SIZE=0 -D_POSIX_THREAD_SAFE_FUNCTIONS=200809L -D__LI>Impact
Annoyance – Minor irritation; no significant impact on usability or functionality.
Environment
Commit: c960d0e
Additional Context
No response
Metadata
Metadata
Labels
area: Base OSBase OS Library (lib/os)Base OS Library (lib/os)bugThe issue is a bug, or the PR is fixing a bugThe issue is a bug, or the PR is fixing a bug