[Bug 842509] Review Request: libdbusmenu - A helper library for libindicator

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

 



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

--- Comment #8 from Jef Spaleta <jspaleta@xxxxxxxxx> ---
(In reply to comment #7)
yes all those deps including the pkgconfig are redundant. Removed in latest
version of the spec and srpm which are now uploaded back to the url locations.

Guidelines still have ldconfig as needed in post and postun.
https://fedoraproject.org/wiki/Packaging/Guidelines#Shared_Libraries

Some of the deps you mention were added due to my misreading of the current
directory ownership policy. I've added the directories into the owned files
list instead. You'll see the files section updated for all subpackages as a
result of that correction.

I re-reviewed the licensing in the package. Only the tests are GPL'd. All the
payload files in the binary rpm generated from C code that are explicitly
licensed LGPLv2 OR LGPLv3 in the C file boiler plate headers. I've changed the
License tag accordingly and added a note. 

Michael, does that address everything?

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