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

ci: test all c++ versions #246

Merged
merged 1 commit into from
Feb 19, 2025
Merged

ci: test all c++ versions #246

merged 1 commit into from
Feb 19, 2025

Conversation

iMichka
Copy link
Contributor

@iMichka iMichka commented Feb 17, 2025

Clarify that we are using clang
Cleanup CI
Add more macos versions

@iMichka
Copy link
Contributor Author

iMichka commented Feb 17, 2025

@johnnynunez I need to check something before merging this. I think the cppstd value has no effect ... I would have expected this PR to fail.

@johnnynunez
Copy link
Contributor

@johnnynunez I need to check something before merging this. I think the cppstd value has no effect ... I would have expected this PR to fail.

I change it in macos-15 and has effect from ++98 to 17. And works in arm

@johnnynunez
Copy link
Contributor

johnnynunez commented Feb 18, 2025

seems that affects

2025-02-18T06:47:22.7836215Z ##[group]Run echo "ccflags=-std=c++20" >> tests/xml_generator.cfg
2025-02-18T06:47:22.7836649Z �[36;1mecho "ccflags=-std=c++20" >> tests/xml_generator.cfg�[0m

@johnnynunez
Copy link
Contributor

@iMichka how is it going?

@iMichka iMichka force-pushed the extend branch 7 times, most recently from 39dda54 to ad9a9ef Compare February 19, 2025 21:20
@iMichka
Copy link
Contributor Author

iMichka commented Feb 19, 2025

Ok I know what is going on:

  • I made a cleanup to make clear that the tests are running with clang++ and not gcc
  • I added tests for more clang versions (13 up to 16)
  • c++20 and c++23 can not be tested: clang 16 does not support c++20 fully, and c++23 is not supported at all. We will have to wait for clang 17

@iMichka iMichka force-pushed the extend branch 2 times, most recently from f9679d0 to 35a2d2b Compare February 19, 2025 21:40
Clarify that we are using clang
Cleanup CI
Add more macos versions
@iMichka
Copy link
Contributor Author

iMichka commented Feb 19, 2025

Clang 15 on macOS is not working: skipping it for the time being, this might need a new castxml version.

@iMichka iMichka merged commit 8d118f8 into develop Feb 19, 2025
16 checks passed
@iMichka iMichka deleted the extend branch February 19, 2025 21:55
@johnnynunez
Copy link
Contributor

Ok I know what is going on:

  • I made a cleanup to make clear that the tests are running with clang++ and not gcc
  • I added tests for more clang versions (13 up to 16)
  • c++20 and c++23 can not be tested: clang 16 does not support c++20 fully, and c++23 is not supported at all. We will have to wait for clang 17

thank you for amazing work and the information! I will rebase that PR

@mahmoudeA
Copy link

Gh

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.

3 participants