Re: git: 677f8ffcc79c - main - www/py-beautifulsoup: update dependency on devel/py-lxml to devel/py-lxml5

From: Daniel Engberg <diizzy_at_FreeBSD.org>
Date: Mon, 21 Jul 2025 05:26:36 UTC
On 2025-07-21 01:09, Hiroki Tagato wrote:
> Hi Vladimir,
>
>>> Hi Max,
>>>
>>>>> The branch main has been updated by tagattie:
>>>>>
>>>>> URL: https://cgit.FreeBSD.org/ports/commit/? 
>>>>> id=677f8ffcc79c820eb19482ccf9a7e2244176aace
>>>>>
>>>>> commit 677f8ffcc79c820eb19482ccf9a7e2244176aace
>>>>> Author:     Hiroki Tagato <tagattie@FreeBSD.org>
>>>>> AuthorDate: 2025-07-19 06:11:52 +0000
>>>>> Commit:     Hiroki Tagato <tagattie@FreeBSD.org>
>>>>> CommitDate: 2025-07-19 18:59:40 +0000
>>>>>
>>>>>      www/py-beautifulsoup: update dependency on devel/py-lxml to 
>>>>> devel/py-lxml5
>>>>>           PR:             288047
>>>>>      Approved by:    wen (maintainer)
>>>>
>>>> The switch seems incomplete. At least finance/skrooge and 
>>>> science/py- pyteomics
>>>> fail to package after this change, because they depend both on py- 
>>>> lxml and
>>>> py-lxml5 now.
>>>>
>>>> Have you checked all ports that depend on py-beautifulsoup, 
>>>> directly and indirectly?
>>>
>>> I thought I did. I checked ports depending on both py-beautifulsoup 
>>> and py-lxml directly and indirectly and switched their dependencies:
>>> https://cgit.freebsd.org/ports/commit/? 
>>> id=230fb2661c78f4501af92ad1ced9793a655f640d
>>>
>>> But as you pointed out, it seems incomplete.
>>>
>>> I will check the dependencies again. Sorry for the breakage.
>>>
>>> Best regards,
>>> Hiroki
>>
>> Hi!
>>
>> The situation is more complicated - py-lxml and py-lxml5 conflict 
>> during installation, but both are used as runtime dependencies for 
>> various ports, which may well be installed on the same host. If we 
>> migrate from 4 to 5, then for all ports at once. And it is even 
>> better to simply replace 4 with 5, rather than leaving 2 separate ports.
>
> Thanks for the suggestion.
>
> I checked all the ports which depend on py-lxml or py-lxml5, and found 
> that only one port (devel/py-pymaven-patch) wanted to stick with 
> py-lxml (version 4). There are no ports depending on py-pymaven-patch 
> so that does not seem a big problem.
>
> I will change dependencies of all the ports (including listed below) 
> except devel/py-pymaven-patch from py-lxml to py-lxml5 at once.
>
>> As I wrote in https://bugs.freebsd.org/288047 py311-lxml is in 
>> RUN_DEPENDS for all 3 ports:
>> $ pkg info -r py311-lxml
>> py311-lxml-4.9.3_2:
>>          py311-streamlink-6.5.1_1
>>          inkscape-1.4.2
>>          lv2-1.18.10
>
> Cheers,
> Hiroki

Hi,

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=287144 and 
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=287143 might also be 
of interest.

Best regards,
Daniel