search

OSVVM / OSVVM-Scripts

OSVVM project simulation scripts. Scripts are tedious. These scripts simplify the steps to compile your project for simulation
Other
8 stars 13 forks source link

Arguments passed by OptionalCommands to the simulator are not working as expected #6

Closed JimLewis closed 3 years ago

JimLewis commented 3 years ago

Submitted to me via email. Logging here for tracking:

The second issue concerns the same tcl file / tcl proc (vendor_simulate). It takes a third argument, OptionalCommands, But I am not sure how to use it. The problem is that I cannot pass more arguments separated by space:

simulate file.vhd -suppress 1234        (-> fails because too many args)
simulate file.vhd "-suppress 1234"     (-> fails, see below)
simulate file.vhd {-suppress 1234}     (-> fails, see below)

vsim always fails with complaining that "-suppress 1234" is not a valid option. I think the problem is that "-suppress 1234" is somehow interpreted as single argument, not as actually two arguments. When I call vsim directly in the console, and put those arguments inside double-quotes, it doesn't work as well.

JimLewis commented 3 years ago

Thanks. That is a bug in the script. It looks like I missed the eval before the vsim in VendorScripts_Mentor.tcl:

eval vsim -voptargs="+acc" -t $::SIMULATE_TIME_UNITS -lib ${LibraryName} ${LibraryUnit} ${OptionalCommands} -suppress 8683 -suppress 8684 -suppress 8617

You will find the above updates in the dev branch of Scripts.

JimLewis commented 3 years ago

Fixed in the current release