Skip to content

Conversation

@cuiziweizw
Copy link
Contributor

…e upstream integrity

Summary

  • Add c++config.h to improve libstdc++ configuration support
  • Update CMakeLists.txt and libsupc++.defs to integrate the new config header
  • Remove the obsolete 0001-libsupc-Add-missing-header-files.patch to clean up the codebase

These changes enhance the maintainability and portability of the C++ standard library and simplify the build process.

Impact

  • c++config.h is a configuration file that supports the libstdc++ library. Due to its upstream origin, the code style within this file cannot be adjusted to meet the Apache NuttX community's CI formatting requirements. It may be necessary to skip the community CI format checks for this file to avoid unnecessary failures.

Testing

Test Environment

Host: Linux x86_64
Build config: default libstdc++ options
Board: QEMU/simulator

Test Steps

  1. Clean and rebuild nuttx
  2. Enable CONFOG_LIBSUPCXX and CONFIG_CXXTEST
  3. Ensure successful compilation with no errors
  4. Run the cxxtest example to verify libsupc++ functionality

Test Results

Simulated Test Scenario

./tools/configure.sh sim:nsh
make menuconfig
make -j16

nsh> cxxtest

…e upstream integrity

- Add c++config.h to improve libstdc++ configuration support
- Update CMakeLists.txt and libsupc++.defs to integrate the new config header
- Remove the obsolete 0001-libsupc-Add-missing-header-files.patch to clean up the codebase

These changes enhance the maintainability and portability of the C++ standard library and simplify the build process.
@github-actions github-actions bot added Area: OS Components OS Components issues Size: XL The size of the change in this PR is very large. Consider breaking down the PR into smaller pieces. labels Jan 26, 2026
@xiaoxiang781216
Copy link
Contributor

@cuiziweizw please fix ci check error.

@cuiziweizw
Copy link
Contributor Author

c++config.h is a configuration file that supports the libstdc++ library. Due to its upstream origin, the code style within this file cannot be adjusted to meet the Apache NuttX community's CI formatting requirements. It may be necessary to skip the community CI format checks for this file to avoid unnecessary failures.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: OS Components OS Components issues Size: XL The size of the change in this PR is very large. Consider breaking down the PR into smaller pieces.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants