efabless / caravel_user_project

https://caravel-user-project.readthedocs.io
Apache License 2.0
186 stars 329 forks source link

Build script deletes contents of OPENLANE_ROOT #186

Open xobs opened 2 years ago

xobs commented 2 years ago

Following the Quickstart README at https://github.com/efabless/caravel_user_project/blob/main/docs/source/index.rst#L87 I built and installed both openlane and a PDK into /opt/Si and set the environment variable OPENLANE_ROOT to /opt/Si/. I then cloned the caravel_example project and ran make setup.

During this, it deleted the contents of /opt/Si, including my intial OpenROAD installation.

It should be noted in the README that OPENLANE_ROOT is a temporary directory that WILL be deleted, and not a permanent directory where users should store their work.

cpacejo commented 5 months ago

This just bit me too, as I was trying to reinstall the PDK. make setup deleted my designs from OPENLANE_ROOT without prompting. Not a good user experience. Imagine if someone typoed OPENLANE_ROOT as their home directory.