https://open-source-silicon.dev logo
p

proppy

02/01/2022, 12:36 AM
@User @User noticed that the open_pdks version requirements for
magic
seems to be different from the one shipped in
efabless/openlane:latest
Copy code
/open_pdks/sky130/sky130A/libs.tech/magic/sky130A.tech: line 19: section version:
	Error:  Magic version 8.3.260 is required by this techfile, but this version of magic is 8.3.209.
t

Tim Edwards

02/01/2022, 1:32 AM
It's not precisely a requirement because I added a check for the version of magic and implement a workaround for earlier versions. So I could dial back that requirement if needed.
p

proppy

02/01/2022, 4:22 AM
@User that does seems to make the open_pdk build fails for me, is that supposed to be not fatal? (or is there a way to override it?)
I think it might be better to get openlane updated nonetheless (assuming they're fine with the newer version)
I'll draft a PR!
oh wait I forgot that openlane does pin open_pdk on a specific version, so that must be the desired behavior
i.e: earlier version of open_pdk don't have that check
now I'm curious if
open_pdks
pins the same version of skywater-pdk as openlane does https://github.com/The-OpenROAD-Project/OpenLane/blob/aac68a0ca1de2ffe512d88f9e907f4fb89a61aed/dependencies/tool_metadata.yml#L100 🙂
🌏 1
d

donn

02/01/2022, 12:59 PM
Later versions of magic break things. I only support the sky130/open_pdks/magic combination I tested and know works.
openlane-bot checks the newest version of magic every 24 hours. You'll find that the CI is failing. https://github.com/The-OpenROAD-Project/OpenLane/pull/820
p

proppy

02/01/2022, 2:10 PM
@User thanks for the pointer!