The problem is that it is not possible to import Address Books from Eudora if Eudora is using the '.nnt' file format. The Penelope importer does not recognize this file format. To reproduce: - Set Eudora to hide Address Book files from simple viruses - Attempt to import the Address Book into Penelope - Note error in Penelope: "No address books were found to import".
Created attachment 260603 [details] [diff] [review] Patch that fixes bug. Note patch was not created with CVS.
Comment on attachment 260603 [details] [diff] [review] Patch that fixes bug. Note patch was not created with CVS. I'll try to port this to the trunk after I've finished getting rid of nsFileSpec...
Standard8 points out that the patch has tabs in it - in general, we want spaces instead of tabs, and a 2 or 4 space indent, depending on the prevailing code. (Yeah, I realize that the import code is *full* of tabs, but we don't want to add more) Since I'm going to have to apply this patch by hand, it's not such a big deal, but in the future
We can chat in #penelope or via AIM tomorrow (Thurs.) regarding this. I realize that typically spaces would normally be used, but assumed that since the importer code has almost no use of spaces (i.e. tabs are pretty much all that is used) I shouldn't introduce a few lines with spaces (mixing tabs and spaces can be ugly depending on the settings of the person viewing your code). Sorry for the incorrect assumption and trouble here.
Created attachment 263805 [details] [diff] [review] fix I'm going to check in This is what I'm going to land on the trunk - it uses nsIFile instead of nsfilespec.
fixed on trunk