On 28. 12. 18 18:58, Avram Lubkin wrote:
On Fri, Dec 28, 2018 at 12:38 PM Igor Gnatenko
<ignatenkobrain@xxxxxxxxxxxxxxxxx
<mailto:ignatenkobrain@xxxxxxxxxxxxxxxxx>> wrote:
You can't make it work in EPEL easily because python modules do not
have pythonX.Ydist() provides.
Didn't realize that and not sure why that wasn't backported.
Because stuff like this is not usually backported to already released RHELs.
Honestly,
if we aren't going to be consistent across EPEL and Fedora when we can,
then why are they even under the same umbrella?
Fedora is newer than EPEL. EPEL is old. There will always be differences
even if the project is under the same umbrella.
Anyway, looks like enabling it for EPEL is a bigger task. I still don't
see the point of enabling it by default in Fedora at this point.
The point is to make Fedora packaging easier. Note the "Fedora" in this
sentence. This will make future EL packaging easier as well, however not
past ELs.
It can
easily be enabled for those who want it and doesn't add work for those
who it won't work for.
It is now enabled. You can use it together with manual deps to maintain
a single spec across EPEL and Fedora. I don't know what exactly is such
a big deal here. It can also be easily disabled if you really like to.
Especially since there is no test code!
And this is the only thing when I agree with you.
Neal, where would be the best place to add some tests for the generator?
I can scratch something like [1] during next week.
[1]
https://github.com/rpm-software-management/rpm-extras/blob/master/brpscripts/Fedora/test_brp_mangle_shebangs.py
--
Miro Hrončok
--
Phone: +420777974800
IRC: mhroncok
_______________________________________________
devel mailing list -- devel@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/devel@xxxxxxxxxxxxxxxxxxxxxxx
