[Bug 844080] Review Request: libmatecomponent -- Libraries for matecomponent package of MATE-Desktop

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

Mario Blättermann <mario.blaettermann@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |mario.blaettermann@xxxxxxxx
                   |                            |m

--- Comment #1 from Mario Blättermann <mario.blaettermann@xxxxxxxxx> ---
You might drop a few packages from BuildRequires because they are recursive
dependencies:

autoconf (needed by automake)
automake (needed by gtk2-devel and intltool)
glib2-devel (needed by gtk2-devel)
libxml2 (needed by libxml2-devel)
mate-corba (needed by mate-corba-devel)
gettext (needed by intltool)

And you should review the runtime requirements in the same manner. Don't
mention anything there if it is not really needed. Moreover, it would be better
to write one line per dependency, or write the packages in an alphabetical
order at least. Would be better readable.

After doing a scratch build without any entries in "Requires:" you should have
a look at the resulting packages to see which of the requirements has been
picked up automatically by rpm.

BTW, the gnome-doc-utils package is no really requirement for MATE, isn't it?
Shouldn't this be mate-doc-utils instead? But I've seen that mate-doc-utils
needs gnome-doc-utils anyway. Somewhat odd... We speak about an independent
GNOME fork an require its tools...?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review



[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]