On 03/02/15 12:44, Stanislav Ochotnicky wrote:
As with several other patches wrt %license flag the problem is that this
is really new rpm feature and rpm in EPEL 6 doesn't support it (-L
switch)
One option would be to say we drop support for EPEL which I personally
wouldn't mind but there were different opinions on the topic...
Another option would be to use the existing flag mechanisms. Basically:
- Register a new EPEL6 flag by amending
plugins/generic.py:Register;;register()
- Conditionalize the patch using self.flags['EPEL6']; there is plenty
of examples using 'EPEL5' in generic.py.
This way the default usage would work with %license, epel users would
need to use fedora-review --define EPEL6 <...> which should please all (?).
Cheers!
--alec