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 #163

Closed
wants to merge 3 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

@timlinux
Copy link
Member

Hi Etienne

Can you regenerate this merge request against current master?

Thanks

Tim

@etiennesky
Copy link
Contributor Author

ok - I guess it doesn't apply any more, will do!

@etiennesky etiennesky closed this Jun 15, 2012
@etiennesky
Copy link
Contributor Author

see pull request 177

yjacolin pushed a commit to yjacolin/QGIS that referenced this pull request Oct 4, 2014
User manual screenshots for Plugins
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.

None yet

2 participants