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

Bug/fix gh action aarch64 tarball #872

Merged
merged 1 commit into from Jun 13, 2023

Conversation

m2nx
Copy link
Contributor

@m2nx m2nx commented Apr 13, 2023

this PR is going to fix a bug that github action will not create aarch arm64 tar file correctly.

you can see the error here https://github.com/ellie/atuin/actions/runs/4584398463/jobs/8095857446
image

how to fix

the first commit will fix this problem. just add a RUSTFLAGS to set linker aarch64-linux-gnu-gcc
RUSTFLAGS=-C linker=aarch64-linux-gnu-gcc

the second commit is to bypass generate completion job because you can't run arm format on a Github hosted runner which is x86-64. we need to discuss how to fix the completion file.

here's result:

image

FYI:

https://crates.io/crates/grb/1.3.0
https://users.rust-lang.org/t/unable-to-compile-to-aarch64-unknown-linux-gnu/53517

@vercel
Copy link

vercel bot commented Apr 13, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
atuin ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 11, 2023 5:08am

@ellie
Copy link
Member

ellie commented Apr 14, 2023

Hey! Thank you for working on this!

the second commit is to bypass generate completion job because you can't run arm format on a Github hosted runner which is x86-64. we need to discuss how to fix the completion file.

It would be nice to solve this 🤔

Maybe we should virtualize ARM for generating completions?

@m2nx
Copy link
Contributor Author

m2nx commented May 10, 2023

got it. I will give it a try to fix this.

@m2nx
Copy link
Contributor Author

m2nx commented May 11, 2023

image
image

plz let me know if there is any issue.

@m2nx
Copy link
Contributor Author

m2nx commented May 23, 2023

Hi @ellie is there anything else I can do about this PR?

Copy link
Member

@ellie ellie left a comment

Choose a reason for hiding this comment

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

Thank you for this! And sorry for being so slow to review it 😓

Seeing as this is your first time contributing, if you would like a holographic contributors-only Atuin sticker, then please fill out this form!

We do also have a Discord if you'd like to ask any questions, or just fancy hanging out!

@ellie ellie merged commit dea6898 into atuinsh:main Jun 13, 2023
2 checks passed
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