[Bug 530021] Review Request: moovida - Media Center

[bugzilla at redhat.com]

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


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


Christoph Wickert <cwickert at fedoraproject.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |cwickert at fedoraproject.org




--- Comment #4 from Christoph Wickert <cwickert at fedoraproject.org>  2009-10-25 09:18:39 EDT ---
What's the use of splitting this into two packages? Only for building the
plugins or can the moovida-base package actually be used without movida?
Even without splitting the package I cannot see the dependency loop: The
plugins BuildRequire the movida package, but at runtime movida requires (at
least) some plugins. So where is the loop? Note the difference between
BuildRequires and Requires here.

%description is a little short and not really useful. Users don't care if this
is based on gstreamer or something else, but they want to know what the package
actually does.

I doubt moovida-base needs "Requires: python-setuptools".

The package installs a dbus service so I guess it needs to require dbus-x11 to
provide dbus-launch.

You are not validating the moovida.desktop

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.