TinyTapeout / analog-virtualbox-vm-sky130a

Virtual Machine for analog with the open source Sky130A PDK
Other
29 stars 7 forks source link

xschem needs a HEAD version for compatibility with selected PDK version #3

Closed algofoogle closed 5 months ago

algofoogle commented 6 months ago

Current manifest includes xschem 3.4.4 but I've noticed that the chosen recent PDK version bdc9412 relies on a newer (pre-release) version of xschem 3.4.5 in order to fully support this (e.g. to get all graphs working in the test_env example).

Existing guides don't differentiate currently and just go with the HEAD commit (latest in master). I guess we could set XSCHEM_VERSION to master or a known good that is recent enough (currently 659231e7ae06c745caa6e1776cceca0c1afe7322 seems fine?)

(Unimportant, but I also just noticed a tiny typo in the install_xschem.sh script with the keyword "simualtion")

urish commented 6 months ago

Let's go with a specific commit - we want reproducible builds :) Thanks!

urish commented 6 months ago

@StefanSchippers any plans to cut 3.4.5 sometime soon?

StefanSchippers commented 6 months ago

I will work on this. Will restrict commits to bug-fix only and if no regression pops up will tag 3.4.5

urish commented 5 months ago

@mattvenn implemented a workaround at e0e6790