search

freechipsproject / chisel-testers

Provides various testers for chisel users
Apache License 2.0
98 stars 50 forks source link

Polyfill formerly deprecated now removed APIs #322

Closed jackkoenig closed 2 years ago

jackkoenig commented 2 years ago

Many APIs that have been deprecated in firrtl and chisel3 for a long time are removed in anticipation of the X.5 release. This project is now on life support--users should switch to using chiseltest (aka chisel-testers2).

There is a limited compatibility layer for iotesters.PeekPokeTesters, see https://github.com/ucb-bar/chisel-testers2/pull/348.

These are only the changes required to get this passing with master of chisel3 and firrtl. I intend to follow this PR with a PR deprecating as many APIs in this repo as possible and pointing to chiseltest. It would be helpful to have at least some doc on updating code.

ekiwi commented 2 years ago

Seem like you will have to fix:

 Error:  /home/runner/work/chisel-testers/chisel-testers/src/test/scala/chisel3/iotesters/BlackBoxVerilogDeliverySpec.scala:89:71: value fileListName is not a member of object firrtl.transforms.BlackBoxSourceHelper
Error:          manager.targetDirName, firrtl.transforms.BlackBoxSourceHelper.fileListName).exists() should be (true)
jackkoenig commented 2 years ago

Done!