Skip to content

[cppyy] Enable tests on windows #18872

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

aaronj0
Copy link
Contributor

@aaronj0 aaronj0 commented May 26, 2025

Trigger a verbose CI run with Windows to examine the cppyy test status

@aaronj0 aaronj0 requested a review from guitargeek May 26, 2025 18:04
@aaronj0 aaronj0 requested review from bellenot and dpiparo as code owners May 26, 2025 18:04
@aaronj0 aaronj0 added the clean build Ask CI to do non-incremental build on PR label May 26, 2025
@aaronj0 aaronj0 closed this May 26, 2025
@aaronj0 aaronj0 reopened this May 26, 2025
@aaronj0 aaronj0 self-assigned this May 26, 2025
Copy link

github-actions bot commented May 26, 2025

Test Results

    18 files      18 suites   3d 10h 58m 0s ⏱️
 2 776 tests  2 760 ✅ 0 💤 16 ❌
48 793 runs  48 776 ✅ 0 💤 17 ❌

For more details on these failures, see this check.

Results for commit 67dc353.

♻️ This comment has been updated with latest results.

@aaronj0 aaronj0 force-pushed the cppyy-win-tests branch from 81cad25 to 7025fed Compare May 26, 2025 19:04
@guitargeek
Copy link
Contributor

2025-05-26T19:53:28.0200811Z 24:         if sc == -1:
2025-05-26T19:53:28.0200984Z 24: >           raise RuntimeError("Unable to load reflection library "+name)
2025-05-26T19:53:28.0201385Z 24: E           RuntimeError: Unable to load reflection library C:\ROOT-CI\build\bindings\pyroot\cppyy\cppyy\test/stltypesDict

Looks like we have to use a proper os.path.join in Python...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
clean build Ask CI to do non-incremental build on PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants