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

Update CI hosts and fix clang-cl build #189

Merged
merged 5 commits into from
Aug 13, 2022

Conversation

skrobinson
Copy link
Contributor

There have been hosting changes and we could expand to more test environments.

windows-latest changed to Windows 2022 during first quarter of 2022.  This
adds windows-2019 to CI test matrix do that we can catch changes that break
the older configuration.

Signed-off-by: Sean Robinson <[email protected]>
This expands test environment coverage to two MacOS versions.

Signed-off-by: Sean Robinson <[email protected]>
Several people using clang-cl on Windows are reporting build errors after
a upgrading to v13.  This tries to replicate the error to allow testing
a fix.

Signed-off-by: Sean Robinson <[email protected]>
The new "windows-latest-clang" target does not honor the CXX environment
variable and uses the default MSVC compiler.  This tries to tell cmake to
handle things via a toolset name.

Signed-off-by: Sean Robinson <[email protected]>
@skrobinson skrobinson changed the title Update CI hosts Update CI hosts and fix clang-cl build Aug 11, 2022
@skrobinson skrobinson marked this pull request as ready for review August 11, 2022 20:45
clang-ci >= 13.0 is failing to compile a constexpr function pointer
assignment.  This is possibly due to strtof (and family) not being
implemented as constexpr in the relevant libc++.

Closes p-ranav#136
Closes p-ranav#156

Signed-off-by: Sean Robinson <[email protected]>
@skrobinson skrobinson force-pushed the wip-update-ci-hosts branch from dbdacc2 to a8e2823 Compare August 11, 2022 21:02
@p-ranav p-ranav merged commit c44dde5 into p-ranav:master Aug 13, 2022
@skrobinson skrobinson deleted the wip-update-ci-hosts branch September 19, 2022 13:44
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