8000 Default to python3 in the install script by cheriimoya · Pull Request #1105 · mininet/mininet · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Default to python3 in the install script #1105

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Default to python3 in the install script #1105

wants to merge 10 commits into from

Conversation

cheriimoya
Copy link
Contributor

when running mininet/util/install.sh -3fy, i ran into the following problem:

  File "setup.py", line 17, in <module>
    import setuptools
ImportError: No module named setuptools

this is because the installation script is not setting the PYPKG variable at all python dependencies correctly.

so down the rabbit hole i went and fixed some more stuff in the installation script.

the commits with their commit messages should be speaking for themselves.

also

Installing ryu script to /usr/local/bin
Installing ryu-manager script to /usr/local/bin
ln: failed to create symbolic link '/usr/local/bin/ryu-manager': File exists

gets fixed by this.

i tested this change by running mininet/util/install.sh -3fy and passed

@cheriimoya cheriimoya mentioned this pull request < 8000 a class="Link--secondary" href="#ref-issue-231177987" > Feb 2, 2022
Open
@cheriimoya cheriimoya force-pushed the feat/default-to-python3-in-install branch from 1e0866f to 7748940 Compare April 3, 2022 22:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@cheriimoya
0