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

set CROSS_COMPILE env variable #308

Open
sampaioletti opened this issue May 27, 2024 · 1 comment
Open

set CROSS_COMPILE env variable #308

sampaioletti opened this issue May 27, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@sampaioletti
Copy link

Would it be feasible to have the CROSS_COMPILE environment variable set by this crate's build.rs. Building for a lot of esp targets requires manually changing it as cargo doesn't support per target env variables or using an external build runner.

Not sure if this would be done here or if it would have to come from the embuild crate so it would happen in the build.rs of the users project.

It could be a 'set if not already set' by user. Just a thought, Happy to PR if it makes sense.

@Vollbrecht
Copy link
Collaborator

Vollbrecht commented May 30, 2024

A target alone is not enough information to successfully compile a project for it. You would at least also change the MCU variant, for example the riscvimc target is valid for both esp32c2 and esp32c3, but certain esp-idf options are coupled to the actual mcu variant and not the compile target.

In general we are happy for every PR but its a non-trivial problem to look for the right places to make it work. If you want to dig deeper and have questions we are always happy to answer them!

@Vollbrecht Vollbrecht added the enhancement New feature or request label Jun 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Todo
Development

No branches or pull requests

2 participants