verify_sdp.sh should summarise errors at the end
It would be nice to finish by grepping the resulting log file for Error - something users otherwise has to do because errors are often hidden in the output.
That version was in review but never made it to the dev branch. It had a beautiful summary at the end. Alas, the beautiful summary was tightly bound to the too-slow change-by-change algorithm that was abandoned. As per comments in that review, even though change-by-change verification wasn't sufficiently performant (even when done in parallel), the nice summary is worth another look.
The improved summary will also come with other useful details, like per-depot verification durations (not just start/end times) that are an easy read.
Aye!
FYI, we almost had this. See: https://swarm.workshop.perforce.com/reviews/27662#comments
That version was in review but never made it to the dev branch. It had a beautiful summary at the end. Alas, the beautiful summary was tightly bound to the too-slow change-by-change algorithm that was abandoned. As per comments in that review, even though change-by-change verification wasn't sufficiently performant (even when done in parallel), the nice summary is worth another look.
The improved summary will also come with other useful details, like per-depot verification durations (not just start/end times) that are an easy read.