Skip to content
This repository has been archived by the owner on Nov 4, 2021. It is now read-only.

make yosys version string compatible with what nmigen expects #33

Closed
edbordin opened this issue Jun 29, 2020 · 1 comment · Fixed by #35 or #36
Closed

make yosys version string compatible with what nmigen expects #33

edbordin opened this issue Jun 29, 2020 · 1 comment · Fixed by #35 or #36

Comments

@edbordin
Copy link
Collaborator

Currently because I changed the version string format, nmigen can't parse it and ignores our yosys build. Standard yosys version string is something like 0.9+3292 which can be updated with make bumpversion

@edbordin
Copy link
Collaborator Author

I'll revise this so that it doesn't run bumpversion as per whitequark's advice in YosysHQ/yosys#2207. Will also pass GIT_REV in to the yosys makefile since it runs after .git has been stripped away (strangely git rev-parse seems to be outputting something anyway).

edbordin added a commit that referenced this issue Jun 30, 2020
edbordin added a commit that referenced this issue Jun 30, 2020
edbordin added a commit that referenced this issue Jun 30, 2020
edbordin added a commit that referenced this issue Jun 30, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
1 participant