[Bug 841003] Review Request: mate-corba - CORBA Object Request Broker for MATE Desktop

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

 



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

--- Comment #24 from Rex Dieter <rdieter@xxxxxxxxxxxx> ---
OK, now that the %files list is clearer,

4. MUST omit static libraries and libtool archives, ie, items that match
%{_libdir}/lib*.a
%{_libdir}/lib*.la
(or provide justification why they warrant an exception)

5. SHOULD own includedir subdirectories.  One easy way to do that, is to just
include:
%{_includedir}/matecorba-2.0/
which will include it, and recursively everything under it

6. SHOULD.  this
%{_bindir}/matecorba-linc-cleanup-sockets
looks like something that ought to be in the main runtime pkg instead of -devel

7. -devel MUST have an arch'd dependency on the main pkg, use this instead:
Requires: %{name}%{?_isa} = %{version}-%{release}

8.  MUST drop
Requires: mate-common
this is a BuildRequires only, not required for normal runtime use.

-- 
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]