[Bug 576023] Review Request: libwebcam - user-space configuration of the uvcvideo driver

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=576023

--- Comment #11 from Michael Cronenworth <mike at cchtml.com> 2010-07-12 20:58:29 EDT ---
(In reply to comment #10)
> Hi Michael,

Hi Hans, wow big reply! I will try to digest it.

> Oh and since circa 2 years I work for RedHat now, but that is of little
> relevance in this case, this is me with my Fedora hat firmly on :)

Yes, I am familiar with who you are. Nice to see you here.

> We want the libwebcam package to install udev rules to automatically run
> uvcdynctrl to tell the driver about these controls when a relevant webcam gets
> plugged in (or is plugged in wen the system boots up). I see that you're
> current package is missing uvcdynctrl and the udev rules, so that will be a
> first thing to fix. It is probably best to base your work for this on top of
> the work Paulo has done for Debian, instead of:
> # svn export -r 90 http://svn.quickcamteam.net/svn/qct/webcam-tools/trunk/
> libwebcam-0.2.0
> do:
> # svn export -r 90
> http://svn.quickcamteam.net/svn/qct/webcam-tools/branches/deb libwebcam-0.2.0

I was waiting[1] on upstream to merge in the patches for this, but it looks
like it won't happen any time soon. I'll add it to my spec.

> And I have matching patches for libwebcam / uvcdynctrl for this. I've send
> these to libwebcam upstream, but unfortunately libwebcam upstream is not really
> fast with processing these, so we will need to add these to the Fedora
> libwebcam package for now.

Are these in the kernel already?

> 1) You should add uvcdynctrl + udev rules to the libwebcam package, it is best
> to base your work on the Debian branch for this, to get the Debian branch do:
> svn export -r 90 http://svn.quickcamteam.net/svn/qct/webcam-tools/branches/deb
> libwebcam-0.2.0

New spec: http://michael.cronenworth.com/RPMS/libwebcam.spec
SRPM:
http://michael.cronenworth.com/RPMS/libwebcam-0.2.0-0.3.20100322svn.fc13.src.rpm

> 
> 2) I would like to see some patches added to the Fedora package which add
> support for menu and button controls, but lets first just tackle 1). If you can
> provide an updated spec file and srpm with 1) fixed I'll do a more thorough
> review of that.

I'll be more than happy to work with you on those patches.

> An important part of the getting sponsored process is showing that you've got
> the necessary packaging skills and knowledge of the packaging guidelines as
> mentioned already in comment #8 one way of doing this is doing reviews of other
> packages which are awaiting review (when you do this please add a note you are
> not doing an official review).

I haven't gotten to do some unofficial reviews yet.

> 
> An other option is to show your skills by creating a couple of packages. Given
> that your current libwebcam package only packages the library itself, I'm
> guessing that you are packaging libwebcam because it is needed by something
> else you plan to package. You could for example also submit that for review (it
> is ok to have multiple review requests outstanding while waiting for
> sponsoring). Or if I may suggest something you could package v4l-utils, the
> successor of libv4l:
> http://www.linuxtv.org/downloads/v4l-utils/
> http://cvs.fedoraproject.org/viewvc/devel/libv4l/libv4l.spec?view=co

I'll let you handle that, if you wish. I am going to package uvcdynctrl plus my
own app, uvcdynctrl-gtk (pretty obvious what that is). I wanted to start with
libwebcam first, but if having those two packages in review as well will help
this process along, I'll add them. I'll start with uvcdynctrl. My app needs a
.desktop and icon file, which I have not made yet.

Bug 613857

> 
> Regards,
> 
> Hans    

Thanks for the pointers!

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.