On Thu, 05 Dec 2024 19:55:08 +0100,
Stuart Henderson <s...@spacehopper.org> wrote:
> 
> On 2024/12/05 18:13, Kirill A. Korinsky wrote:
> > On Thu, 05 Dec 2024 17:42:34 +0100,
> > "Jonathan Armani" <jonat...@armani.tech> wrote:
> > > 
> > > 
> > > 
> > > On Thu, Dec 5, 2024, at 16:52, Kirill A. Korinsky wrote:
> > > > On Thu, 05 Dec 2024 11:49:39 +0100,
> > > > Benoit Lecocq <b...@arcane-labs.net> wrote:
> > > >> 
> > > >> 
> > > >> Le 05/12/2024 à 11:39, Kirill A. Korinsky a écrit :
> > > >> > ports@,
> > > >> > 
> > > >> > Here a trivial update of security/wapiti to 3.2.2. Realli trivial:
> > > >> > https://github.com/wapiti-scanner/wapiti/compare/3.2.1...3.2.2
> > > >> > 
> > > >> > I also would like to take formal maintainership on that port.
> > > >> > 
> > > >> > Ok?
> > > >> 
> > > >> ok benoit@
> > > >> 
> > > >>
> > > >
> > > > Seems that updae isn't so trivial. They had switched to use of their own
> > > > arsenic which hasn't got any changes, only renaming:
> > > > https://github.com/fabaff/arsenic/compare/main...wapiti-scanner:arsenic:main
> > > 
> > > You compare between a fork and wapiti.
> > > 
> > > What about these changes between the fork and HENNGE/arsenic ?
> > > https://github.com/HENNGE/arsenic/compare/main...fabaff:arsenic:main
> > >
> > 
> > Thanks, now it makes sense.
> > 
> > Changes are related to compatibility with python-3.12
> > https://github.com/HENNGE/arsenic/pull/166
> > 
> > I think this should be backported to arsenic in our tree like this:
> 
> Read the other part of the PR, this adds another run dependency.
>

Thanks to notice that packaging need to be in RDEP as well.

Index: Makefile
===================================================================
RCS file: /home/cvs/ports/www/py-arsenic/Makefile,v
diff -u -p -r1.1.1.1 Makefile
--- Makefile    30 Oct 2024 15:40:31 -0000      1.1.1.1
+++ Makefile    5 Dec 2024 19:17:32 -0000
@@ -1,6 +1,7 @@
 COMMENT =              asynchronous WebDriver client
 
 MODPY_EGG_VERSION =    21.8
+REVISION =             0
 
 DISTNAME =             arsenic-${MODPY_EGG_VERSION}
 PKGNAME =              py-${DISTNAME}
@@ -22,6 +23,7 @@ MODPY_PYBUILD =               poetry-core
 
 RUN_DEPENDS =          devel/py-attrs${MODPY_FLAVOR} \
                        devel/py-structlog${MODPY_FLAVOR} \
+                       sysutils/py-packaging${MODPY_FLAVOR} \
                        www/py-aiohttp${MODPY_FLAVOR}
 
 .include <bsd.port.mk>
Index: patches/patch-src_arsenic_services_py
===================================================================
RCS file: patches/patch-src_arsenic_services_py
diff -N patches/patch-src_arsenic_services_py
--- /dev/null   1 Jan 1970 00:00:00 -0000
+++ patches/patch-src_arsenic_services_py       5 Dec 2024 17:12:30 -0000
@@ -0,0 +1,32 @@
+Backport compatibility with Python 3.12
+
+https://github.com/HENNGE/arsenic/pull/166
+
+Index: src/arsenic/services.py
+--- src/arsenic/services.py.orig
++++ src/arsenic/services.py
+@@ -2,7 +2,7 @@ import abc
+ import asyncio
+ import re
+ import sys
+-from distutils.version import StrictVersion
++from packaging import version
+ from functools import partial
+ from typing import List, TextIO, Optional
+ 
+@@ -91,11 +91,11 @@ class Geckodriver(Service):
+                     "disable version checking, set `version_check` to "
+                     "`False`."
+                 )
+-            version_str = match.group(1)
+-            version = StrictVersion(version_str)
+-            if version < StrictVersion("0.16.1"):
++            binary_version_str = match.group(1)
++            binary_version = version.parse(binary_version_str)
++            if binary_version < version.parse("0.16.1"):
+                 raise ValueError(
+-                    f"Geckodriver version {version_str} is too old. 0.16.1 or 
"
++                    f"Geckodriver version {binary_version_str} is too old. 
0.16.1 or "
+                     f"higher is required. To disable version checking, set "
+                     f"`version_check` to `False`."
+                 )

-- 
wbr, Kirill

Reply via email to