https://bugzilla.rpmfusion.org/show_bug.cgi?id=1901 --- Comment #23 from Jaroslav Škarvada <jskarvad(a)redhat.com&gt; 2012-03-01 11:21:31 CET --- (In reply to comment #22) Thanks for the review. Updated files: http://jskarvad.fedorapeople.org/bino-1.2.1-2.fc16.src.rpm http://jskarvad.fedorapeople.org/bino.spec Fixed Added comment, this patch is required in order to build due to Fedora DSO linkage change. This patch has been accepted by upstream and is not needed for their devel release. Fixed I am not expert about licensing, but I am not sure if there was any problem (but I implemented all the proposed changes): * The pkg stuff is not used/released in the RPM binary package, thus I think it was OK according to: http://fedoraproject.org/wiki/Licensing/FAQ#Does_the_License:_tag_cover_t... Even after the removal, the source code is still distributed in the srpm package. So I am not sure if the proposed change resolves anything. * For the oxygen icons, IMHO they are licensed under LGPLv3+ thus we probably don't need to explicitly note this because the resulting work is released under GPLv3+. I used the following reference: http://fedoraproject.org/wiki/Licensing/FAQ#What_is_.22effective_license.... http://fedoraproject.org/wiki/Licensing/FAQ#How_should_I_handle_multiple_... -- Configure bugmail: https://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.