On Thu, May 26, 2022 at 5:24 PM Miro Hrončok <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/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.

Thanks!
Owen


- Owen