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

Deliberately remove a previous venv for the python clients #1300

Merged
merged 1 commit into from
Nov 4, 2023

Conversation

rdmark
Copy link
Member

@rdmark rdmark commented Nov 4, 2023

Since a broken python venv after software upgrades is a perennial issue, and reported moments ago on Discord by @benjamink when testing the latest develop code, I propose the brute force approach of deleting the venv dir for each python client when running their respective installation script via easyinstall.sh

Copy link
Collaborator

@benjamink benjamink left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@rdmark
Copy link
Member Author

rdmark commented Nov 4, 2023

@benjamink Did you test the modified script on your RPi? I haven't tested it in a Linux OS yet. Need to spin up my very slow UTM emulator to test it in a regular Ubuntu, which I'll do myself shortly too. :)

(Only have a MacBook handy at the moment, due to moving internationally)

@rdmark rdmark merged commit cad3df6 into develop Nov 4, 2023
6 checks passed
@rdmark rdmark deleted the rdmark-issue-1151 branch November 4, 2023 12:21
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