-
Notifications
You must be signed in to change notification settings - Fork 16.9k
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
Tools: add --upload-port option to waf #12404
Conversation
22a623c
to
06fe734
Compare
I've done changes to improve this on WSL and it works for me fine. Are you still having problems with the latest master implementation? |
It's been working well for me on WSL too |
Yes, it basically hangs for me, whereas if I specify the tty it works instantly. I think it depends what devices you have connected to the other serial ports. There are also a lot of ports (192 in my case) so to get to ttyS14 say you have to look at 46 other ports first. Given that the uploader has this option (and I wouldn't be able to use it if it didn't) what's the issue in adding the option? |
@andyp1per this should be merged, but can you add the default=None? |
Yes certainly |
06fe734
to
e2c90b0
Compare
@tridge ok done |
thanks! |
On windows WSL at least, autodetect on upload does not work well and specifying a port is necessary. This allows this be done from the waf command-line
@peterbarker FYI