Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 204495
Summary: | Review Request: perl-GStreamer - GStreamer Perl module | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Chris Weyl <cweyl> |
Component: | Package Review | Assignee: | Patrice Dumas <pertusus> |
Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Package Reviews List <fedora-package-review> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | squentin |
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
URL: | http://search.cpan.org/dist/GStreamer/ | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2006-09-03 05:14:22 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | |||
Bug Blocks: | 163779, 204497 |
Description
Chris Weyl
2006-08-29 15:27:07 UTC
One remark (not a blocker), test.patch could be named with the package name prepended. * rpmlint is silent * package named according to guidelines * free software, licence included * meets packaging guidelines * spec legible * source match upstream 4f6f27edded250b00055d076eb738ff5 GStreamer-0.09.tar.gz * sane provides (with the known insane GStreamer.so) Provides: GStreamer.so perl(GStreamer) perl(GStreamer::Caps) perl(GStreamer::Install::Files) * %files section right I tried to test using player.pl in examples, but it doesn't seems to work. Or maybe I am missing something: $ perl /usr/share/doc/perl-GStreamer-0.09/examples/player.pl /usr/share/wesnoth/music/elf-land.ogg Playing: /usr/share/wesnoth/music/elf-land.ogg Resource not found. at /usr/share/doc/perl-GStreamer-0.09/examples/player.pl line 21. Ok, that's weird. Works for me: [cweyl@zeus ~]$ perl /usr/share/doc/perl-GStreamer-0.09/examples/player.pl /usr/share/wesnoth/music/elf-land.ogg Playing: /usr/share/wesnoth/music/elf-land.ogg Artist: Aleksi AUBRY-CARLSON Title: Album: Track number: 0 My system is a (mostly <grin>) stock fc5/x86_64 system, with all updates. Is yours a devel/fc6tX box? Yep it is devel, i386. The requires seem right: $ rpm -q --requires perl-GStreamer libc.so.6 libc.so.6(GLIBC_2.0) libc.so.6(GLIBC_2.1.3) libdl.so.2 libglib-2.0.so.0 libgmodule-2.0.so.0 libgobject-2.0.so.0 libgstreamer-0.10.so.0 libgthread-2.0.so.0 libm.so.6 libxml2.so.2 libz.so.1 perl >= 0:5.008 perl(:MODULE_COMPAT_5.8.8) perl(DynaLoader) perl(Exporter) perl(Glib) perl(constant) perl(strict) perl(warnings) rpmlib(CompressedFileNames) <= 3.0.4-1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1 rtld(GNU_HASH) I have gstreamer-plugins-base-0.10.9-4 gstreamer-plugins-good-0.10.4-1.fc6 gstreamer-devel-0.10.9-2 gstreamer-tools-0.10.9-2 gstreamer-0.10.9-2 Ok, a strace shows that it tries to open localhost/usr/share/wesnoth/music/elf-land.ogg Changing the scipt a bit such that it doesn't call Glib::filename_to_uri and constructing the uri myself, it works APPROVED In fact gstreamer doesn't resolve an uri like file://localhost/usr/share/wesnoth/music/elf-land.ogg to /usr/share/wesnoth/music/elf-land.ogg Not sure that it is a bug. (In reply to comment #5) > In fact gstreamer doesn't resolve an uri like > file://localhost/usr/share/wesnoth/music/elf-land.ogg > to /usr/share/wesnoth/music/elf-land.ogg > Not sure that it is a bug. It is, from RFC 3986 section 3.2.2: "For example, the "file" URI scheme is defined so that no authority, an empty host, and "localhost" all mean the end-user's machine [...]" Is it a bug of gstreamer or perl-GStreamer? (In reply to comment #7) > Is it a bug of gstreamer or perl-GStreamer? To me, it sounds like a bug in gstreamer, if it works here but not on devel. It looks as if it centers on not parsing the file:/... uri correctly from what you're saying.... For reference, these are my installed gstreamer packages: gstreamer-0.10.4-1.x86_64.rpm gstreamer-devel-0.10.4-1.x86_64.rpm gstreamer-plugins-base-0.10.5-1.x86_64.rpm gstreamer-plugins-good-0.10.2-1.x86_64.rpm gstreamer-plugins-ugly-0.10.2-1.lvn5.x86_64.rpm gstreamer-tools-0.10.4-1.x86_64.rpm gstreamer08-0.8.12-5.fc5.x86_64.rpm gstreamer08-plugins-0.8.12-5.fc5.x86_64.rpm Ok, I'm going to chalk this one up to devel being in a state of perpetual rebuilding. If errors persist into fc6t3, let's open bugs and I'll drive them to where they need to be. Trying to rebuild on devel the tests failed: t/GstRegistry...........ok 1/21undef is not of type GStreamer::PluginFeature at t/GstRegistry.t line 22. # Looks like you planned 21 tests but only ran 3. # Looks like your test died just after 3. t/GstRegistry...........dubious Test returned status 255 (wstat 65280, 0xff00) DIED. FAILED tests 4-21 Failed 18/21 tests, 14.29% okay gstreamer-0.10.11-1.fc7 gstreamer-devel-0.10.11-1.fc7 (In reply to comment #10) > Trying to rebuild on devel the tests failed: > > t/GstRegistry...........ok 1/21undef is not of type GStreamer::PluginFeature at > t/GstRegistry.t line 22. This test the presence of the alsasink from the alsa plugin > gstreamer-0.10.11-1.fc7 > gstreamer-devel-0.10.11-1.fc7 and the alsa plugin is in gstreamer-plugins-base... Maybe the tests shouldn't test specific plugins, or maybe GStreamer should depend on gstreamer-plugins-base, as I'm not sure the gstreamer package by itself is very useful. So... After poking around with various patches, etc, to remove tests failing under mock, I've just wrapped "make test" with a %{?_with_network_tests:... construct. The rpm builds & tests just fine for me outside of mock, and is usable; it just seems that there's not enough of an environment under a mock build for the module to sucessfully test itself. I've commited changes; this package is (finally!) built in extras for fc5->devel. |