[Bug 1030] Review request: xbmc - Media center

[RPM Fusion Bugzilla]

http://bugzilla.rpmfusion.org/show_bug.cgi?id=1030





--- Comment #17 from rc040203 at freenet.de  2010-01-13 08:13:20 ---
More comments:

* Package contains a patch backup:
/usr/lib64/xbmc/system/python/spyce/spyce.py.spyce

Proposal: Don't use patch backups (I personally consider them to be harmful).


* This file is bogus:
/usr/lib64/xbmc/visualisations/xbmc_vis.h

Proposal: Add 
rm -f $RPM_BUILD_ROOT%{_libdir}/xbmc/visualisations/xbmc_vis.h
to %install 
(I thought this already had been in Rolf's original *.spec ?)


* Building goom triggers this warning:
...
/builddir/build/BUILD/xbmc-9.11/xbmc/visualizations/Goom/goom2k4-0/missing:
Unknown `--run' option
Try `/builddir/build/BUILD/xbmc-9.11/xbmc/visualizations/Goom/goom2k4-0/missing
--help' for more information
configure: WARNING: `missing' script is too old or missing
...

The solution to this issue would be to either upgrade/patch
xbmc/visualizations/Goom/goom2k4-0/missing to a newer version of "missing" or
to extend "bootstrap" to be run inside of xbmc/visualizations/Goom/goom2k4-0
(Dangerous!)


* RPM_OPT_FLAGS are still not acknowledged correctly in all places.


* xbmc's webserver frequently dumps core on x86_64.
No actual surprise, because it's based on GoAhead, a piece of SW which is known
not to be 64bit compliant.

I am currently testing a hack to GoAhead with my local rpm and keep you posted.


* xbmc still applies local files from its local copies of zlib.
[I am working on it - Patch to come soon.]


* I am not convinced about Rolf's "win patch".
Instead, I am using this in %prep:
...
# The tarball contains prebuilt binaries
find \( -name '*.so' -o -name '*.DLL' -o -name '*.dll' -o -name '*.lib' -o
-name '*.zlib' -o -name '*.obj' -o -name '*.exe' -o -name '*.vis' \) | xargs rm
-f

# Force using the system-wide versions
rm -rf lib
...


* Has anybody tested "xbmc-standalone" rsp. the xbmc.desktop?
My attempts were without much success. AFAICT, xbmc-standalone doesn't seem to
be applicable on Fedora and needs to be reworked.


* /usr/bin/xbmc contains some magic to produce "verbose core dump logs".
I am not sure, this logic is useful in general and on Fedora in particular
(abrt already takes care about it), but I don't have a strong opinion on 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.