Bug 2142064 - elements fails to build with pytest 7.2.0 - problem with dependency on `py` package
Summary: elements fails to build with pytest 7.2.0 - problem with dependency on `py` p...
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: elements
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Alejandro Alvarez
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 2137514
TreeView+ depends on / blocked
 
Reported: 2022-11-11 13:34 UTC by Lumír Balhar
Modified: 2022-11-23 19:20 UTC (History)
1 user (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2022-11-23 19:20:15 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Lumír Balhar 2022-11-11 13:34:45 UTC
I'm testing an update of pytest to 7.2.0 and I've discovered that this package fails to build with it. The reason is a change related to `py` package.

`py` is an independent package developed by the same team as pytest and it has been bundled in pytest for quite some time. The `py` package itself is now in maintenance mode and pytest developers decided to bundle only a subset of its functionalities in pytest since version 7.2.0.

It seems that elements needs some parts of `py` that are no longer available in pytest (including deprecated `py.test` itself). There are two possible solutions:

1. Drop dependency on `py`. That requires some coordination with upstream but it's generally a good idea long-term.
2. Add python3-py to build dependencies. `py` package installed from python3-py will have a higher priority than py.py module bundled in pytest. This is a quick workaround but given the status of `py` project, it's only temporary.

If you want to, you can find the failed build in https://copr.fedorainfracloud.org/coprs/lbalhar/pytest/

Comment 1 Lumír Balhar 2022-11-14 11:14:22 UTC
The issue is already fixed upstream: https://github.com/degauden/Elements/commit/f339c678fbcadbcc011e099a620725d47dc105ff

Comment 2 Alejandro Alvarez 2022-11-14 21:27:27 UTC
(In reply to Lumír Balhar from comment #1)
> The issue is already fixed upstream:
> https://github.com/degauden/Elements/commit/
> f339c678fbcadbcc011e099a620725d47dc105ff

Thanks for the tip!

Comment 3 Lumír Balhar 2022-11-23 12:20:40 UTC
Our plan is to ship the updated pytest during the next week. If you need any assistance from us, feel free to say so.

Comment 4 Alejandro Alvarez 2022-11-23 19:20:15 UTC
Sorry, I didn't mark this as closed, but I had pushed the changes to rawhide.


Note You need to log in before you can comment on or make changes to this bug.