search

dropbox / dbxcli

A command line client for Dropbox built using the Go SDK
Other
1.06k stars 99 forks source link

Add --recursive to the rm command #31

Closed charignon closed 7 years ago

charignon commented 8 years ago

I tested with the following folder/file structure, Junk and Junk2 are folders, aa and aaa are files:

Junk ├── Junk2 │   └── aaa └── aa

./dbxcli rm Junk # Refuses to remove a folder Error: rm: cannot remove ‘/Junk’: Is a directory

./dbxcli rm -r Junk # Works and remove the folder recursively

charignon commented 8 years ago

So we are good to go with this PR?

diwakergupta commented 8 years ago

@charignon I added some more comments. FYI I pushed some major changes to dbxcli as well as the Go SDK, so you'll have to rebase regardless of how we proceed. Sorry about that!

grantseltzer commented 8 years ago

Since #43 was merged, this can be closed.

diwakergupta commented 7 years ago

The --force flag to rm should handle this now, so closing the PR. Feel free to file an issue if needed.