Makefile: add YOSYS_VER and friends to CXXFLAGS for plugin compat che… #4757
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
As Yosys interfaces change, plugin developers may decide to stay compatible with a broad spectrum of versions. We will provide per-interface feature defines of some sort, but as a fallback in case we don't, I'm adding Makefile variable
YOSYS_VER
toCXXFLAGS
so thatyosys-config
reports reports it to the plugin sources when building them. It also splitsYOSYS_VER
toYOSYS_MAJOR
,YOSYS_MINOR
,YOSYS_COMMIT
which don't need quoting and can be used as integers. For example:This could also be the base for requiring plugins to report the version they were built against.