| Subject: | 'shipwright update' should report failure if it fails to update |
I added DateTime::Locale to my Shipwright repository, only to find that
I actually needed an older version. I thought I could do:
shipwright update -r fs:`pwd` cpan-DateTime-Locale --version 0.42
...to revert to v0.42, and upon doing this shipwright told me:
CPAN related output will be at /tmp/shipwright_cpan.log
Fetching with LWP:
http://cpan.mirror.facebook.com/authors/id/D/DR/DROLSKY/DateTime-Locale-0.42.tar.gz
importing cpan-DateTime-Locale
imported with success
updated with success
When I went to check the changes (this repository is stored in git),
there were none. So I added logging:
shipwright update -r fs:`pwd` -l info cpan-DateTime-Locale --version 0.42
When this finished, it told me:
path scripts/cpan-DateTime-Locale alreay exists, need to set overwrite
arg to overwrite
imported with success
updated with success
I'm of the opinion that this is not a success, this is actually a
failure to import this module. Shipwright should report failure to
import when it doesn't import.