[Bug 1226926] Review Request: eclipse-e4-importer - Alternative importer of Eclipse projects

[Red Hat Bugzilla]

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

--- Comment #2 from Richard Shaw  ---
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.