-
Notifications
You must be signed in to change notification settings - Fork 8
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
Some examples can't be run from the root directory #8
Comments
I've started looking into this, and if I understand it correctly, in order to let an example be runnable with this command the following things need to be fixed (let's say in the case of In the root/Cargo.toml, this needs to be added: Then, since "https://github.com/rahix/avr-hal" is used in all our examples, this dependency seems needed in the root/Cargo.toml, with 'features = ["arduino-nano"]'. (To get the uno-examples to work, I assume 'features = ["arduino-uno"]' needs to be added too.) After this, cargo complains about this:
I saw your Issue "Switch to stable rust #11", so I paused here, since it seemed to add up to a bit too much for these additions to be reasonable. I think we should at least wait until Issue #11 is worked out (either confirming that it can be built using stable and implemented, or that it is not possible), and then considering if it's worth adding dependencies to this crate in the form of What do you think? |
Sounds good. I took a stab at building with stable this morning, but didn't get it to work. |
I started looking into this again, and made some progress I believe. A smoother way forward might be to add this to the root/Cargo.toml (in this case, only regarding the Uno example):
Then, running from root: After fighting a bit to get it to compile without running into
This is something I haven't been able to get by, and not sure if it will be possible in this case. I wanted to leave a comment about it here anyway, because this seems to be a better way of running examples in some cases from |
Normally, you can run examples with-
The text was updated successfully, but these errors were encountered: