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

rust-toolchain.toml has no effect on build (?) #8

Open
fralalonde opened this issue Jun 29, 2022 · 8 comments
Open

rust-toolchain.toml has no effect on build (?) #8

fralalonde opened this issue Jun 29, 2022 · 8 comments

Comments

@fralalonde
Copy link

It seems that rust-toolchain.toml has no effect on build: make build fails if the default toolchain is set to stable. But if renamed to rust-toolchain (without extension) then it works as expected: the specified nightly version gets downloaded and is used for the build.

I don't know why that is since the doc says both file names should work.

(env: Arch Linux, manually installed rustup)

@nviennot
Copy link
Owner

nviennot commented Jun 29, 2022

This is strange, do you have a recommendation at this point? Should we make a symlink rust-toolchain to rust-toolchain.toml ?

@fralalonde
Copy link
Author

I don't personally need this fixed now as I was only test-building the project, which succeeded once I renamed the file. So unless it is also confirmed by someone else things can stay as they are for now? Or you could preemptively change the filename based on my sole report, it's up to you.

Context: My intention is to use the USB host code to advance the embedded USB host stack project. Your impl was suggested by @Dirbaio on Rust + USB matrix channel. Licenses are compatible (GPL3) and if it works, I'll of course add the appropriate credits and links.

@Dirbaio
Copy link

Dirbaio commented Jun 29, 2022

@fralalonde try updating rustup (not rust), I think older rustups didn't take the .toml file.

@Dirbaio
Copy link

Dirbaio commented Jun 29, 2022

also, would you guys consider licensing the usb host code as Apache+MIT instead? I'm interested to add in host functionality for embassy-usb eventually, but can't take GPL/LGPL so I'll have to reinvent the wheel if not :(

@fralalonde
Copy link
Author

I just reinstalled rustup:

❯ rustup --version
rustup 1.24.3 (ce5817a94 2021-05-31)

Seems to be the latest, but it's 1+ year old?

@nviennot
Copy link
Owner

I can change the turboresin's USB host code to Apache+MIT, absolutely. Will do tomorrow.

@nviennot
Copy link
Owner

The license is updated for the USB host stack to Apache2.0+MIT

@nviennot
Copy link
Owner

Also, please note that my implementation feels a little brittle on USB port activation (see this line). After that, the core may or may not need a port reset (see here) depending if we are running a stm32f1 or stm32f4. It's strange.
Aside from that, things are working smoothly.

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

No branches or pull requests

3 participants