-
Notifications
You must be signed in to change notification settings - Fork 338
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
textfsm window version check #1019
Conversation
if sys.version_info[:2] == (2, 6): | ||
install_reqs.append('importlib>=1.0.3') | ||
if sys.platform == 'win32': | ||
install_reqs.append('textfsm==0.4.1') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why did we remove importlib ?
Kindly add the comment saying win32 supports testfsm 0.4.1.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We dont support python 2.6 and importlib was explicitly needed for that version.
>=2.7
python got importlib by default
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ship it!
Juniper#1019 no longer addresses issue of textfsm import in windows because of a change in ntc-templates==1.4.2 : ImportError: cannot import name 'clitable' from 'textfsm' (<redacted>\lib\site-packages\textfsm\__init__.py)
Right now only around 800 test cases get executed for windows and others are skipped. will fix skipped one in future.