shp2pgsql-gui crashes when given filenames with dots

[robe]

I'm not sure if it's the length of the file or because there is an extra ., I suspect its the dot that is the problem.

I downloaded this file: ​http://osm-extracted-metros.s3.amazonaws.com/taipei.osm2pgsql-shapefiles.zip

And extracted them and then tried to load up the

taipei.osm-line one (using table name osm_line in a schema taipei) and was rudely awoken from my daze it claimed it couldn't find the corresponding .dbf file and then promptly crashed.

After various renamings of all the files, I was able to successfully load by renaming

taipei.osm-line.* to: osm_line.*

I haven't tried with the command line, since I have since forgotten how to use a command line and too lazy to relookup the instructions.

I'm using the 2.1.0 version 64-bit version on my windows 7 64-bit. I'll try to check next if its the same issue with the 32-bit.

[robe]

Same issue with shp2pgsql except shp2pgsql doesn't crash, just gives a can't locate .dbf. However with shp2pgsql I can work around the issue by explicitly referencing the .dbf file and weirdly it has not problems finding the companion .shp file.

If I try to do the same with shp2pgsql-gui (pointing it at the dbf instead of shp)and it STILL complains it can't find the .dbf and crashes.

[pramsey]

Both GUI and CLI carefully strip off the trailing .shp extension by walking from the end back to the first '.'

This is fine, except then shpopen tries the same trick on the string *it* receives.

I think the solution is for our code to stop trying to strip the extension and leave it to shpopen to do.

[mcayland]

There are actually 2 bugs here, and I'm just about to commit a fix for them in the next few minutes…

[mcayland]

Okay - I've fixed the crasher and the GUI in trunk r11292 and 2.0 branch at r11293. Paul - your observation is absolutely correct about the .shp extension, however it's going to require a little bit more thought as the result is reused for the table name. Let me see if I can untangle the CLI in a slightly more sensible way.

[mcayland]

I've committed a fix for this as r11294 for trunk and r11295 for 2.0 branch. Please test and feel free to close if it works for you.

[robe]

okay that seems to work for shp2pgsql-gui on trunk. Haven't retried shp2pgsql.

I was able to make it crash again by selecting filter to .dbf and doing something stupid like load in geography (even though I also checked only dbf). I can't replicate the crash conditions though so I'm going to close this out. I'll opne a separate ticket or reopen this one if I discover other issues.

Would be nice if we had a regress test to stress these unusual file names though.