<@U020J2W6Q84> <@U016EM8L91B> noticed that the ope...
# openlanep
proppy
02/01/2022, 12:36 AM@User @User noticed that the open_pdks version requirements for
magicefabless/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.proppy
02/01/2022, 12:37 AMt
Tim Edwards
02/01/2022, 1:32 AMIt'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?)
proppy
02/01/2022, 4:22 AMI think it might be better to get openlane updated nonetheless (assuming they're fine with the newer version)
proppy
02/01/2022, 4:22 AMI'll draft a PR!
proppy
02/01/2022, 4:26 AMoh wait I forgot that openlane does pin open_pdk on a specific version, so that must be the desired behavior
proppy
02/01/2022, 4:27 AMi.e: earlier version of open_pdk don't have that check
proppy
02/01/2022, 4:27 AMproppy
02/01/2022, 4:29 AMnow I'm curious if
open_pdks🌏 1
d
donn
02/01/2022, 12:59 PMLater versions of magic break things. I only support the sky130/open_pdks/magic combination I tested and know works.
donn
02/01/2022, 1:03 PMopenlane-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!