search

raymanfx / libv4l-rs

Video4Linux2 bindings for Rust
MIT License
152 stars 65 forks source link

Maintenance #50

Closed spebern closed 2 years ago

spebern commented 2 years ago

Hi there,

I wanted to ask if this crate is still maintained and if you do not have the time to maintain it anymore could add a contributor who can review and merge the pull requests and publish new versions on crates.io.

This crate is really useful and in my opinion the best wrapper around v4l for rust currently.

I'd be happy to help here!

MarijnS95 commented 2 years ago

I was about to ask the same, it seems @raymanfx hasn't contributed anything to GH since the end of July. I hope everything is alright!

However, it'd also be great if this crate can move on. Ie. get build-fixes like #43 in, and a new release published. I don't use this crate on a daily basis but wouldn't mind stepping in as an occasional reviewer and maintainer as well for the little traffic that this already-in-great-state crate gets :+1:

EDIT: For completeness I started working on support for the v4l media API in https://github.com/MarijnS95/libv4l-rs/compare/media, and some other fixes. If activity on this repo picks up I might be able to continue that effort and open some more PRs :)

raymanfx commented 2 years ago

Hey there,

I am still actively maintaining this crate, but these days I am mostly working on the higher-level eye stack [1]. The reason why there has been no activity in this repo is that I did not have to make any more changes for my projects. You are absolutely correct though that PRs like https://github.com/raymanfx/libv4l-rs/pull/43 deserve more attention; I am not sure how I 'forgot' about it to be honest. I'll try to give that a review today!

New features such as the media API support you mentioned are most welcome, too! Let me know once you want to get more stuff merged, or just open a PR and I'll review whatever I see coming my way.

[1] https://github.com/raymanfx/eye-rs

MarijnS95 commented 2 years ago

@raymanfx Thanks so much! I hadn't (still haven't) seen any activity on the eye-rs repo either and thought you were AWOL, glad to see and hear that's not the case!

Merging all the open PRs is much appreciated, I'll get on the case of continuing and completing media API support :). I presume notifications are reconfigured now though, so that we don't have to @ you directly?