search

themperek / cocotb-test

Unit testing for cocotb
BSD 2-Clause "Simplified" License
144 stars 71 forks source link

Questa vcom does not know "-incr" #155

Closed benbr8 closed 2 years ago

benbr8 commented 2 years ago

It seems to be verilog (vlog) only, I checked the documentation.

INFO     cocotb:simulator.py:281 #  vcom -mixedsvvh -incr -work some_lib  /path/to/cocotb-test/tests/dff.vhdl
INFO     cocotb:simulator.py:281 # QuestaSim-64 vcom 2021.1_1 Compiler 2021.02 Feb 21 2021
INFO     cocotb:simulator.py:281 # ** Error (suppressible): (vcom-1902) Option "-incr" is not a valid option for this tool.

Was there any reason not using Simulator.outdated() for Questa, other than having a built-in function available?

themperek commented 2 years ago

Yes, this is a bug.

Was there any reason not using Simulator.outdated() for Questa, other than having a built-in function available?

In principle, it is better and it will check all dependencies also automatic included but it has many issues and I think this should be rewarded.

benbr8 commented 2 years ago

Issue was fixed in #166, however now Questa VHDL sources will be compiled every time, outdated or not.