Re: git: 677f8ffcc79c - main - www/py-beautifulsoup: update dependency on devel/py-lxml to devel/py-lxml5
Date: Mon, 21 Jul 2025 07:39:05 UTC
Hi Daniel, >> 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. Thanks for letting me know the related PRs. I'm going to include the patch for bug 287144 (textproc/yelp-tools) in my batch of next updates. Regarding bug 287143, I will add a comment indicating a block has lifted after the updates. Cheers, Hiroki