devel/py-ordered-set: convert to USE_PYTHON=pep517
ClosedPublic
Actions

Authored by vishwin on Aug 6 2024, 2:01 PM.

Details

Reviewers

Commits

R11:1ce890d549a0: devel/py-ordered-set: convert to USE_PYTHON=pep517

Summary

setup.py only still exists for some kind of legacy compatibility; the package itself uses flit-core to build. The correct pytest configuration is included in the PEP-517 configuration.

While here, update WWW

Test Plan

Verified no functional changes.

Diff Detail

Repository

R11 FreeBSD ports repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

vishwin requested review of this revision.Aug 6 2024, 2:01 PM

vishwin created this revision.

Harbormaster completed remote builds in B58955: Diff 141856.Aug 6 2024, 2:01 PM

rfyu28uyeg_snkmail.com added a subscriber: rfyu28uyeg_snkmail.com.Feb 9 2025, 8:02 PM

This works fine with py38 and py311, including tests.

This seems fine, is there a reason it is not moving forward?

agh_riseup.net added a subscriber: agh_riseup.net.Feb 20 2025, 6:45 AM

In D46235#1118265, @allanjude wrote:

This seems fine, is there a reason it is not moving forward?

I don't think so (except maybe ENOTIME). It's very much low hanging fruit and marked as a blocker or helps for some pending python updates that could really stand to get unstuck (like bug 270358, bug 270510, not to mention the bug for this: 280661).

kiwi added a subscriber: kiwi.Mar 12 2025, 5:18 PM

This revision was not accepted when it landed; it landed in state Needs Review.Mar 13 2025, 4:13 PM

Closed by commit R11:1ce890d549a0: devel/py-ordered-set: convert to USE_PYTHON=pep517 (authored by vishwin). · Explain Why

This revision was automatically updated to reflect the committed changes.

vishwin added a commit: R11:1ce890d549a0: devel/py-ordered-set: convert to USE_PYTHON=pep517.