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

COMPILING.md: Add section about cross compiling for Raspberry Pi #1405

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

oyv
Copy link

@oyv oyv commented Nov 24, 2024

Include some notes on compiling with armv6.

Include some notes on compiling with armv6.
RUSTFLAGS="-C linker=arm-linux-gnueabihf-gcc" cargo build --no-default-features --target armv7-unknown-linux-gnueabihf
```

Note that this does not work with armv6 (found e.g. in Raspberry Pi 1 and Zero). Previously, you could use the linker in https://github.com/raspberrypi/tools, but this is now too old and will give an error when compiling Librespot (since `3e85d77bfb3520e16c35bb7db3912c76e8cc90d7`). More info: https://github.com/japaric/rust-cross/issues/42
Copy link
Contributor

Choose a reason for hiding this comment

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

I compile librespot for armv6 using the official raspi tools no problem. But I don't use cargo cross. I suggest just stating your method doesn't work for pi 1 and zero.

Copy link
Author

Choose a reason for hiding this comment

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

I'm also not using cargo cross. After commit 3e85d77 I get an error from the rpi tools linker saying it found dwarf version 6, but only supports versions 1-4. Maybe I'm using a newer version of Rust (I'm on 1.82.0)?

Copy link
Contributor

Choose a reason for hiding this comment

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

I'll update and see what I get. Does this happen even when compiling with the release profile?

Copy link
Member

Choose a reason for hiding this comment

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

@oyv friendly nudge - did you get around to testing what @kingosticks asked?

@roderickvd roderickvd requested a review from Copilot December 15, 2024 20:29

Choose a reason for hiding this comment

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.

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