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@xxxxxxxxxx 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. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review