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.

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

setups/osflow/synthesis: recent versions of yosys need command 'read_verilog' #252

Merged
merged 1 commit into from
Jan 8, 2022

Conversation

@umarcor umarcor added bug Something isn't working CI Continuous integration-related labels Jan 8, 2022
@stnolting
Copy link
Owner

Awesome! Thanks for fixing!

@stnolting stnolting merged commit 8c4f2e0 into stnolting:master Jan 8, 2022
@umarcor
Copy link
Collaborator Author

umarcor commented Jan 8, 2022

Note that Windows CI is still broken. That is a different issue. I think some dependency of nextpnr is missing, so it crashes. I'm looking into it.

@umarcor umarcor deleted the fix-read_verilog branch January 8, 2022 11:24
@stnolting
Copy link
Owner

Note that Windows CI is still broken. That is a different issue. I think some dependency of nextpnr is missing, so it crashes.

I have notices that, but that issue has not yet reached "urgent level" on my to-do stack 😉

I'm looking into it.

Thanks!

@umarcor
Copy link
Collaborator Author

umarcor commented Jan 11, 2022

I have notices that, but that issue has not yet reached "urgent level" on my to-do stack 😉

Anyway, it's fixed now 😄. nextpnr-ecp5 is crashing arbitrarily (orangecrab and/or ulx3s), however, that is another issue... One step at a time!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working CI Continuous integration-related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

❌ MixedLanguage Implementation keeps failing
2 participants