1.
The "p5 revert" clean up of abandoned files is best-effort.
It shouldn't abort if an unlink fails.
2. "p4 shelve" is too verbose for "p5 share" to just echo unfiltered.
But throwing it out isn't okay either, since it may contain a useful
error message. Compromise: strip out the lines that we're expecting,
and don't care about.