Surelog
Surelog copied to clipboard
Please allow to use external dependencies instead of bundling them
These are: antlr4, UHDM, flatbuffers, googletest.
Can you motivate your ask? What problem are you facing? The only optional is antlr4, you can switch to the official antlr4 (See CMakeLists.txt).
But it looks like you should use the pre-packaged Surelog here that does work as a front-end to Yosys: https://antmicro.com/blog/2021/10/openlane-asic-build-flow-with-systemverliog-support/
Stable dependencies like flatbuffers
should be used through external packages for these reasons:
- In case security vulnerabilities would be later found in them - they should be fixed in one place, not in many bundled copies.
- Maintenance costs are lower when there's no bundling - compilation times are shorter, no need to update hashes/versions of bundled packages with each version update.
- In case the package requires/would require patches - they would be done in one place, vs. in all bundled copies.
- Bundling wastes CPU resources - compilation time grows exponentially with bundling because same software is recompiled again and again.
Can you submit a pull request?
Fixed with #3498