On Sun, May 6, 2018 at 7:17 PM, brian m. carlson
<sandals@xxxxxxxxxxxxxxxxxxxx> wrote:
> This series introduces an SHA1 prerequisite which checks if the hash in
> use is SHA-1, and can be used to skip the test if it is not.
> Additionally, because NewHash will be 256-bit, I introduced aliases for
> the test constants $_x40 and $_z40 which will be less confusing when the
> hash isn't 40 hex characters long. I opted to leave the old names in
> place for the moment to prevent any potential conflicts with other
> series and will clean up any stragglers later.
>
> Several tests are skipped because of SHA-1-specific dependencies: some
> of these are core tests which test basic expected hash values, some
> depend on colliding short names, and some depend on specially named
> object (the pack tests).
Was I wrong to expect this series to annotate[1] tests
t3404 "short SHA-1 setup"
t3404 "short SHA-1 collide"
with the SHA1 prerequisite?
[1]: https://public-inbox.org/git/CAPig+cR==SNfGdhwqPdvW75fUxXg-VSQ5Tz_OR7Sy_c0L94axQ@xxxxxxxxxxxxxx/T/#m7bb98bd57a3189bb5fe01993b22b0c480a601259
