Checking the length of a hash about bake HOT 2 CLOSED

Thanks for the report.

1. I am expecting all git SHA1's to be the full 40 element hash, and expecting that if people want short SHA1 options they will do it themselves as a wrapper script on top. In my implementation we have something that resolves using git rev-parse, defaulting to HEAD if nothing is specified. I could allow shorter hashes, but I'd prefer to always work with full hashes in the app and shorter hashes outside. That said, I'm on the fence as to whether to allow shorter hashes or not.

2. Retry doesn't work for identical patch names, so I prepend a ' to them. Using the ovenPretty helper you write something like neil=ccc... (where neil is the branch name), and then retry makes it 'neil=ccc..., which is still valid. Dropping leading ' from the sha1 seems like the right thing to do.

So both of these could be fixed, and the second almost certainly should, but both are easily worked around.

from bake.

I changed the docs, and your code in #17 and #18 fixed the other, so this is all now sorted.

from bake.