Re: git: 5386980e3a21 - main - devel/py-pymaven-patch: switch from devel/py-lxml to devel/py-lxml5

From: Ruslan Makhmatkhanov <rm_at_FreeBSD.org>
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