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

incorrect compile order with UVVM #233

Closed
nathanaelhuffman opened this issue Mar 14, 2017 · 2 comments
Closed

incorrect compile order with UVVM #233

nathanaelhuffman opened this issue Mar 14, 2017 · 2 comments
Labels
ThirdParty: UVVM Related to UVVM.

Comments

@nathanaelhuffman
Copy link
Contributor

I get a bunch of these warnings:
failed to find a primary design unit 'td_cmd_queue_pkg' in library 'bitvis_vip_uart' etc for every vvc. Something about the generic packages maybe?

even though the target dependent files are added to the correct library. I have cases where the dependency order works out ok, but I also have some where it does not and the files that are not found in the correct order

working compile order (not used in testbench though) from run.py -f:
image

Non-working order, you can see the td_queue_pkg is way at the bottom from run.py -f:
image

To make things even stranger, a co-worker can build this same test-bench just fine (his paths are different than mine, but all the same files) so there seems to be some kind of non-determinism here. I get the same results under python3.6 and 2.7.

Sigasi with the project set up exports the compile order as expected.

@LarsAsplund
Copy link
Collaborator

We have a pending update for recognizing package instantiations. Without that some dependencies are missing. Meanwhile you can use the add_dependency_on method (http://vunit.github.io/python_interface.html) to manually create the missing dependency.

@kraigher
Copy link
Collaborator

I added dependency scanning usage of a package instance now. To discriminate against nested package instances I use the indentation level as a heuristic. It should work well in practice, if it doesn't we will have to pursue more complicated solutions.

@eine eine added the ThirdParty: UVVM Related to UVVM. label Oct 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ThirdParty: UVVM Related to UVVM.
Projects
None yet
Development

No branches or pull requests

4 participants