Skip to content

Add 'gdal vfs list/copy/delete' #12132

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

Merged
merged 11 commits into from
Apr 14, 2025
Merged

Add 'gdal vfs list/copy/delete' #12132

merged 11 commits into from
Apr 14, 2025

Conversation

rouault
Copy link
Member

@rouault rouault commented Apr 12, 2025

  • Add 'gdal vfs list', alias 'gdal vfs ls' (replacement for gdal_ls.py sample script)
  • Add 'gdal vfs copy', alias 'gdal vfs cp' (replacement for gdal_cp.py sample script)
  • Add 'gdal vfs delete', alias 'gdal vfs rm' (replacement for gdal_rm.py sample script)

@rouault rouault added enhancement funded through GSP Work funded through the GDAL Sponsorship Program gdal_cli Anything related to the new 3.11 "gdal" CLI frontend labels Apr 12, 2025
@rouault rouault added this to the 3.11.0 milestone Apr 12, 2025
@jratike80
Copy link
Collaborator

Would it be too long as vfs list, stands for "virtual file system - list"?

@rouault
Copy link
Member Author

rouault commented Apr 12, 2025

I'm open for vfs instead of fs. The advantage of ls is that this is a standard Unix name (and the -l and -R flags are exactly the ones of ls). That said we have the concept of aliases for command names, so the long name could be list, with ls as alias

@coveralls
Copy link
Collaborator

coveralls commented Apr 12, 2025

Coverage Status

coverage: 70.585% (-0.006%) from 70.591%
when pulling 58b157d on rouault:gdal_fs
into 29aee5d on OSGeo:master.

@rouault rouault changed the title Add 'gdal fs ls' (replacement for gdal_ls.py sample script) Add 'gdal vfs list/copy/delete' Apr 13, 2025
@rouault
Copy link
Member Author

rouault commented Apr 13, 2025

Would it be too long as vfs list, stands for "virtual file system - list"?

done. As well as the addition of "copy" and "delete"

@rouault rouault force-pushed the gdal_fs branch 4 times, most recently from ae14566 to 9d0aff7 Compare April 13, 2025 19:36
@rouault rouault merged commit c98cbbc into OSGeo:master Apr 14, 2025
37 checks passed
@hobu
Copy link
Contributor

hobu commented Apr 14, 2025

Sorry I didn't see this until just now. I wonder if it should maybe be named gdal vsi {ls|delete|copy} to align it with everything else and to denote that it can and should take VSI links.

@rouault
Copy link
Member Author

rouault commented Apr 14, 2025

that it can and should take VSI links.

it can also accept regular paths. Can people watching this PR vote whether they prefer "vfs" or "vsi" ?

@jratike80
Copy link
Collaborator

They have 3 letters both but I do not understand what they really mean, so please select your favourite. +0 for both from me.

@msmitherdc
Copy link
Contributor

i guess vsi

@rouault
Copy link
Member Author

rouault commented Apr 14, 2025

ok let's go for vsi ==> #12142

@jratike80
Copy link
Collaborator

Finally found what VSI means - naturally "Virtual System Interface" https://lists.osgeo.org/pipermail/gdal-dev/2005-September/006426.html

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement funded through GSP Work funded through the GDAL Sponsorship Program gdal_cli Anything related to the new 3.11 "gdal" CLI frontend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants