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

python-msp430-tools: use python3 #42223

Closed
wants to merge 2 commits into from

Conversation

paper42
Copy link
Member

@paper42 paper42 commented Feb 12, 2023

Testing the changes

  • I tested the changes in this PR: briefly

The readme mentions that python2 is needed for most modules, but we are slowly removing python2 packages, so I would recommend people to use a python2 venv and pip.

cc @leahneukirchen

@leahneukirchen
Copy link
Member

sgtm, I can't test it as I don't work with MSP430 currently.

@github-actions
Copy link

Pull Requests become stale 90 days after last activity and are closed 14 days after that. If this pull request is still relevant bump it or assign it.

@github-actions github-actions bot added the Stale label May 14, 2023
@leahneukirchen
Copy link
Member

Can you rebase?

@github-actions github-actions bot removed the Stale label May 16, 2023
paper42 and others added 2 commits August 13, 2023 11:04
python2, not used by any other packages anymore
@classabbyamp
Copy link
Member

I think it might be better to just recommend using a py2 venv + pip2 for msp430-tools too. upstream has done 1 commit in the past 5 years, no indication of them moving most of the package to py3.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
remove Package removal
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants