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

[Red Hat Bugzilla]

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

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

Summary: Review Request: gstreamer-plugins-flumpegdemux - MPEG
                    demuxer for GStreamer
           Product: Fedora
           Version: rawhide
          Platform: All
        OS/Version: Linux
            Status: NEW
          Severity: medium
          Priority: medium
         Component: Package Review
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]
         QAContact: [e-mail]
                CC: [e-mail],[e-mail]

Spec URL: http://www.gnome.org/~hadess/gst-fluendo-mpegdemux/gstreamer-plugins-mpegdem...
SRPM URL: http://www.gnome.org/~hadess/gst-fluendo-mpegdemux/gstreamer-plugins-flumpeg...
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.

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

[Red Hat Bugzilla]

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 [e-mail]  2008-05-06 05:48 EST -------
FYI, I'll fix the spec file naming before uploading.

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

[Red Hat Bugzilla]

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 [e-mail]  2008-05-06 05:49 EST -------
Original review request:
http://bugzilla.livna.org/show_bug.cgi?id=1523

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

[Red Hat Bugzilla]

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 [e-mail]  2008-05-06 09:38 EST -------
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."

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

[Red Hat Bugzilla]

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 [e-mail]  2008-05-06 11:24 EST -------
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.

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

[Red Hat Bugzilla]

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

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|[e-mail]    |[e-mail]
               Flag|                            |fedora-review?

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

[Red Hat Bugzilla]

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 [e-mail]  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

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

[Red Hat Bugzilla]

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 [e-mail]  2008-05-06 11:49 EST -------
Fixed, updated SRPM:
http://www.gnome.org/~hadess/gst-fluendo-mpegdemux/gstreamer-plugins-flumpeg...
updated spec:
http://www.gnome.org/~hadess/gst-fluendo-mpegdemux/gstreamer-plugins-flumpeg...

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

[Red Hat Bugzilla]

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

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-review?              |fedora-review+

------- Additional Comments From [e-mail]  2008-05-06 12:01 EST -------
Looks fine now. Approved

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

[Red Hat Bugzilla]

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

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|                            |fedora-cvs?

------- Additional Comments From [e-mail]  2008-05-06 14:10 EST -------
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

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

[Red Hat Bugzilla]

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

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-cvs?                 |fedora-cvs+

------- Additional Comments From [e-mail]  2008-05-06 16:59 EST -------
cvs done.

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

[Red Hat Bugzilla]

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

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |CLOSED
         Resolution|                            |RAWHIDE

------- Additional Comments From [e-mail]  2008-05-06 19:30 EST -------
Built in rawhide and F-9.

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

[Red Hat Bugzilla]

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 [e-mail]  2008-05-06 20:17 EST -------
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?

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

[Red Hat Bugzilla]

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 [e-mail]  2008-05-07 08:15 EST -------
(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.