Skip to content

Enable static component registration in application executables#7194

Merged
hkaiser merged 1 commit intoTheHPXProject:masterfrom
abhishek593:components
Apr 19, 2026
Merged

Enable static component registration in application executables#7194
hkaiser merged 1 commit intoTheHPXProject:masterfrom
abhishek593:components

Conversation

@abhishek593
Copy link
Copy Markdown
Contributor

Follow-up to PR #7180

Proposed Changes

  • Remove the app-exe gate so the ctors fire in application TUs as well, mirroring what Add static plugin loading support for HPX plugins #7180 did for plugin modules.

  • Link the new test against hpx_runtime_configuration (not hpx_core):
    load_component_factory_static is declared HPX_CXX_CORE_EXPORT, which is only active under the C++20 modules build. In the default dynamic build, it resolves to empty, so the symbol is hidden in libhpx_core.so by -fvisibility=hidden but remains globally visible in the per-module archive. Other tests in this directory already follow the same pattern.

Checklist

Not all points below apply to all pull requests.

  • I have added a new feature and have added tests to go along with it.
  • I have fixed a bug and have added a regression test.
  • I have added a test using random numbers; I have made sure it uses a seed, and that random numbers generated are valid inputs for the tests.

Signed-off-by: Abhishek Bansal <abhibansal593@gmail.com>
@abhishek593 abhishek593 requested a review from hkaiser as a code owner April 17, 2026 10:28
@StellarBot
Copy link
Copy Markdown

Can one of the admins verify this patch?

@codacy-production
Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

TIP This summary will be updated as you push new changes. Give us feedback

Copy link
Copy Markdown
Contributor

@hkaiser hkaiser left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@hkaiser hkaiser added this to the 2.0.0 milestone Apr 17, 2026
@hkaiser hkaiser merged commit 2d022a0 into TheHPXProject:master Apr 19, 2026
64 of 65 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants