On 07. 06. 22 20:20, Owen Taylor wrote:
On Thu, May 26, 2022 at 5:24 PM Miro Hrončok <mhroncok(a)redhat.com
<mailto:mhroncok@redhat.com>> wrote:
On 26. 05. 22 22:08, Owen Taylor wrote:
> OK, I'll work on coming up with some patches. Do you have an opinion about
> whether it makes sense to make the %_prefix support complete with
something like:
>
> - %python3_sitelib %(RPM_BUILD_ROOT= %{__python3} -Ic "import sysconfig;
> print(sysconfig.get_path('purelib'))")
> +%python3_sitelib %(RPM_BUILD_ROOT= %{__python3} -Ic "import sysconfig;
> print(sysconfig.get_path('purelib', sysconfig.get_default_scheme(),
> {'platbase': '%{_prefix}', 'base':
'%{_prefix}')})
>
> Without that, the --prefix %{_prefix} additions won't work in isolation,
and
> will require something else to redefine python3_sitelib/python3_sitearch.
Using vars={'platbase': '%{_prefix}', 'base':
'%{_prefix}')} makes sense to me
(you can omit the second parameter (scheme) if you pass vars by name).
Ah, yes, thanks.
I made a pair of pull requests:
https://src.fedoraproject.org/rpms/python-rpm-macros/pull-request/135
<
https://src.fedoraproject.org/rpms/python-rpm-macros/pull-request/135>
https://src.fedoraproject.org/rpms/pyproject-rpm-macros/pull-request/288
<
https://src.fedoraproject.org/rpms/pyproject-rpm-macros/pull-request/288>
that I believe have all the necessary changes, and did some testing (there's
also a test added in pyproject/rpm-macros/tests, which seems to work following
the instructions for local testing in the README.md)
Unfortunately, I need to take time off from work for personal reasons for a
while, so if you could go ahead and make whatever changes are needed and merge
them, that would be greatly appreciated. Stephan Bergmann can help you if you
want confirmation that these are actually fixing the Flatpak build issues that
we were running into.
Stephan, could you please verify?
--
Miro Hrončok
--
Phone: +420777974800
IRC: mhroncok