Thanks for the patch! I would however like to change it a bit. Dependencies listed in the main section are always pulled in, but since vlog_tb_utils is only used during simulations, I would like to list it in the simulator-specific sections instead. This is slightly more awkward as we need to do this for all supported simulators, but it helps a bit with avoiding potential problems when synthesizing. I also realize that this behaviour isn't really documented anywhere.
So to sum it up, the proper way would be to remove vlog_tb_utils from the depend list in main and add:
...
[icarus]
depend = vlog_tb_uitls-1.0
[isim]
depend = vlog_tb_uitls-1.0
[modelsim]
depend = vlog_tb_uitls-1.0
[xsim]
depend = vlog_tb_uitls-1.0
...
I'm ok to pull the patch as it is however (it fixes an issue). Just mentioning the above for future reference
Thanks for the patch! I would however like to change it a bit. Dependencies listed in the main section are always pulled in, but since vlog_tb_utils is only used during simulations, I would like to list it in the simulator-specific sections instead. This is slightly more awkward as we need to do this for all supported simulators, but it helps a bit with avoiding potential problems when synthesizing. I also realize that this behaviour isn't really documented anywhere.
So to sum it up, the proper way would be to remove vlog_tb_utils from the depend list in main and add:
... [icarus] depend = vlog_tb_uitls-1.0
[isim] depend = vlog_tb_uitls-1.0
[modelsim] depend = vlog_tb_uitls-1.0
[xsim] depend = vlog_tb_uitls-1.0 ...
I'm ok to pull the patch as it is however (it fixes an issue). Just mentioning the above for future reference