On Tue, Nov 11, 2014 at 2:00 PM, Rex Dieter <rdie...@math.unl.edu> wrote:
> Jeff Sheltren wrote:
>
>> On Fri, Oct 31, 2014 at 5:42 AM, Rex Dieter
>> <rdie...@math.unl.edu> wrote:
>>
>>> Rex Dieter wrote:
>>>
>>> > +%epel %{?rhel}%{!?:rhel:7}
>>>
>>> typo alert ^^ (in the second part), but hopefully you get the idea.
>>>
>>> Or, if you'd rather not depend on %rhel macro, and just hard-code to 7,
>>> that
>>> would be fine too.
>>>
>>
>>
>> The approach looks good to me.
>>
>> The patch needs a little work though: besides the typo you mentioned, the
>> install line is copying the wrong SOURCE file.
>>
>> I see %rhel is present on CentOS -- I can't speak to other rebuilds
>> though. It would be nice to verify that.
>>
>> Bonus points for a second patch to remove all the extra trailing
>> whitespace happening in this spec :)
>
> OK (attached).
>+1 (FWIW) -AdamM > -- Rex > _______________________________________________ > epel-devel mailing list > epel-devel@lists.fedoraproject.org > https://admin.fedoraproject.org/mailman/listinfo/epel-devel > _______________________________________________ epel-devel mailing list epel-devel@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/epel-devel
