mirror of https://github.com/espressif/esp-idf.git synced 2024-10-05 20:47:46 -04:00

History

Marc Finet 9456c157ff feat(build): Add config to disable warn be considered as errors The -Werror=all activates error for all warnings in -Wall, however, it does not activate error for other default warnings, such as: - int-conversion (pointer from integer w/o a cast) - incompatible-pointer-types - discarded-qualifiers Which are IMO even more important that -Wall. This commit fixes that by activating error for all warnings (i.e. from -Wall and default ones) and removing those from -Wextra, as the culprit commit seemed to address. Fixes: `60f29236f6` "Build system: Raise warning level" (2016-11-16) In order to avoid long analysis during esp-idf upgrade, provide a way to restore the previous -Werror=all behavior that consider only warnings from -Wall (and not default ones). Also add a hint to use the Kconfig option on compilation error, but warn that fixing the code is the preferred way. Merges https://github.com/espressif/esp-idf/pull/11239 Suggested-By: Ivan Grokhotkov <ivan@espressif.com>	2024-06-20 14:43:26 +08:00
..
README.md	ci: fixed test apps overriding pytest configs	2023-04-26 11:07:35 +08:00
sdkconfig.debug_helpers	feat(build): Add config to disable warn be considered as errors	2024-06-20 14:43:26 +08:00

Marc Finet 9456c157ff feat(build): Add config to disable warn be considered as errors

The -Werror=all activates error for all warnings in -Wall, however, it
does not activate error for other default warnings, such as:
- int-conversion (pointer from integer w/o a cast)
- incompatible-pointer-types
- discarded-qualifiers

Which are IMO even more important that -Wall.

This commit fixes that by activating error for all warnings (i.e. from
-Wall and default ones) and removing those from -Wextra, as the culprit
commit seemed to address.

Fixes: 60f29236f6 "Build system: Raise warning level" (2016-11-16)

In order to avoid long analysis during esp-idf upgrade, provide a way to
restore the previous -Werror=all behavior that consider only warnings
from -Wall (and not default ones).

Also add a hint to use the Kconfig option on compilation error, but warn
that fixing the code is the preferred way.

Merges https://github.com/espressif/esp-idf/pull/11239

Suggested-By: Ivan Grokhotkov <ivan@espressif.com>

2024-06-20 14:43:26 +08:00

README.md ci: fixed test apps overriding pytest configs 2023-04-26 11:07:35 +08:00

sdkconfig.debug_helpers feat(build): Add config to disable warn be considered as errors 2024-06-20 14:43:26 +08:00

README.md

Configs

Certain combinations of configs, e.g. stack and heap debug configs, are useful to set in most test apps.

To facilitiate re-use, this folder contains some of the most common ones.

These can then be added to the list of default configs in test apps:

list(PREPEND SDKCONFIG_DEFAULTS "$ENV{IDF_PATH}/tools/test_apps/configs/sdkconfig.debug_helpers" "sdkconfig.defaults")

Note that this must be set in the top-level project CMakelist.txt.

These files should not be considered stable, and are thus not recommended for use outside of IDF.