ipbus / ipbb

IPbus Builder Tool
GNU General Public License v3.0
12 stars 12 forks source link

Add simflags argument for HDL sources #141

Closed dinyar closed 3 years ago

dinyar commented 3 years ago

Hi,

In some cases it's quite useful to be able to pass special flags to Modelsim and Questasim. (e.g. at the moment I'm working around a dislike between TCDS2 sources and Modelsim where I need to downgrade an error to a warning with the -warning [err no] flag) With the --simflags argument one can now pass these options to Modelsim/Questasim.

Cheers, Dinyar

alessandrothea commented 3 years ago

Hi, thanks for the addition. We probably want a test to cover this case in the ci. I can explain how to add it later.

dinyar commented 3 years ago

Cool, will wait for the instructions to add this.

alessandrothea commented 3 years ago

So, the CI performs basic dep syntax/structure checks using tests/scripts/generate-ipbb-repo.py and the yaml files in tests/repogen. These files represent different test cases as dummy repos, composed of dep files (real) and source files (dummy). generate-ipbb-repo.py takes the yaml repo file as an argument, converts it in a directory structure, and runs depparser on the top-levels defined in the file. In the CI these tests are invoked by ci/tests.yml.

What is required is adding an existing example or adding a new one that contains a --simflags. The existing examples should be rather straightforward, but let me know if that's not the case.

dinyar commented 3 years ago

Hi again,

Thanks for the explanation! I added an example of a flag for Modelsim to the simple examples. Was that what you had in mind?

Cheers, Dinyar

alessandrothea commented 3 years ago

Yes, let's see how it works.

alessandrothea commented 3 years ago

Run tests.