Usage of -compat-int on yosys should depend on its version #14
Description
#13 did not account for the fact that older versions of yosys do not have the -compat-int option for write_json. This causes this extension to fail when an older version of yosys is used (in particular the version of yosys currently present on conda does not have this option).
The version of yosys should be checked to determine whether the -compat_int options should be added to write_json.
YosysHQ/yosys@20ce411 is the commit that introduced this option.
Metadata
Metadata
Assignees
Labels
No labels
Type
Projects
Milestone
Relationships
Development
No branches or pull requests
Participants
Activity
[-]Usage of -compat-int on yosys should depends on its version[/-][+]Usage of -compat-int on yosys should depend on its version[/+]mithro commentedon Apr 18, 2020
@daniellimws - I would say we just drop support for any version of Yosys that is too old? IE The minimum yosys version required is XXX where that is after the
-compat_intwas added?daniellimws commentedon Apr 19, 2020
Ok. There should be no problem with this once yosys in conda is updated.