Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rework for supporting ESP-IDF 5 and Catch2 v3 #1

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

5p4k
Copy link

@5p4k 5p4k commented Oct 3, 2023

The latest version of Catch2 has a compiled component. This means that

  1. the target must be updated to download as well a cpp file
  2. there is no more need for the -Wl,--whole-archive option (which is now available as an option of idf_component_register as well)
  3. there is no need to bundle the tests into a separate component in order to use --whole-archive only on the tests.

ESP-IDF as well got updated, and now starting from version 4 there is no more legacy makefile support, so I removed that; stack size also had to be increased a little more.

The options CATCH_CONFIG_NO_POSIX_SIGNALS and CATCH_CONFIG_DISABLE_EXCEPTIONS are still necessary, so I created a wrapping header that sets them before including catch_amalgamated.hpp (they also had to be set as compile flags).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant