Removed the "gds unique" option from the read-in of the openRAM
GDS, because (1) it became obvious that "gds unique" and "gds
readonly" options cannot go together without causing major issues,
and (2) the naming conflicts in openRAM that originally forced
the use of "gds unique" have been resolved and it should not be
necessary to use the option. That, however, needs confirmation.
diff --git a/VERSION b/VERSION
index 65854d7..996d476 100644
--- a/VERSION
+++ b/VERSION
@@ -1 +1 @@
-1.0.517
+1.0.518
diff --git a/sky130/custom/scripts/gds_import_sram.tcl b/sky130/custom/scripts/gds_import_sram.tcl
index af00660..ad4c35e 100644
--- a/sky130/custom/scripts/gds_import_sram.tcl
+++ b/sky130/custom/scripts/gds_import_sram.tcl
@@ -1,3 +1,12 @@
cif istyle sky130()
# Ensure unique cell names when reading multiple GDS files
-calma unique true
+#
+# Note 1: It is not possible to handle a hierarchy of modified
+# names when the names haven't been changed in the GDS without
+# adding a lot of bookkeeping to magic. Therefore "gds unique"
+# should not be used in conjunction with "gds readonly".
+#
+# Note 2: The original issue with name collisions was addressed
+# in openRAM, so this should not be needed anyway.
+#
+# calma unique true
