https://bugzilla.redhat.com/show_bug.cgi?id=1226926
--- Comment #2 from Richard Shaw <hobbes1069(a)gmail.com> ---
I'm not familiar enough with eclipse to take this review but with a drive by
spec review it looks pretty good. The only thing I see missing in the files
section is a license file. Is one included in the source?
If so the license macro should be used for f21+ (and I think epel7). If you
plan to support older releases then you'll have to use a conditional.
Something like:
%if 0%{?rhel} || 0%{?fedora} < 21
%doc COPYING
%else
%license COPYING
%endif
And if epel7 does support the license macro then a "< 7" can be added to the
rhel portion.
--
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