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

Add note about CARGO_TARGET_DIR requiring CARGO_WORKSPACE_DIR #221

Merged
merged 1 commit into from
Aug 6, 2023

Conversation

faern
Copy link
Contributor

@faern faern commented Jul 23, 2023

My sdkconfig.defaults file is always ignored. And it takes so much time before I figure out why. I never have CARGO_TARGET_DIR unset, because I don't want to fill up my source code folders with build artifacts. This embuild caveat is poorly documented. It would be great if the esp-idf-sys documentation would mention it.

I would add a link to a reference on why this is needed if I had one. But I can't remember where I learned about this workaround. I learned about it a long time ago but then I forget about it every time I pick up esp programming.

@N3xed
Copy link
Collaborator

N3xed commented Aug 6, 2023

CI failure unrelated. Thanks for the addition!

@N3xed N3xed merged commit c0dfced into esp-rs:master Aug 6, 2023
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.

2 participants