-
Notifications
You must be signed in to change notification settings - Fork 123
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
Fix macOs Actions #4976
Fix macOs Actions #4976
Conversation
Have you tried |
Brew install didnt work because a higher version is installed, brew pin I didnt try. |
Well, it wouldn't work either way, just tried it. Seems like this solution here is the only way to do it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thx, when it solves the problem, please merge.
Yea, the best would be probably to port libelektra to 2.5.6 and avoid dark magic in the yml file. |
Does libelektra actually directly depend upon |
It is only a development dependency, for ASAN builds. I don't think anyone would use it for production builds. |
Will try some stuff on the macos github actions on my fork, will close this to not start the other pipelines. |
These tests behave very weird, I removed all my changes, and the macos/cirrus tests pass now. |
jenkins build libelektra please |
It seems like time itself fixed this issue. So there's probably been a package update in homebrew, this PR is no longer needed. Thanks for working on it anyways! |
Nice! |
#4973
Basics
(added as entry in
doc/news/_preparation_next_release.md
which contains_(my name)_
)Please always add them to the release notes.
(first line should have
module: short statement
syntax)close #X
, are in the commit messages.doc/news/_preparation_next_release.md
scripts/dev/reformat-all
Checklist
(not in the PR description)
Review
Labels