search

xiph / awcy

http://arewecompressedyet.com/
MIT License
70 stars 46 forks source link

Allow building codecs from any reference #202

Closed rzumer closed 4 years ago

rzumer commented 4 years ago

As far as I can tell, there's no reason to restrict the field to a commit, since the reference is fed directly to git checkout. This allows users to enter a tag or a branch name instead.

Also removes unnecessary build steps for rav1e, since it no longer uses a submodule.

tdaede commented 4 years ago

It would be nice to block gerrit change IDs like I7b675ed36f147f7c0d3272a12bd24a12eedb285b

rzumer commented 4 years ago

Done, references starting with I and followed by 40 characters after trimming will be blocked.

tdaede commented 4 years ago

You mentioned in IRC there was a backend validation as well? I don't see a change to that here.

rzumer commented 4 years ago

I changed the pattern to match.