Spec URL: http://www.gnome.org/~hadess/gst-fluendo-mpegdemux/gstreamer-plugins-mpegdemux.spec SRPM URL: http://www.gnome.org/~hadess/gst-fluendo-mpegdemux/gstreamer-plugins-flumpegdemux-0.10.15-1.fc8.src.rpm Description: This package contains an MPEG demuxer for the GStreamer multimedia framework. It allows to separate audio and video in MPEG streams, so that they can be decoded.
FYI, I'll fix the spec file naming before uploading.
Original review request: http://bugzilla.livna.org/show_bug.cgi?id=1523
I can confirm that this is fine legal wise, I just got the following from Spot: "Looks fine, demuxing is ok, just not encode/decode."
Builds ok in mock rpmlint output: gstreamer-plugins-flumpegdemux.i386: E: zero-length /usr/share/doc/gstreamer-plugins-flumpegdemux-0.10.15/README gstreamer-plugins-flumpegdemux.i386: W: invalid-license MPL 1.1 gstreamer-plugins-flumpegdemux.i386: W: no-url-tag ...and its right ! Those should be fixed.
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
Fixed, updated SRPM: http://www.gnome.org/~hadess/gst-fluendo-mpegdemux/gstreamer-plugins-flumpegdemux-0.10.15-2.fc8.src.rpm updated spec: http://www.gnome.org/~hadess/gst-fluendo-mpegdemux/gstreamer-plugins-flumpegdemux.spec
Looks fine now. Approved
New Package CVS Request ======================= Package Name: gstreamer-plugins-flumpegdemux Short Description: MPEG demuxer for GStreamer Owners: hadess Branches: F-9 devel InitialCC: Cvsextras Commits: yes
cvs done.
Built in rawhide and F-9.
Right I installed this package and still no luck. A little more research showed that at some stage totem-xine had been installed (I didn't do it, or at least not intentionally) and that totem was trying to use the xine backend. I removed this package and all is good now. I still can't view the menu's on DVD. I thought totem shoudl now do this. Should I file a bug report?
(In reply to comment #11) <snip> > I still can't view the menu's on DVD. I thought totem shoudl now do this. > Should I file a bug report? No, it doesn't do menus. It just gives you all the tracks listed in the playlist.