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 handling for Windows "readline.__doc__ is NoneType" error. #11

Merged
merged 1 commit into from
Feb 16, 2020
Merged

Conversation

bafulton
Copy link
Contributor

@bafulton bafulton commented Oct 9, 2019

For certain Windows installs, readline.doc is None. Check before trying to search for the phrase "libedit" inside it.

For certain Windows installs, readline.__doc__ is None. Check before trying to search within it.
@bafulton
Copy link
Contributor Author

bafulton commented Oct 9, 2019

Python 2.6 and 3.3 are both EOL--hence the failing builds.

@bafulton
Copy link
Contributor Author

@clarkperkins, could this be merged?

@clarkperkins
Copy link
Owner

Hi @bafulton, sorry I have been slow to respond to this! I'll merge and get a new release pushed to pypi.

@clarkperkins clarkperkins merged commit 9ec2113 into clarkperkins:master Feb 16, 2020
@bafulton
Copy link
Contributor Author

@clarkperkins, no worries! Thanks for merging!

fwkz added a commit to fwkz/riposte that referenced this pull request Mar 4, 2024
## Changes
In certain Windows installs, readline.__doc__ is None. Added a check
before iterating.
## Issue
Improving Windows compatibility #9 
## Related pull request
Add compatibility for Windows #14 
## Inspiration
[Add handling for Windows "readline.__doc__ is NoneType"
error](clarkperkins/click-shell#11)

---------

Co-authored-by: Mariusz Kupidura <2480867+fwkz@users.noreply.github.com>
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.

None yet

2 participants