[Bug 289] Review request: gstreamer-pitfdll - GStreamer plugin for using MS Windows binary codecs
http://bugzilla.rpmfusion.org/show_bug.cgi?id=289
Erik van Pienbroek changed:
What|Removed |Added
CC||erik-
||rpmfus...@vanpienbroek.nl
Status|NEW |ASSIGNED
--- Comment #1 from Erik van Pienbroek
2009-01-01 19:38:44 ---
Some initial comments:
- Does this package still work with GStreamer 0.10 ? The website only mentions
support for GStreamer 0.8 and the CVS doesn't seem to have any activitity in
the last 10 months
- The rpmlint warning about the executable stack indicates this package will
conflict with SELinux. This is really something that should be looked into
- IIRC, this package only works on the i386 architecture. Thus the spec should
require an ExclusiveArch: i386
- This package is only useful in combination with win32 codecs. Are you (or
someone else) willing to package/maintain this for RPMFusion (non-free) ?
- Why do you perform an 'mkdir -p $RPM_BUILD_ROOT%{_libdir}/codecs/' in the
%install phase? It doesn't seem to have any purpose right now
Full review coming soon
--
Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
You are the assignee for the bug.
[Bug 289] Review request: gstreamer-pitfdll - GStreamer plugin for using MS Windows binary codecs
http://bugzilla.rpmfusion.org/show_bug.cgi?id=289 Erik van Pienbroek changed: What|Removed |Added Blocks||3 -- Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. You are the assignee for the bug.
[Bug 289] Review request: gstreamer-pitfdll - GStreamer plugin for using MS Windows binary codecs
http://bugzilla.rpmfusion.org/show_bug.cgi?id=289
--- Comment #2 from Andrea Musuruane 2009-01-01 20:17:25
---
(In reply to comment #1)
> - This package is only useful in combination with win32 codecs. Are you (or
> someone else) willing to package/maintain this for RPMFusion (non-free) ?
Why should the submitter not be willing to package/maintain this if he put this
for review here?
> - Why do you perform an 'mkdir -p $RPM_BUILD_ROOT%{_libdir}/codecs/' in the
> %install phase? It doesn't seem to have any purpose right now
I think that that directory is the one where win32 codecs will be placed by
users. Thus this package will own that directory.
--
Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
You are the assignee for the bug.
[Bug 289] Review request: gstreamer-pitfdll - GStreamer plugin for using MS Windows binary codecs
http://bugzilla.rpmfusion.org/show_bug.cgi?id=289
--- Comment #3 from Erik van Pienbroek
2009-01-01 20:35:54 ---
(In reply to comment #2)
> (In reply to comment #1)
> > - This package is only useful in combination with win32 codecs. Are you (or
> > someone else) willing to package/maintain this for RPMFusion (non-free) ?
>
> Why should the submitter not be willing to package/maintain this if he put
> this
> for review here?
I think this is a small misunderstanding. The point I'm trying to make here is
that this package requires win32 codecs in order to be functional, so these
win32 codecs should also be packaged for RPMFusion (in a seperate package). I'm
not saying that this a task for the submitter. This is just meant as a general
statement for the people reading along on the mailing list so that anybody who
is interested in packaging win32 codecs can do so.
> > - Why do you perform an 'mkdir -p $RPM_BUILD_ROOT%{_libdir}/codecs/' in the
> > %install phase? It doesn't seem to have any purpose right now
>
> I think that that directory is the one where win32 codecs will be placed by
> users. Thus this package will own that directory.
The .spec file lacks a '%dir %{_libdir}/codecs/' in the files list, so this
created directory is ignored by rpm.
--
Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
You are the assignee for the bug.
[Bug 289] Review request: gstreamer-pitfdll - GStreamer plugin for using MS Windows binary codecs
http://bugzilla.rpmfusion.org/show_bug.cgi?id=289 --- Comment #4 from Kevin Kofler 2009-01-01 22:21:41 --- The w32codecs aren't legally distributable. I know some sites do it anyway, but that doesn't mean it's legal. ;-) -- Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. You are the assignee for the bug.
[Bug 289] Review request: gstreamer-pitfdll - GStreamer plugin for using MS Windows binary codecs
http://bugzilla.rpmfusion.org/show_bug.cgi?id=289 --- Comment #5 from Erik van Pienbroek 2009-01-01 23:36:06 --- (In reply to comment #4) > The w32codecs aren't legally distributable. I know some sites do it anyway, > but > that doesn't mean it's legal. ;-) So you're saying the win32 codecs which are published at http://www.mplayerhq.hu/design7/dload.html can't be used in RPMFusion due to legal issues? What is it that makes it illegal for RPMFusion? -- Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. You are the assignee for the bug.
[Bug 289] Review request: gstreamer-pitfdll - GStreamer plugin for using MS Windows binary codecs
http://bugzilla.rpmfusion.org/show_bug.cgi?id=289 --- Comment #6 from Kevin Kofler 2009-01-02 01:41:21 --- There's no license (at least not from the actual copyright holders) which would allow distributing them. The default in copyright law is that distribution is not allowed. A license is needed to allow it. -- Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. You are the assignee for the bug.
[Bug 289] Review request: gstreamer-pitfdll - GStreamer plugin for using MS Windows binary codecs
http://bugzilla.rpmfusion.org/show_bug.cgi?id=289 --- Comment #7 from Erik van Pienbroek 2009-01-04 15:05:31 --- Here's the review: Please add an extra comment to the description mentioning this plugin requires external win32 codecs which can't be redistributed in RPMFusion due to legal reasons and that these codecs need to be saved in the directory /usr/lib/codecs --- While testing, I've found out you need to install the win32 codecs BEFORE installing this plugin. If you don't, the win32 codecs won't be detected by gstreamer: $ gst-inspect pitfdll Plugin Details: Name: pitfdll Description: DLL-loader elements Filename: /usr/lib/gstreamer-0.10/libpitfdll.so Version: 0.9.1.1 License: GPL Source module:pitfdll Binary package: PitfDLL Origin URL: http://ronald.bitfreak.net/pitfdll/ 0 features: If this plugin was installed while the win32 codecs weren't installed yet, you can force a gstreamer recheck by removing the directory ~/.gstreamer-0.10 : $ rm -rf ~/.gstreamer-0.10 $ gst-inspect-0.10 pitfdll Plugin Details: Name: pitfdll Description: DLL-loader elements Filename: /usr/lib/gstreamer-0.10/libpitfdll.so Version: 0.9.1.1 License: GPL Source module:pitfdll Binary package: PitfDLL Origin URL: http://ronald.bitfreak.net/pitfdll/ qtadec_bin: quicktime binary audio decoder dmodec_wmspdmodv1: DMO wmspdmod decoder version 1 dmodec_wmadmodv3: DMO wmadmod decoder version 3 dmodec_wmadmodv2: DMO wmadmod decoder version 2 dmodec_wmadmodv1: DMO wmadmod decoder version 1 dmodec_wmvdmodv3: DMO wmvdmod decoder version 3 dmodec_wmvdmodv2: DMO wmvdmod decoder version 2 dmodec_wmvdmodv1: DMO wmvdmod decoder version 1 dmodec_wmv9dmodv3: DMO wmv9dmod decoder version 3 dshowdec_ir41_32v4: DS ir41_32 decoder version 4 dshowdec_ir50_32v5: DS ir50_32 decoder version 5 11 features: +-- 11 elements Perhaps it is a good idea to add this information to the description to avoid future (false) bug reports. --- During the %install phase, the directory /usr/lib/codecs is generated, but it isn't part of the %files list. I don't know if this directory should really be owned by this package as it is also used by MPlayer and Xine. Can somebody on the mailing list clarify whether two or more packages can own a directory? --- This package only works on the i386 architecture (it doesn't even compile on x86_64). Please add an 'ExclusiveArch: i386' to the .spec file --- rpmlint has indicated that the plugin has an executable stack. This causes issues with SELinux. To fix this, add the line ... export LDFLAGS="-Wl,-znoexecstack" ... to the .spec file, right above the %configure line. I haven't tested if this causes any breakage as I don't have any files at hand which can only be decoded by the win32 codecs. -- Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. You are the assignee for the bug.
[Bug 289] Review request: gstreamer-pitfdll - GStreamer plugin for using MS Windows binary codecs
http://bugzilla.rpmfusion.org/show_bug.cgi?id=289 --- Comment #10 from Allisson Azevedo 2009-01-05 14:28:08 --- spec: http://allisson.fedorapeople.org/packages/gstreamer-pitfdll/gstreamer-pitfdll.spec srpm: http://allisson.fedorapeople.org/packages/gstreamer-pitfdll/gstreamer-pitfdll-0.9.1.1-3.20080215cvs.fc10.src.rpm Changelog: * Mon Jan 5 2009 Allisson Azevedo 0.9.1.1-3.20080215cvs - Move information about w32codecs for README.Fedora. -- Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. You are the assignee for the bug.
[Bug 289] Review request: gstreamer-pitfdll - GStreamer plugin for using MS Windows binary codecs
http://bugzilla.rpmfusion.org/show_bug.cgi?id=289 --- Comment #8 from Allisson Azevedo 2009-01-05 12:51:10 --- Update package: spec: http://allisson.fedorapeople.org/packages/gstreamer-pitfdll/gstreamer-pitfdll.spec srpm: http://allisson.fedorapeople.org/packages/gstreamer-pitfdll/gstreamer-pitfdll-0.9.1.1-2.20080215cvs.fc10.src.rpm Changelog: * Mon Jan 5 2009 Allisson Azevedo 0.9.1.1-2.20080215cvs - Added information about w32codecs in description - Added ExclusiveArch for i386 - Added LDFLAGS rules -- Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. You are the assignee for the bug.
[Bug 289] Review request: gstreamer-pitfdll - GStreamer plugin for using MS Windows binary codecs
http://bugzilla.rpmfusion.org/show_bug.cgi?id=289 --- Comment #9 from Andrea Musuruane 2009-01-05 14:04:02 --- (In reply to comment #8) > - Added information about w32codecs in description Please delete end user information from description and put them in a file called README.Fedora or similar that must be included among the doc files. http://fedoraproject.org/wiki/Packaging/Guidelines#Summary_and_description -- Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. You are the assignee for the bug.
[Bug 289] Review request: gstreamer-pitfdll - GStreamer plugin for using MS Windows binary codecs
http://bugzilla.rpmfusion.org/show_bug.cgi?id=289 --- Comment #11 from Erik van Pienbroek 2009-01-09 17:29:32 --- (In reply to comment #9) > (In reply to comment #8) > > - Added information about w32codecs in description > > Please delete end user information from description and put them in a file > called README.Fedora or similar that must be included among the doc files. > > http://fedoraproject.org/wiki/Packaging/Guidelines#Summary_and_description This is kind of a 'chicken and egg' situation. The w32codecs need to be installed before this plugin is installed. If this isn't done, the user is forced to perform some manual action (removing the ~/.gstreamer-0.10 directory) before this plugin becomes functional. The README.Fedora file is only readable after this plugin is installed. I realize this is a diverge from the Fedora package guidelines, but I think we don't really have another option... The only other solution I can think about is patching this plugin so that it somehow triggers a gstreamer plugin cache regeneration when new/changed files are detected in /usr/lib/codecs, but I don't know if that is actually possible (I don't have any experience with the GStreamer API yet..) -- Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. You are the assignee for the bug.
[Bug 289] Review request: gstreamer-pitfdll - GStreamer plugin for using MS Windows binary codecs
http://bugzilla.rpmfusion.org/show_bug.cgi?id=289 Conrad Meyer changed: What|Removed |Added CC||kon...@tylerc.org --- Comment #12 from Conrad Meyer 2009-01-09 17:33:03 --- Why not add a Requires on w32codec? -- Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. You are the assignee for the bug.
[Bug 289] Review request: gstreamer-pitfdll - GStreamer plugin for using MS Windows binary codecs
http://bugzilla.rpmfusion.org/show_bug.cgi?id=289 --- Comment #14 from Conrad Meyer 2009-01-09 18:00:05 --- So? It still needs w32codecs -- just because we cannot provide it does not mean it is not a dependency for this package (to be at all useful). -- Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. You are the assignee for the bug.
[Bug 289] Review request: gstreamer-pitfdll - GStreamer plugin for using MS Windows binary codecs
http://bugzilla.rpmfusion.org/show_bug.cgi?id=289 Andreas Thienemann changed: What|Removed |Added CC||andr...@bawue.net --- Comment #15 from Andreas Thienemann 2009-01-09 18:19:06 --- (In reply to comment #14) > So? It still needs w32codecs -- just because we cannot provide it does not > mean > it is not a dependency for this package (to be at all useful). Having a Requires: dependency on the w32 codecs makes this package not installable on most systems. This is surely not what is intended. I'm perfectly happy with having this gstreamer plugin installable from rpmfusion with the user manually needing to download the w32codecs. That way we're shipping the free bits and the user has to see for himself where to get the w32codecs. We could probably tie this in with the game-downloader (forgot the correct name) to get the needed files in the %post script but this is not needed IMHO. -- Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. You are the assignee for the bug.
[Bug 289] Review request: gstreamer-pitfdll - GStreamer plugin for using MS Windows binary codecs
http://bugzilla.rpmfusion.org/show_bug.cgi?id=289 --- Comment #13 from Erik van Pienbroek 2009-01-09 17:53:55 --- That package isn't allowed in RPMFusion (according to https://bugzilla.rpmfusion.org/show_bug.cgi?id=289#c4) -- Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. You are the assignee for the bug.
[Bug 289] Review request: gstreamer-pitfdll - GStreamer plugin for using MS Windows binary codecs
http://bugzilla.rpmfusion.org/show_bug.cgi?id=289
--- Comment #16 from Erik van Pienbroek
2009-01-09 20:01:17 ---
Another issue we have with this package is PackageKit integration ( which is a
F10 feature:
https://fedoraproject.org/wiki/Features/GStreamer_dependencies_in_RPM ). Right
now, this package lacks the RPM provides for this feature:
$ rpmquery --provides gstreamer-pitfdll
libpitfdll.so
gstreamer-pitfdll = 0.9.1.1-1.20080215cvs.fc10
gstreamer-pitfdll(x86-32) = 0.9.1.1-1.20080215cvs.fc10
After some investigation I've found out this is caused by a 'bug' in the
gstreamer package. The file /usr/lib/rpm/gstreamer.prov (provided by
gstreamer-devel) assumes GStreamer plugins always contain the text 'libgst' in
the filename. If I add the command ...
mv $RPM_BUILD_ROOT%{_libdir}/gstreamer-0.10/libpitfdll.so
$RPM_BUILD_ROOT%{_libdir}/gstreamer-0.10/libgstpitfdll.so
.. to the %install phase of the .spec file of gstreamer-pitfdll (and the
w32codecs are installed during the rpm build!), the auto generation works:
$ rpmquery --provides gstreamer-pitfdll
gstreamer0.10(decoder-audio/x-qdm2)
gstreamer0.10(decoder-audio/x-wma)(wmaversion=1)
gstreamer0.10(decoder-audio/x-wma)(wmaversion=2)
gstreamer0.10(decoder-audio/x-wma)(wmaversion=3)
gstreamer0.10(decoder-audio/x-wms)
gstreamer0.10(decoder-video/x-indeo)(indeoversion=4)
gstreamer0.10(decoder-video/x-indeo)(indeoversion=5)
gstreamer0.10(decoder-video/x-wmv)(wmvversion=1)
gstreamer0.10(decoder-video/x-wmv)(wmvversion=2)
gstreamer0.10(decoder-video/x-wmv)(wmvversion=3)
libpitfdll.so
gstreamer-pitfdll = 0.9.1.1-3.20080215cvs.fc10
gstreamer-pitfdll(x86-32) = 0.9.1.1-3.20080215cvs.fc10
Do the other people here think this is worth a bugreport against the gstreamer
package or should we permanently add the hack I did above to this package?
--
Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
You are the assignee for the bug.
[Bug 289] Review request: gstreamer-pitfdll - GStreamer plugin for using MS Windows binary codecs
http://bugzilla.rpmfusion.org/show_bug.cgi?id=289 --- Comment #17 from Erik van Pienbroek 2009-01-09 20:49:19 --- I just looked into autodownloader. Unfortunately this program is a non-root program, so it can't write files to /usr/lib/codecs. If we patch gstreamer-pitfdll so it also searches in the user's homedir for the w32codecs and integrate autodownloader in it, users will get to see the autodownloader dialog at the first time any GStreamer application (Totem e.g.) is started. This should be do-able, however it introduces some caveats: - If the user presses the 'cancel' button when autodownloader is run, the user needs to remove the ~/.gstreamer-0.10 directory before the gstreamer plugin cache can be updated - Autodownloader is a GUI program. If some non-graphical gstreamer application is started (when no graphical environment is available) things will mess up - The autodownloader dialog will even get shown when the user tries to play an 'open format' media file (like .ogg files) using a gstreamer-based application - It is possible that the autodownloader dialog will get shown directly after the user login (when a gstreamer based application gets started during the login) On second guess, I don't think autodownloader is a (user-friendly) solution to the problem were having here. Perhaps we can ask the upstream gstreamer devs whether it is possible to add some trick that will allow plugins to request a (forced) plugin cache regeneration? That way, gstreamer-pitfdll can automatically request a plugin cache regeneration whenever new/changed files are found in /usr/lib/codecs -- Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. You are the assignee for the bug.
[Bug 289] Review request: gstreamer-pitfdll - GStreamer plugin for using MS Windows binary codecs
http://bugzilla.rpmfusion.org/show_bug.cgi?id=289 --- Comment #18 from Conrad Meyer 2009-01-09 20:57:09 --- (In reply to comment #15) > Having a Requires: dependency on the w32 codecs makes this package not > installable on most systems. > This is surely not what is intended. Isn't it? This package *needs* w32codec to do *anything*. If they don't have w32codec not being able to install it isn't a problem, as it wouldn't do anything anyway when installed. -- Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. You are the assignee for the bug.
[Bug 289] Review request: gstreamer-pitfdll - GStreamer plugin for using MS Windows binary codecs
http://bugzilla.rpmfusion.org/show_bug.cgi?id=289 --- Comment #19 from Andreas Thienemann 2009-01-09 21:07:22 --- (In reply to comment #18) > > Having a Requires: dependency on the w32 codecs makes this package not > > installable on most systems. > > This is surely not what is intended. > > Isn't it? This package *needs* w32codec to do *anything*. If they don't have > w32codec not being able to install it isn't a problem, as it wouldn't do > anything anyway when installed. No, it isn't. The w32codecs are delivered as a tar.gz. Without building your own rpm, this dependency cannot be resolved. We're here to make software installation easier to our users not to ask them to jump through hoops. After all, httpd does need some html files to really be useful. Nevertheless, we allow people to install it even without a dependency on the meta-package "website". -- Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. You are the assignee for the bug.
[Bug 289] Review request: gstreamer-pitfdll - GStreamer plugin for using MS Windows binary codecs
http://bugzilla.rpmfusion.org/show_bug.cgi?id=289 --- Comment #20 from Conrad Meyer 2009-01-09 21:17:46 --- No, there is actually a w32codec package floating around various places. (Search for it on rpm.pbone.net.) -- Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. You are the assignee for the bug.
[Bug 289] Review request: gstreamer-pitfdll - GStreamer plugin for using MS Windows binary codecs
http://bugzilla.rpmfusion.org/show_bug.cgi?id=289 --- Comment #21 from Conrad Meyer 2009-01-09 21:18:35 --- (Mine came from ATrpms, but it appears they no longer ship it.) -- Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. You are the assignee for the bug.
[Bug 289] Review request: gstreamer-pitfdll - GStreamer plugin for using MS Windows binary codecs
http://bugzilla.rpmfusion.org/show_bug.cgi?id=289 --- Comment #22 from Andreas Thienemann 2009-01-09 21:29:37 --- (In reply to comment #20) > No, there is actually a w32codec package floating around various places. > (Search for it on rpm.pbone.net.) Right, but you're not suggesting that we put rpms into our repo which are not installable by our repo + fedora? I don't think so... -- Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. You are the assignee for the bug.
[Bug 289] Review request: gstreamer-pitfdll - GStreamer plugin for using MS Windows binary codecs
http://bugzilla.rpmfusion.org/show_bug.cgi?id=289 --- Comment #23 from Conrad Meyer 2009-01-09 21:42:32 --- As opposed to not explicitly listing the dependency at all and expecting users to somehow provide the pieces? How is that better? -- Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. You are the assignee for the bug.
[Bug 289] Review request: gstreamer-pitfdll - GStreamer plugin for using MS Windows binary codecs
http://bugzilla.rpmfusion.org/show_bug.cgi?id=289 --- Comment #24 from Manuel Wolfshant 2009-01-10 03:03:06 --- (In reply to comment #23) > As opposed to not explicitly listing the dependency at all and expecting users > to somehow provide the pieces? How is that better? > Adding a Requires for a package which neither fedora nor rpmfusion provides is a no-no. IIRC it's not even permitted. You cannot ever be sure that a 3rd party repo will fill the gap. What one can do is to apply Andreas's suggestion (i.e. ship the free bits), own the directory were the codecs should land and - either download those codecs automatically (in %post) if possible or - include a fedora.readme containing the instructions on how to manually download the missing bits -- Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. You are the assignee for the bug.
[Bug 289] Review request: gstreamer-pitfdll - GStreamer plugin for using MS Windows binary codecs
http://bugzilla.rpmfusion.org/show_bug.cgi?id=289 --- Comment #25 from Gianluca Sforna 2009-01-10 13:57:29 --- (In reply to comment #24) > What one can do is to apply Andreas's suggestion (i.e. ship the free bits), > own the directory were the codecs should land and > - either download those codecs automatically (in %post) if possible or > - include a fedora.readme containing the instructions on how to manually > download the missing bits > In the latter case (i.e. if user intervention is needed after installation) it would be nice to mention it in the package description, see for instance mailman spec file. -- Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. You are the assignee for the bug.
[Bug 289] Review request: gstreamer-pitfdll - GStreamer plugin for using MS Windows binary codecs
http://bugzilla.rpmfusion.org/show_bug.cgi?id=289 alsadi changed: What|Removed |Added CC||als...@ojuba.org --- Comment #26 from alsadi 2009-01-10 17:37:21 --- > If this plugin was installed while the win32 codecs weren't installed yet, you > can force a gstreamer recheck by removing the directory ~/.gstreamer-0.10 : > rm -rf ~/.gstreamer-0.10 should we put something like find /home -type d -name '.gstreamer-0.10' | xargs -r /bin/rm -rf in %post or in %triggerin BTW: we offer emulator with no ROMs and we don't tell the users where to get the ROMs, so I guess there is no problem in shipping this even if there is rpms for dlls -- Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. You are the assignee for the bug.
[Bug 289] Review request: gstreamer-pitfdll - GStreamer plugin for using MS Windows binary codecs
http://bugzilla.rpmfusion.org/show_bug.cgi?id=289 --- Comment #27 from Erik van Pienbroek 2009-01-10 17:53:55 --- (In reply to comment #26) > should we put something like > > find /home -type d -name '.gstreamer-0.10' | xargs -r /bin/rm -rf > > in %post > or in %triggerin No, that doesn't solve the problem. The problem is that the w32codecs need to be installed before any gstreamer application is launched after the installation of gstreamer-pitfdll. If this isn't done, the w32codecs won't be detected and the user is forced to remove the ~/.gstreamer-0.10 directory before the w32codecs will be detected by gstreamer. The only situation where this command would give the right effect is in a w32codecs RPM package...but as we can't put it in RPMFusion it's a no-go.. -- Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. You are the assignee for the bug.
[Bug 289] Review request: gstreamer-pitfdll - GStreamer plugin for using MS Windows binary codecs
http://bugzilla.rpmfusion.org/show_bug.cgi?id=289 --- Comment #28 from Gianluca Sforna 2009-01-10 19:56:55 --- (In reply to comment #27) > No, that doesn't solve the problem. The problem is that the w32codecs need to > be installed before any gstreamer application is launched after the > installation of gstreamer-pitfdll. If this isn't done, the w32codecs won't be > detected and the user is forced to remove the ~/.gstreamer-0.10 directory > before the w32codecs will be detected by gstreamer. Sounds like a bug. Or is that just me? -- Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. You are the assignee for the bug.
[Bug 289] Review request: gstreamer-pitfdll - GStreamer plugin for using MS Windows binary codecs
http://bugzilla.rpmfusion.org/show_bug.cgi?id=289
--- Comment #29 from Erik van Pienbroek
2009-01-11 13:53:14 ---
(In reply to comment #28)
> (In reply to comment #27)
> > No, that doesn't solve the problem. The problem is that the w32codecs need
> > to
> > be installed before any gstreamer application is launched after the
> > installation of gstreamer-pitfdll. If this isn't done, the w32codecs won't
> > be
> > detected and the user is forced to remove the ~/.gstreamer-0.10 directory
> > before the w32codecs will be detected by gstreamer.
>
> Sounds like a bug. Or is that just me?
Not really. GStreamer only performs a plugin recheck when a new or changed file
in %{_libdir}/gstreamer-0.10 is detected. So, after the installation of
gstreamer-pitfdll, gstreamer notices there's a new plugin and thus performs a
plugin recheck. However, if at this time, the w32codecs aren't installed in
/usr/lib/codecs, the pitfdll plugin will be detected, but no 'features'
(supported audio/video formats) will be provided by the plugin. Without any
'features', this plugin is useless.
--
Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
You are the assignee for the bug.
[Bug 289] Review request: gstreamer-pitfdll - GStreamer plugin for using MS Windows binary codecs
http://bugzilla.rpmfusion.org/show_bug.cgi?id=289 --- Comment #30 from Erik van Pienbroek 2009-01-11 14:45:52 --- I just found another bug in this package. When trying to open a wma9 file, I get this error: $ totem attachment.cgi\?id\=122237\&action\=view wine/module: Win32 LoadLibrary failed to load: wmadmod.dll, /usr/lib/win32/wmadmod.dll, /usr/local/lib/win32/wmadmod.dll Failed creating an audio decoder: could not open DMO filter from DLL wmadmod.dll The wma9 file was found at http://bugzilla.gnome.org/show_bug.cgi?id=559924 It looks like there's still some place in pitfdll where the folders /usr/lib/win32 and /usr/local/lib/win32 are mentioned and the './configure --with-dlldir=/usr/lib/codecs' option isn't respected. -- Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. You are the assignee for the bug.
[Bug 289] Review request: gstreamer-pitfdll - GStreamer plugin for using MS Windows binary codecs
http://bugzilla.rpmfusion.org/show_bug.cgi?id=289 Allisson Azevedo changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution||WONTFIX --- Comment #31 from Allisson Azevedo 2009-01-13 23:20:33 --- Hi all, There is some problems with this package for inclusion on rpmfusion, and the upstream is probably dead. Thanks for all comments, i close this bug as wontfix. -- Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. You are the assignee for the bug.
[Bug 289] Review request: gstreamer-pitfdll - GStreamer plugin for using MS Windows binary codecs
http://bugzilla.rpmfusion.org/show_bug.cgi?id=289 --- Comment #32 from Erik van Pienbroek 2009-01-13 23:41:24 --- I'm sorry to hear you've retracted this package for review. I personally think this package is welcome for RPMFusion. We've uncovered some issues with the package and that's a good thing. That's what reviews are for. In this case, there just needs to be some discussion with GStreamer upstream to find out the best way to solve these issues (like influencing the plugin cache regeneration). -- Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. You are the assignee for the bug.
[Bug 289] Review request: gstreamer-pitfdll - GStreamer plugin for using MS Windows binary codecs
http://bugzilla.rpmfusion.org/show_bug.cgi?id=289 --- Comment #33 from Allisson Azevedo 2009-01-14 00:23:37 --- Hi Erik, Thanks for your review, if anyone wish work on this package feel free to use my spec file. I'll submit other packages for rpmfusion soon. -- Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. You are the assignee for the bug.
[Bug 289] Review request: gstreamer-pitfdll - GStreamer plugin for using MS Windows binary codecs
http://bugzilla.rpmfusion.org/show_bug.cgi?id=289 Richard changed: What|Removed |Added CC||hobbes1...@gmail.com Blocks|3 |2 --- Comment #34 from Richard 2011-10-06 20:53:43 --- Moving this back to the review tracker. Maybe someone will pick it up. -- Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. You are the assignee for the bug.
[Bug 289] Review request: gstreamer-pitfdll - GStreamer plugin for using MS Windows binary codecs
https://bugzilla.rpmfusion.org/show_bug.cgi?id=289 Jeremy Newton changed: What|Removed |Added CC||alexjn...@hotmail.com Blocks|2 | -- Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. You are the assignee for the bug.
