Hi Carlos, Ummm, I looked at the comment in the Change Log. I haven't looked at the code (yet). I guess I didn't speak clearly (or think clearly either). I think that spice-sdb should handle *both* .end and .END -- i.e. be case insensitive -- not only one or the other.
Do you want me to make the change to support case insensitive .end? It seems like you have started the change, so I am happy if you prefer to do it (hint hint!) Stuart > > Hi Stuart (and all), > I wanted to send the first mail in this thread to the geda-dev mailing > list. Instead of that, I sent it to Stuart privately. Please see the > whole thread below. Sorry for the inconveniences. > > I applied the change and closed the bug. However, it would be better if > someone having spice3 could test it. Werner, you seem to be the original > submitter of the bug report. Can you test the fix? Do you know if there > is any other card expected to be in lowercase? > > Thanks, > > Carlos > > El lun, 10-04-2006 a las 16:44 -0400, Stuart Brorson escribi=C3=B3: > > Changing .END to .end (or vice versa) shouldn't cause a problem. I > > don't know if SPICE is supposed to be case sensitive or not, but I > > suspect not.=20 > >=20 > > Maybe if there are other case-sensitive things in the code, they > > should be fixed too. > >=20 > > Go ahead and make the changes. Don't forget to describe your change > > in the comment section at the beginning of gnet-spice-sdb.scm. > >=20 > > Thanks for doing this!!! > >=20 > > Stuart > >=20 > >=20 > >=20 > > >=20 > > > Hi Stuart and Dan, > > >=20 > > > I'm doing some bug hunting... > > >=20 > > > I noticed there are two bugs in the BTS regarding the spice-sdb and > > > cascade backends. > > > - The cascade bug is at: > > > (http://sourceforge.net/tracker/index.php?func=3Ddetail&aid=3D1442905= > &group_id=3D161080&atid=3D818426) It has a simple patch submitted by Dan = > some time ago. I don't know if it is up to date... > > >=20 > > > - The spice-sdb bug is at: > > > (http://sourceforge.net/tracker/index.php?func=3Ddetail&aid=3D1442912= > &group_id=3D161080&atid=3D818426) > > > It looks like changing the .END statement to .end will fix it. Will t= > his > > > create any other problem? > > >=20 > > > Based on your feedback, I can commit the patches into CVS and close t= > he > > > bugs... > > >=20 > > > Thanks, > > >=20 > > > Carlos > > >=20 > > >=20 > > >=20 > >=20 > >
