Re: Is __python now required to be defined in rawhide?

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



* Florian Weimer:

>> diff --git a/macros.python-srpm b/macros.python-srpm
>> index 514a449..f25189b 100644
>> --- a/macros.python-srpm
>> +++ b/macros.python-srpm
>> @@ -5,6 +5,10 @@
>>  %__python2 /usr/bin/python2
>>  %__python3 /usr/bin/python3
>>  
>> +# This now errors unless redefined to user provided value
>> +%__python %{error:attempt to use unversioned python, define %%__python to /usr/bin/python2 or /usr/bin/python3 explicitly}
>
> /usr/lib/rpm/redhat/macros contains this:
>
> %__brp_python_bytecompile /usr/lib/rpm/brp-python-bytecompile "%{__python}" "%{?_python_bytecompile_errors_terminate_build}" "%{?_python_bytecompile_extra}"
>
> And htis:
>
> %__os_install_post    \
> […]
>     %{?py_auto_byte_compile:%{?__brp_python_bytecompile}} \
>
> I think this means that %{__python} is always expanded during the build.
>
> Does this mean that all spec files now need to define __python?

Probably not.  The build has already been untagged:

Mon Jun 17 15:59:56 2019: python-rpm-macros-3-43.fc31 untagged from f31 by ignatenkobrain

So I assume that was just a python-rpm-macros bug.

Thanks,
Florian
_______________________________________________
devel mailing list -- devel@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/devel@xxxxxxxxxxxxxxxxxxxxxxx




[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Fedora Announce]     [Fedora Users]     [Fedora Kernel]     [Fedora Testing]     [Fedora Formulas]     [Fedora PHP Devel]     [Kernel Development]     [Fedora Legacy]     [Fedora Maintainers]     [Fedora Desktop]     [PAM]     [Red Hat Development]     [Gimp]     [Yosemite News]

  Powered by Linux