Skip to content

Support module-manager kernels in CMake builds - #813

Open
Tisya-Gupta wants to merge 2 commits into
eclipse-threadx:devfrom
Tisya-Gupta:feature/cmake-module-manager-support
Open

Tisya-Gupta wants to merge 2 commits into
eclipse-threadx:devfrom
Tisya-Gupta:feature/cmake-module-manager-support

Conversation

@Tisya-Gupta

@Tisya-Gupta Tisya-Gupta commented Oct 22, 2026 •

Copy link
Copy Markdown

This PR adds support for building ThreadX with module manager in CMake. Right now CMake only supports standard and SMP builds, so anyone using FetchContent or add_subdirectory cannot build the module manager kernel without patching the build files.

I added a THREADX_MODULE_MANAGER option that pulls the port sources from ports_module and compiles the common module manager files, defines TXM_MODULE_MANAGER, and exports the module headers like txm_module.h. It also adds a small check target to make sure consumer code compiles, updates the Cortex-M CI workflow, and adds build steps to the README.

Closes #770

Tisya-Gupta and others added 2 commits October 22, 2026 11:36
Signed-off-by: Tisya-Gupta <tisyagupta793@gmail.com>
The file is new in this change, so it carries only the Eclipse ThreadX contributors
copyright.

Assisted-by: Claude Code (Opus 5.5) <noreply@anthropic.com>
@fdesbiens

Copy link
Copy Markdown
Contributor

Thank you for this, @Tisya-Gupta. CI is green across every workflow, and I also built the module-manager kernel and the consumer check locally for Cortex-M0+, M3, M4, M7, M23, M33 and R52 with GCC 14.3; all of them build.

I pushed one small commit to your branch: the new common_modules/module_manager/CMakeLists.txt carried a 2024 Microsoft Corporation copyright line, but since the file is new in this change it carries only the Eclipse ThreadX contributors line.

A possible follow-up, not needed for this PR: CI covers the GNU M0+, M3, M4 and M7 module ports, and M23, M33 and R52 also build with THREADX_MODULE_MANAGER=ON, so they could be added to the workflow later.

This branch has not been deployed

No deployments
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.

2 participants