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

unable to use rshell with 8.0.3 but OK with 8.1.0-beta.0 #7720

Closed
jerryneedell opened this issue Mar 13, 2023 · 3 comments
Closed

unable to use rshell with 8.0.3 but OK with 8.1.0-beta.0 #7720

jerryneedell opened this issue Mar 13, 2023 · 3 comments

Comments

@jerryneedell
Copy link
Collaborator

jerryneedell commented Mar 13, 2023

I'm not sure this is really an "issue" since it appears to have been resolved in 8.1.0-beta.0 but I wanted to bring it to your attention.
A forum post reported an issue using rshell with 8.0.3. on a feather esp32 v2. In general, you do have to download and install rshell from GitHub to get a necessary fix that has been merged but not released. https://github.com/dhylands/rshell
I showed the original poster that rshell did work OK but I was using 8.1.0-beta.0 and they still reported an error with 8.0.3. I then reverted to 8.03 and was able to reproduce the error.
After reinstalling 8.1.0-beta.0 it now works again.

Here is a link to the forum post https://forums.adafruit.com/viewtopic.php?p=964655#p964655 with some examples.

It is not obvious to me what "fixed" this in 8.1.0-beta.0 but I though it worth reporting.

@jerryneedell jerryneedell changed the title unable to use shell with 8.0.3 but OK with 8.1.0-betat unable to use rshell with 8.0.3 but OK with 8.1.0-beta.0 Mar 13, 2023
@Neradoc
Copy link

Neradoc commented Mar 13, 2023

That would be from #7652 (which is in the 8.0.x branch but happened after 8.0.3)
It's not a perfect fix, since the message that rshell and ampy expect are localized in Circuitpython.
But using human readable text for a machine interface will do that.

@jerryneedell
Copy link
Collaborator Author

Thanks -- No need to keep this open.

@tannewt
Copy link
Member

tannewt commented Mar 14, 2023

@Neradoc should we make it not translated?

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