[Bug 445338] Review Request: gstreamer-plugins-flumpegdemux - MPEG demuxer for GStreamer

[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 report.

Summary: Review Request: gstreamer-plugins-flumpegdemux - MPEG demuxer for GStreamer


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





------- Additional Comments From mclasen at redhat.com  2008-05-06 11:42 EST -------
Formal review:

package name: ok, considering we've done the gst -> gstreamer expansion
   consistently for all gstreamer packages in the distro
spec file name: ok
packaging guidelines: just add the url tag and you are fine
license: ok
license field: the official tag is MPLv1.1 I think, it should be used
licence file: ok
spec language: ok
spec legibility: excellent
upstream sources: ok
builds: yes
excludearch: n/a
build deps: ok
locale handling: ok
ldconfig: ok
relocatable: n/a
directory ownership: ok, since it requires gstreamer
%files: ok
file permissions: ok
%clean: ok
content: permissible
large docs: ok
headers: ok
static libs: ok
pc files: ok
shared libs: ok
devel package: n/a
la files: ok
gui app: n/a
file ownership: ok
%install: ok

Summary:
- fix license tag
- add url tag




-- 
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, or are watching someone who is.