parse_arguments - Githubissues

posborne / putio-sync

Python daemon that automatically downloads files from put.io

MIT License

29 stars 12 forks source link

parse_arguments #15

Open seanieb opened 10 years ago

seanieb commented 10 years ago

The "-k" argument is confusing. It's not intuitive what is the default action actually is. I would recommend that the default is always to keep a copy on Put.io's servers after download and the user has to be explicit to delete from put.io after download.
Another argument that would be worth having would be -n --newonly. Don't download existing items, only download new items. Ala Put.io's RSS feed feature "Don't process the current feed, only download new items".

posborne commented 10 years ago

Both of these requests seem reasonable. The -k change should be relatively straightforward and isolated to argument parsing.

Downloading only new items (-n) is also possible but requires some additional changes in the core logic. The assumption would be that this option would need to be passed each time the daemon is started in order to avoid downloading items already present.