-
Notifications
You must be signed in to change notification settings - Fork 260
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
No warning of erroneous architecture when adding configurations #1026
Comments
Can you provide an MRE. I can't seem to reproduce this so maybe I misunderstood your code. |
Sorry for the late reply. vunit_issue.tar.gz contains two files, run.py and test.vhd. Doing |
This should be fixed now. |
When adding a generic to a testbench with the "add_config" method, no warning is given to the user if the architecture of the testbench is not correct.
I experienced this when writing "t_dummy" instead of "tb_dummy" for the entity of the architecture of a testbench, where the entity is called "tb_dummy". This caused vunit not to create a default configuration, leading to the following error when "add_config" was called:
What caused this error was not obvious at all and took some time to find what caused it. Spelling the architecture correctly fixed the issue. To remedy this, the shown error could be ignored, and continue with the compilation of all design and testbench files, which would highlight the misspelling.
It is possible that different kinds of errors in the testbench file also can cause this, and not just a misspelling of the entity in the architecture declaration.
The text was updated successfully, but these errors were encountered: