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

Incompatible Required Packages #35

Closed
kolojoe opened this issue May 19, 2022 · 3 comments
Closed

Incompatible Required Packages #35

kolojoe opened this issue May 19, 2022 · 3 comments
Labels

Comments

@kolojoe
Copy link

kolojoe commented May 19, 2022

I am interested in using this tool and tried to use the VHDLParser debug tool, but it turns out that there are some incompatibilities with the packages that are currently released vs what the package says is acceptable.

The following requirements are causing issues.

pyTooling>=1.7.0
pyTooling.TerminalUI>=1.5.5

The latest version of pyTooling is 2.0.1 which no longer includes the Singleton metaclass and caused this line to fail:

  File "...\pyVHDLParser\pyVHDLParser\CLI\VHDLParser.py", line 36, in <module>
    from pyTooling.MetaClasses            import Singleton
ImportError: cannot import name 'Singleton' from 'pyTooling.MetaClasses' (...Python\Python39\lib\site-packages\pyTooling
\MetaClasses\__init__.py)

After falling back to pyTooling==1.7.0 there were further issues, but those were indirect import issues not necessarily caused by pyVHDLParser.

I upgraded pyTooling==1.9.5 but that was incompatible with pyTooling.TerminalUI==1.5.8. After downgrading back to pyTooling.TerminalUI==1.5.5 it finally worked.

Suggestion

Require pyTooling to be between 1.7.0 and 1.9.5. I haven't tested later version before 2.0.1 so perhaps just version <= 2 would work.
Require pyTooling.TerminalUI==1.5.5.

@Paebbels
Copy link
Owner

Thanks for reporting this issue.
I'll fix this in the following days.

pyTooling and pyTooling.TerminalUI, which are also written by me, got a major and braking update. I'm now going to update all packages (+20) using these dependencies. I'll need some time to fix all of them.

@Paebbels Paebbels added the Bug label May 19, 2022
@Paebbels
Copy link
Owner

This will be addressed in #48.

@Paebbels Paebbels mentioned this issue Feb 18, 2023
@Paebbels
Copy link
Owner

Unit tests on CI are passing.

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

No branches or pull requests

2 participants