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

The atomic library needs to be linked on more Linux versions #1032

Merged
merged 2 commits into from
Jun 12, 2021

Conversation

sjoelund
Copy link
Member

Previously only 32-bit Linux were linked, but arm64 (on Xenial, but not
more recent versions) also needs it.

@sjoelund sjoelund requested a review from adrpo June 11, 2021 09:22
@adrpo adrpo requested a review from lochel June 11, 2021 09:26
Copy link
Member

@adrpo adrpo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Previously only 32-bit Linux were linked, but arm64 (on Xenial, but not
more recent versions) also needs it.
@sjoelund sjoelund merged commit efc6c8f into OpenModelica:master Jun 12, 2021
@sjoelund sjoelund deleted the atomic-all-linux branch June 12, 2021 13:50
@lochel
Copy link
Member

lochel commented Jun 12, 2021

@sjoelund await the requested reviews before merging!

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