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

Build with Ubuntu 20.04 #35

Merged
merged 4 commits into from
Jun 9, 2023
Merged

Build with Ubuntu 20.04 #35

merged 4 commits into from
Jun 9, 2023

Conversation

bitdivine
Copy link
Member

@bitdivine bitdivine commented Jun 8, 2023

Motivation

Building with ubuntu-latest yielded binaries that work on Ubuntu-22.04 but not on 20.04:
Screenshot from 2023-06-01 18-26-27

But building with Ubuntu 20.04 yields binaries that work with both:

Screenshot from 2023-06-08 19-12-16

Changes

  • Build with Ubuntu 20.04
  • Test builds with both Ubuntu 20.04 and 22.04.

@bitdivine bitdivine changed the title Test DO NOT MERGE: Use Ubuntu 20.04 Build with Ubuntu 20.04 Jun 8, 2023
#- os: ubuntu-22.04
# asset_name: ic-wasm-linux64
- os: ubuntu-22.04
asset_name: ic-wasm-linux64
Copy link
Member Author

Choose a reason for hiding this comment

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

Note: I made an error in my last PR, commenting out Ubuntu 22.04 instead of 20.04. However now both can be uncommented.

@bitdivine bitdivine marked this pull request as ready for review June 8, 2023 17:21
@bitdivine bitdivine requested a review from a team as a code owner June 8, 2023 17:21
@bitdivine bitdivine requested review from chenyan-dfinity and removed request for a team June 8, 2023 17:21
@bitdivine bitdivine merged commit fbeb311 into main Jun 9, 2023
3 checks passed
@bitdivine bitdivine deleted the build-with-u20 branch June 9, 2023 07:21
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.

None yet

2 participants