Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Vsifile - more fixes : bugs and tar support + file open #177

Closed
wants to merge 4 commits into from

Conversation

etiennesky
Copy link
Contributor

This pull request is to fix to issues but also enhances both zip and tar file support

3c132f2 remove Passthru option and change storage of options scanZipInBrowser and scanItemsInBrowser

The Passthru option is confusing and unnecessary - because zip reading is no longer dependent on quazip

7ed8415 add support for .tar/.tgz files and relevant tests ; delay scan of .tgz files and large .zip files until requested

I found that opening a directory with large .zip files was really slow (like minutes), so this patch does not scan for contents of large zip files (>10 items), and as a bonus I added .tar and .tgz support. For performance reasons .tgz files are never scanned until requested (i.e. when the item is opened by the user).

9915c5a add zip item layer dialog to open .zip and .tar files with many files ; fix gdal sublayer names

This adds support for opening files from the regular file Open dialogs. A popup appears, similar to the Choose OGR/GDAL Layers dialog.

I also shortened the item names of gdal files with subdatasets, which could be very long (e.g. NETCDF:"longfilename.nc":variable) and were truncated (e.g. NETCDF:"longfilename.nc), because of changes in layer name for bug #5636

171799f user layerName() when adding raster from browser (#5636)
somehow this didn't make it up to here... re-applied

@etiennesky
Copy link
Contributor Author

This is basically pull request 163 from recent master

If you want a netcdf and/or hdf4 file to test against, I can provide a link or send one

@jef-n
Copy link
Member

jef-n commented Jun 16, 2012

patch applied in master

@jef-n jef-n closed this Jun 16, 2012
@etiennesky
Copy link
Contributor Author

thanks Jef!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants