On Thu, May 26, 2022 at 5:24 PM Miro Hrončok <mhron...@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
_______________________________________________
python-devel mailing list -- python-devel@lists.fedoraproject.org
To unsubscribe send an email to python-devel-le...@lists.fedoraproject.org
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/python-devel@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure

Reply via email to