vunit
vunit copied to clipboard
Fix some Vivado issues (SystemVerilog support, space in file name, environment variable)
Closes #686 Closes #796 Closes #811
Not sure where it can be tested. Didn't really find a test file for vivado.py?
The vivado module is not in the documentation? So not really clear where I should document the environment variable. (But I can improve the documentation for run_vivado.
Yes, please do something about the documentation. vivado.py started out more as an example as we generally do not maintain tool integrations other than what's related to simulators and we generally do not document the examples that much. Since this is becoming more than just an example I suggest that it also gets some documentation.
Now there is some sort of documentation for the functions in the module. Located at the bottom of the "Documentation" section (below "Examples").
Sorry to bother you @LarsAsplund , but can you please see if anything else is required (or if you prefer the Vivado menu entry elsewhere).
Sorry for letting this slip. Will try to find some time "soon" to complete it.
Finally got around to this. Not sure this is exactly what you wanted, but at least a "Tool Integration" menu item and then Vivado shows up there. Not clear to me how the "dynamic menus" work and if this will be an entry there (or if that it a better solution).
Hello, bump on this one @oscargus @LarsAsplund . Any chance this could get pulled into mainline? This is very useful for using with Verification IPs
I'll see what I can do in terms of splitting it. It's been a while.