Exclude some imports so clitable will work on windows #82
Conversation
|
This change: if token == '':is because Python now generates a syntax warning on the previous code. |
|
@ktbyers you are my hero! I was just beating my head against a wall helping a coworker install and use Napalm only to hit this wall. Looks like I'll be manually adding a try/except block to terminal.py for the time being. That said, I will still use this as an excuse to have my employer support Linux workstations... |
|
@harro Any possibility of including this PR and cutting a new release? This issue causes a lot of pain in upstream libraries that use TextFSM including: NAPALM, ntc-templates, juniper pyez, and Nornir. Regards, Kirk |
|
Google, please merge! |
Agreed. Thank you! |
ntc-templates and Netmiko both use clitable, but the import of terminal.py (via texttable.py) breaks support on windows (even though there is no code in terminal.py that needs executed given what we are doing).
So this fixes this behavior.
This improves on the PR in:
#68
Where
fcntlwas moved to the try/except.