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

Fix app installation on the simulator #93

Merged
merged 2 commits into from
Jun 16, 2024

Conversation

nomis
Copy link
Contributor

@nomis nomis commented Jun 1, 2024

Python's tarfile doesn't append a "/" to directory names like MicroPython does, so the root directory detection fails. Normalise the names by removing the "/" and then adding it back when returning the root directory name.

@nomis
Copy link
Contributor Author

nomis commented Jun 1, 2024

I've only tested this on the simulator and the app store appears to be down now, hence the handling of the 404 response from the demo server.

@nomis nomis changed the title Fix the app store on the simulator Fix the app installation on the simulator Jun 1, 2024
@nomis nomis changed the title Fix the app installation on the simulator Fix app installation on the simulator Jun 1, 2024
@thinkl33t
Copy link
Collaborator

The app store is now working again, if you could have a test of this and let u know if its working it'd be appreciated :)

@nomis
Copy link
Contributor Author

nomis commented Jun 1, 2024

I've tested this on the badge and it works

nomis added 2 commits June 1, 2024 19:21
Python's tarfile doesn't append a "/" to directory names like MicroPython
does, so the root directory detection fails. Normalise the names by
removing the "/" and then adding it back when returning the root directory
name.
@nomis nomis force-pushed the app-store-fixes branch from 8de2177 to 788904f Compare June 1, 2024 18:21
@thinkl33t thinkl33t requested a review from hughrawlinson June 1, 2024 21:32
@MatthewWilkes MatthewWilkes merged commit 8d6d6bc into emfcamp:main Jun 16, 2024
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.

3 participants