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.

Sign up for GitHub

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

Jump to bottom

CI: Add MacOs to runs-on, add rust fmt check #442

Merged
merged 14 commits into from
Dec 21, 2022
Merged

CI: Add MacOs to runs-on, add rust fmt check #442

merged 14 commits into from
Dec 21, 2022

Conversation

moozzi
Copy link
Contributor

@moozzi moozzi commented Dec 17, 2022

@moozzi
Copy link
Contributor Author

moozzi commented Dec 19, 2022

Idk why match_expression case fails :/

@yhara
Copy link
Collaborator

yhara commented Dec 20, 2022

ld: library not found for -lm

This means clang failed to load libm. Maybe we don't need the -lm option in runner.rs on macs?

@moozzi
Copy link
Contributor Author

moozzi commented Dec 20, 2022

Idk what's wrong with wrong with this. I found some answer that homebrew binary clang is broken and I'm not sure if there is sense to continue this battle :(

@yhara yhara marked this pull request as ready for review December 21, 2022 14:52
@yhara yhara merged commit 9210931 into shiika-lang:main Dec 21, 2022
@yhara
Copy link
Collaborator

yhara commented Dec 21, 2022

Merged. Nice try :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@moozzi @yhara