CLI: use argparse, clearer logging logic

bagit-profiles / bagit-profiles-validator

A simple Python module for validating BagIt Profiles.

The Unlicense

12 stars 6 forks source link

Closed kba closed 5 years ago

kba commented 5 years ago

optparse is deprecated
logging logic was broken (logging to file could not be disabled / logging to console not be enabled)
version flag
support variant of constructor which doesn't require fetching URL
exits 1 for serialization error
exits 2 for validation error

It should be backwards-compatible to not break existing scripts.

ruebot commented 5 years ago

Thanks!! Good catch on the logging logic!