https://bugzilla.redhat.com/show_bug.cgi?id=1869267
--- Comment #3 from Mat Booth <[email protected]> --- (In reply to Fabio Valentini from comment #2) > One small issue: > > Do not use %{version} before it's defined. Either move "%global gittag > %{version}" between URL and Source0 (here %version is defined and it's > before the first usage of the macro), or drop the macro entirely and replace > its usages with plain "%{version}" if you're not expecting that the tag > format will change in the future. > Ah, I just switched it to use %{version} directly. > The package is a bit weird since it only uses one module from a bigger > package, but it looks fine. > > There's just the fact that I can't seem to figure out the License of the > code. > The file that's marked with %license is rather unhelpful, and there's no > NOTICE or LICENSE or COPYING files anywhere ... > https://github.com/jbosstools/jbosstools-base/blob/master/foundation/ > features/org.jboss.tools.foundation.license.feature/license.html > Yeah, annoyingly this plugin has made abysmal use of the "license feature" feature. > Can you add a license breakdown please? > And I think for ASL 2.0, you'll need to include the text of the license as > well (as is done in other Apache-2.0 Java packages). Okay I added the full text of the licenses: Spec URL: https://fedorapeople.org/~mbooth/reviews/eclipse-usage.spec SRPM URL: https://fedorapeople.org/~mbooth/reviews/eclipse-usage-4.16.0-2.fc33.src.rpm -- 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 -- [email protected] To unsubscribe send an email to [email protected] Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/[email protected]
