search

rust-av / dav1d-rs

libdav1d rust bindings
MIT License
40 stars 20 forks source link

Update for dav1d 1.0.0 #61

Closed aeikum closed 2 years ago

aeikum commented 2 years ago

It looks like dav1d is getting close to releasing 1.0.0. Probably should wait to merge until they officially release, but here's an update so dav1d-rs is ready when that happens.

I bumped the relevant version numbers in this commit, but I'm not sure how you like to manage those.

aeikum commented 2 years ago

I guess the CI config will need to be updated to point to 1.0.0, too. Let's see how they set up their repo after release.

lu-zero commented 2 years ago

Feel free to update it as well in the same commit :) Thank you a lot for the PR!

lu-zero commented 2 years ago

Does the 1.0.0 API has anything we want to expose, @sdroege?

aeikum commented 2 years ago

I've updated it to use the less verbose self syntax, remove &mut from getters, and tweak a couple more version strings.

aeikum commented 2 years ago

Hello. dav1d 1.0.0 has been released and tagged, so this should be good to go. I confirmed it still works for me. Not sure how to re-trigger the CI...

lu-zero commented 2 years ago

update your commit with the CI bits and force push it to your topic branch.

aeikum commented 2 years ago

OK. I think it was already all correct, I just updated the commit time on the last commit so I could force-push it.

lu-zero commented 2 years ago

I'll prepare the release during the weekend :)

aeikum commented 2 years ago

Great! After you've tagged, I'll open a MR for gst-plugins-rs, unless someone beats me to it :)