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

Add Windows support #631

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Add Windows support #631

wants to merge 7 commits into from

Conversation

WandersonGN
Copy link

Status

READY

Description

This pull request adds support for Windows by initializing colorama when loading the interpreter module, so colored output can be printed to Windows's command prompt, and by using pyreadline on Windows systems as a replacement for Linux's readline module.
Closes Issue #619

Verification

Provide steps to test or reproduce the PR.

  1. Clone git clone https://github.com/threat9/routersploit && cd routersploit
  2. Install python3 setup.py install
  3. Execute rsf

Checklist

Add colorama and pyreadline to the dependency list if the running operating system is Windows.
Initialize colorama if the running operating system is Windows, so colored output can be correctly displayed to Windows's command prompt.
Add "Additional requirements for Windows" subsection to "Requirements";
Add "Installation on Windows" subsection to "Installation".
Prevent is_libedit from raising "Type Error: argument of type 'NoneType' is not iterable" when running on Windows with pyreadline instead of default readline module.
@lucyoa
Copy link
Contributor

lucyoa commented Dec 31, 2019

Checks does not pass, please fix it according to what travis says.

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

Successfully merging this pull request may close these issues.

TypeError: argument of type 'NoneType' is not iterable
2 participants