-
Notifications
You must be signed in to change notification settings - Fork 58
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
nextpnr crash: "terminate called after throwing an instance of 'std::out_of_range'" #136
Comments
Yes, this is expected. Until we release 0.1, you should always run nMigen against master branch of Yosys and nextpnr. The next version of Yosys, 0.9, is the current target, and I'll make sure every patch required for functionality is merged there.
Did you attach the repro? |
https://git.m-labs.hk/M-Labs/HeavyX/src/branch/master/examples/simplesoc_ecp5.py |
Can you zip the build directory and attach it here? |
That's a bug in nextpnr, I believe. |
See YosysHQ/nextpnr#301. |
@sbourdeauducq Looks like Nix breaks Versioneer:
Can you fix that? |
Yeah, that's because |
It does not, which you can check if you want. The reason is the gitattributes file, which causes the following block in def get_keywords():
"""Get the keywords needed to look up the version information."""
# these strings will be replaced by git during git-archive.
# setup.py/versioneer.py will grep for the variable names, so they must
# each be defined on a line of their own. _version.py will just call
# get_keywords().
git_refnames = "$Format:%d$"
git_full = "$Format:%H$"
git_date = "$Format:%ci$"
keywords = {"refnames": git_refnames, "full": git_full, "date": git_date}
return keywords to be turned into: def get_keywords():
"""Get the keywords needed to look up the version information."""
# these strings will be replaced by git during git-archive.
# setup.py/versioneer.py will grep for the variable names, so they must
# each be defined on a line of their own. _version.py will just call
# get_keywords().
git_refnames = " (HEAD -> master)"
git_full = "da1f58b7aef99912b32b65fe32cd72b61fd5dc34"
git_date = "2019-07-07 01:03:59 +0000"
keywords = {"refnames": git_refnames, "full": git_full, "date": git_date}
return keywords So nix should do the same thing then. |
Interesting, but even though this replacement does happen correctly in both cases, versionneer still breaks on GitHub archives downloaded and extracted manually and those in the Nix store.
|
Ah, sorry, my bad. I'll take a look at it myself then. |
This is also affecting ARTIQ (and leaving |
I'm not sure is this is actually related to nMigen or if this is purely a nextpnr bug, but HeavyX simplesoc_ecp5 no longer works after attempting to use
nmigen.build
.I had to update Yosys and nextpnr to get the new I/O system to work, otherwise Yosys would complain about an unknown cell
IB
.Now using Yosys YosysHQ/yosys@8455d1f and nextpnr YosysHQ/nextpnr@ff95883.
Repro (without Nix) and error:
The text was updated successfully, but these errors were encountered: