search

VUnit / vunit

VUnit is a unit testing framework for VHDL/SystemVerilog
http://vunit.github.io/
Other
735 stars 263 forks source link

OSVVM update #1046

Open c-thaler opened 3 months ago

c-thaler commented 3 months ago

OSVVM is currently rev-tied to a revision that is over a year old. In the meantime, there was a nice feature added that comes in handy when implementing bus masters (DelayCoverageIDType in DelayCoveragePkg.vhd). It would be nice to have the rev-tie get bumped to the latest stable commit.

LarsAsplund commented 2 months ago

@c-thaler I've pushed an update but I have yet to test over all simulators before merging.

c-thaler commented 2 months ago

I will go one with the branch until it is merged. Thanks a lot!

LarsAsplund commented 2 months ago

Looks like NVC fails to compile this OSVVM version. @nickg do you know anything about this?

nickg commented 2 months ago

Looks like NVC fails to compile this OSVVM version. @nickg do you know anything about this?

This was reported and fixed in nickg/nvc#917. Can you update the CI to use the 1.13.1 release?

LarsAsplund commented 2 months ago

Will do. Thanks!

LarsAsplund commented 2 months ago

@nickg The latest NVC fixed the problem but there is a new ambiguity problem. I'm on my cell phone so I haven't been able to test any workaround. Is that really ambitious?

nickg commented 2 months ago

No it's a bug, I'll fix it this weekend. Sorry about that.

nickg commented 2 months ago

No it's a bug, I'll fix it this weekend. Sorry about that.

This is fixed in the latest 1.13.2 release.

LarsAsplund commented 2 months ago

Thanks @nickg, I will rerun the test once the new NVC image is uploaded.