Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Should nMigen check Yosys version? #55

Closed
mithro opened this issue Apr 1, 2019 · 2 comments
Closed

Should nMigen check Yosys version? #55

mithro opened this issue Apr 1, 2019 · 2 comments
Labels
backend:RTLIL improvement upstream
Milestone
0.1

Comments

@mithro
Copy link

mithro commented Apr 1, 2019

nMigen has a pretty strong dependence on Yosys and you have been fixing a lot of bugs in it as #53 shows.

Maybe nMigen should check the Yosys version is new enough?
@whitequark
Copy link
Contributor

Yes, but we can only pull that off after there's a release that includes the recent fixes. Unless you want to rely on the 0.8+123 notation...

@whitequark
Copy link
Contributor

Triage: check will be added once there is a Yosys release.

@whitequark whitequark added this to the 0.1 milestone Jun 28, 2019
whitequark added a commit that referenced this issue Aug 19, 2019
back.verilog: parse output of yosys -V.
5ad409e 
See #55.
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
backend:RTLIL improvement upstream
Projects
None yet
Milestone
0.1
Development

No branches or pull requests
2 participants
@mithro @whitequark