https://bugzilla.redhat.com/show_bug.cgi?id=1208738 --- Comment #17 from Taylor Braun-Jones <taylor@xxxxxxxxxxxxxxx> --- (In reply to Raphael Groner from comment #16) > Still some open issues I would like to see fixed in the final package: > > - ASL 2.0 is meant for vera.ctest but not for vera.cmake (there's no such > named file!) as you mention it in the comment, though I am not sure if this > license includes then also the full source for the tests. Please be careful > here and better ask upstream if in doubt. That can be pointless since it is > valid only for the tests and they do not go into the compiled RPM. > https://fedoraproject.org/wiki/Licensing:FAQ?rd=Licensing/ > FAQ#Multiple_licensing_situations Sounds good. I've emailed legal@xxxxxxxxxxxxxxxxxxxxxxx to get it figured out. > - You can add "BuildArch: noarch" and "Requires: cmake" (or cmake28 > respectively) to the devel subpackage because it does not include by itself > any arch dependent binaries and is useful only with cmake at hand. Makes sense. Fixed. > APPROVED Thanks for your help, Raphael. Do you mind if I add you as a co-maintainer? If so, what is your Fedora Account username? -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review