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

test if test catches error #39

Draft
wants to merge 8 commits into
base: master
Choose a base branch
from
Draft

test if test catches error #39

wants to merge 8 commits into from

Conversation

jschwe
Copy link
Owner

@jschwe jschwe commented Jun 22, 2023

No description provided.

@jschwe jschwe force-pushed the 339 branch 6 times, most recently from a9b0c00 to 9c4cd31 Compare June 23, 2023 09:25
@jschwe jschwe force-pushed the 339 branch 5 times, most recently from 2ff0ce3 to 4830b12 Compare August 5, 2023 06:55
The PROFILE option is set via the properties and does
not need to be passed through anymore.
That this issue has not been reported yet indicates,
that there are not too many users with older CMake versions.
When cross-compiling with clang, we need to
set the compiler target.
Previously we did not have any CI tests which cross-compile with clang
Building / Linking the cdylib requires us to specify a library
which provides the extern C function.
Test cross-compiling with clang.
This should catch the issue fixed in corrosion-rs#338.
Closes corrosion-rs#339.
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.

1 participant