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

Python3 Cleanup #10

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Python3 Cleanup #10

wants to merge 1 commit into from

Conversation

Zelec
Copy link

@Zelec Zelec commented Nov 21, 2023

To make it clear, I didn't do these edits, @andymcca did in the sshflash-win repo
I'm just pushing some of the cleanup I did to make flashing RetroLeap work with python3 on my laptop.

No real code tweaks required, all worked fine on Linux with Python 3.11 to flash my LeapPad 2 (more testing with other hardware may be required)

Only four cleanup items done on my end that didn't already exist

  • reverted .exe references back to .py
  • Updated shebangs to reference /usr/bin/env across the board (most already had this)
  • Updated python shebangs to target python3
  • Updated remote_flash.sh to reference python3 instead of python2

Pulled code from the sshflash-win repo owned by @andymcca
https://github.com/andymcca/sshflash-win

No real code tweaks required, all worked fine on Linux with Python 3.11

Only three cleanups done
- reverted .exe references back to .py
- Updated shebangs to reference /usr/bin/env across the board
- Updated python shebangs to target python3
@Zelec
Copy link
Author

Zelec commented Nov 21, 2023

Forgot to include this is to close Issue #6

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.

1 participant