Re: git: 5386980e3a21 - main - devel/py-pymaven-patch: switch from devel/py-lxml to devel/py-lxml5
Date: Mon, 21 Jul 2025 14:55:02 UTC
Gleb Popov wrote on 7/21/25 5:25 PM: > On Mon, Jul 21, 2025 at 5:12 PM Hiroki Tagato <tagattie@freebsd.org> wrote: >> >> The branch main has been updated by tagattie: >> >> URL: https://cgit.FreeBSD.org/ports/commit/?id=5386980e3a21bbf6ca731490136d4658d70c4c5f >> >> commit 5386980e3a21bbf6ca731490136d4658d70c4c5f >> Author: Hiroki Tagato <tagattie@FreeBSD.org> >> AuthorDate: 2025-07-21 14:05:15 +0000 >> Commit: Hiroki Tagato <tagattie@FreeBSD.org> >> CommitDate: 2025-07-21 14:10:10 +0000 >> >> devel/py-pymaven-patch: switch from devel/py-lxml to devel/py-lxml5 >> >> This is the last one which depends on devel/py-lxml. Upstream's >> requirements.txt says "lxml>=4.0.0" (no upper bound). So switching to >> devel/py-lxml5 should cause no harm. > > It feels like it would be better if py-lxml should've been renamed to > py-lxml4 and then py-lxml updated to version 5. > Not sure if it is worth renaming now, though. All tests are passing with lxml5 for this particular port, so I'd say no need to. And they both install into the same location, so this adds yet another conflict. If no ports require lxml4, I'd update textproc/py-lxml to 5.x directly. ================= platform freebsd15 -- Python 3.11.13, pytest-8.1.1, pluggy-1.6.0 benchmark: 5.1.0 (defaults: timer=time.perf_counter disable_gc=False min_rounds=5 min_time=0.000005 max_time=1.0 calibration_precision=10 warmup=False warmup_iterations=100000) rootdir: /usr/home/rm/ports/devel/py-pymaven-patch/work-py311/pymaven-patch-0.3.2 configfile: setup.cfg plugins: mock-3.10.0, profiling-1.8.1, benchmark-5.1.0, anyio-4.9.0 collected 52 items test_artifact.py .... [ 7%] test_client.py .......... [ 26%] test_pom.py .......... [ 46%] test_versioning.py ............................ [100%] ================= -- Regards, Ruslan T.O.S. Of Reality