Note: This is a beta release of Red Hat Bugzilla 5.0. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Also email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback here.

Bug 230762

Summary: Review Request: xawtv - TV applications for video4linux compliant devices
Product: [Fedora] Fedora Reporter: Dmitry Butskoy <dmitry>
Component: Package ReviewAssignee: Jason Tibbitts <tibbs>
Status: CLOSED NEXTRELEASE QA Contact: Fedora Package Reviews List <fedora-package-review>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideFlags: tibbs: fedora-review+
notting: fedora-cvs+
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2007-06-25 14:05:03 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Dmitry Butskoy 2007-03-02 18:15:48 UTC
Spec URL:

Xawtv is a simple xaw-based TV program which uses the bttv driver or
video4linux. Xawtv contains various command-line utilities for
grabbing images and .avi movies, for tuning in to TV stations, etc.
Xawtv also includes a grabber driver for vic.

Additional info:
xawtv was in Fedora Core until FC2 (now tvtime is a base apps to watch TV).

Xawtv seems to be a first project capable to show TV, written by the same people as v4l drive.

Besides "xawtv" program itself, "xawtv" package includes a lot of another useful things: command line tools, console radio apps, frame buffer tv, teletext web daemon etc. These "addional tools" are the main reason to package xawtv for Fedora.

This package includes ability to write avi files with motion jpeg video, as well as raw "YUV 4:2:0" called "yuv4mpeg". All those seem to have no legal issues (avi support is already in Fedora (dvgrab, xine-lib), mjpeg and yum4mpeg are just a stream of images (jpeg and raw YUV respectively)).

Another distros like to split xawtv to several subpackages. I dislike that, because initial Fedora's xawtv was a single package, there is no common agreement how to split (various distos do it differently), splitting itself seem to be unuseful (even some pure cmdline tools require X11 libs etc.)

Comment 1 Tom "spot" Callaway 2007-06-13 19:10:10 UTC
I see no legal problems with mjpeg or raw YUV.

Comment 2 Jason Tibbitts 2007-06-21 01:06:50 UTC
rpmlint has only one complaint:
  W: xawtv mixed-use-of-spaces-and-tabs (spaces: line 65, tab: line 84)
No big deal; fix it if you like.

The Source0: URL doesn't seem to be valid.  Trying to fetch it I just end up getting an index.html page.  This seems to work better for me:{name}-%{version}.tar.gz

You aren't using the dist tag.  It's not a requirement, but I always ask packagers if they understand the issues that can occur if they don't.

The compiler seems to be called only with -Wno-pointer-sign instead of the usual compliment of flags.  This breaks the debuginfo packages, among other things.

I guess this is a GUI application (or at least some of the executables are) and is thus supposed have appropriate desktop files so that things show up in the desktop menu properly.

* source files match upstream:
* package meets naming and versioning guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* summary is OK.
* description is OK.
* build root is OK.
* license field matches the actual license.
* license is open source-compatible.
* license text included in package.
* latest version is being packaged.
* BuildRequires are proper.
X compiler flags are incorrect.
* %clean is present.
* package builds in mock (development, x86_64).
* package installs properly
X debuginfo package is incomplete.
* rpmlint has only acceptable complaints.
* final provides and requires are sane:
   config(xawtv) = 3.95-0
   xawtv = 3.95-0
   config(xawtv) = 3.95-0

* %check is not present; no test suite upstream.  I haven't the hardware to be 
  able to do any testing, so I'm only reviewing the packaging.
* no shared libraries are added to the regular linker search paths.
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* no scriptlets present.
* code, not content.
* documentation is small, so no -docs subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.
* no headers.
* no pkgconfig files.
* no static libraries.
* no libtool .la files.
X GUI app, but no desktop files.

Comment 3 Dmitry Butskoy 2007-06-21 11:37:10 UTC
> The Source0:

> dist tag.
Oops... Added.

> The compiler seems to be called only with -Wno-pointer-sign
Thanks, added.

> a GUI application is thus supposed have appropriate desktop files
Hmmm... /usr/share/applcations/fedora-xawtv.desktop is already present...


Comment 4 Jason Tibbitts 2007-06-22 04:58:35 UTC
Looks good.  I don't know how I missed the desktop file there; the only comment
I have about it is that we don't use X-Fedora any longer.

I'll go ahead and approve and you can remove that bit when you check in.


Comment 5 Dmitry Butskoy 2007-06-22 11:30:21 UTC
New Package CVS Request
Package Name: xawtv
Short Description: TV applications for video4linux compliant devices
Branches: FC-6 F-7

Comment 6 Kevin Fenzi 2007-06-22 19:27:07 UTC
cvs done.

Comment 7 Dmitry Butskoy 2007-06-25 11:16:14 UTC
> I have about it is that we don't use X-Fedora any longer.
It is added by "--vendor fedora" in "desktop-file-install", according to

Comment 8 Ville Skyttä 2007-06-25 17:05:56 UTC
(In reply to comment #7)
> > I have about it is that we don't use X-Fedora any longer.
> It is added by "--vendor fedora" in "desktop-file-install"

Really?  That would be a strange desktop-file-install bug.  I find your use of
"--add-category X-Fedora" a much more likely candidate for why it's there... :)

Comment 9 Dmitry Butskoy 2007-06-26 12:12:39 UTC
It seems you managed to convince me. :)

Comment 10 Dmitry Butskoy 2011-02-01 17:25:36 UTC
Package Change Request
Package Name: xawtv
New Branches: el5 el6

Comment 11 Jason Tibbitts 2011-02-01 17:39:53 UTC
Owners and initialCC fields must contain FAS account names, not email addresses.

Comment 12 Dmitry Butskoy 2011-02-01 17:53:12 UTC
Package Change Request
Package Name: xawtv
New Branches: el5 el6
Owners: buc
InitialCC: mchehab

Comment 13 Dmitry Butskoy 2011-02-02 16:29:11 UTC

Comment 14 Bill Nottingham 2011-02-02 22:29:11 UTC
Git done (by process-git-requests).