Commit Briefs
remove `got add' workaround (main)
earlier version of got would fail when trying to `got add' a file that was already added (or committed.) Since the error message was potentially scary, got_add tries to be smart and issue a `got info file' first to see whether the file can be added, and only then eventually issues `got add'. This whole dance is now moot. got doesn't warn for files already added. (also, this code could have been simpler from the beginning.) Noticed since the pipe trick in the got sub was failing on linux. Issue reported by Christoph Cremer, thanks!
use gpg --batch --yes instead of redirecting standard output
last time I couldn't find the right incantation to prevent gpg from prompting when the target file already exists.
don't decode the whole ARGV, just the pattern for cmd_find
UNICODE is subtle, better not slightly tweaks the argument unless we really need them.
decode file names in the current locale
Otherwise we output gibberish and the pattern argument to `plass find' may not even match the entries. Issue reported by Anton Kasimov, thank you!
use File::Path' make_path instead of rolling my own
again from volker' plage.pl, thanks!
use extended regular expressions in `plass find'
from volker' plage.pl, thanks!
use foreach instead of map to print the matching entries
from volker' plage.pl
fix undefined behaviour reported by perlcritic
"return" statement followed by "sort" at line 140, column 2. Behavior is undefined if called in scalar context. (Severity: 5) the fix is the same as in volker' plage(1).
avoid issuing unnecessary `got add'
plass always did a `got add $file' before committing changes. However, this leads to a possibly misleading "file has unexpected status" error being printed to stderr. Instead, `got add $file' only when the file is new.
refactor cmd_edit
Now it always deletes the temporary file.
refactor cmd_tee: simplify the code
inline (with tweaks) writepass, it's redundant to have. While here also read/write blockwise instead that of linewise.
remove handling of PLASS_GOT environment variable
There's no real reason to have this variable, there's only one `got' usually and people are better off putting it somewhere in $PATH. Retain PLASS_GPG as there seem to be a little issue with gpg vs gpg2 so the knob may be useful in practice.