Index | Thread | Search

From:
Björn Ketelaars <bjorn.ketelaars@hydroxide.nl>
Subject:
Re: new jupyter/jupyterlab deps
To:
Daniel Dickman <didickman@gmail.com>
Cc:
ports@openbsd.org
Date:
Mon, 15 Jul 2024 11:47:10 +0200

Download raw body.

Thread
On Sun 14/07/2024 14:51, Daniel Dickman wrote:
> Attached are a few new ports needed in order to update jupyter and 
> jupyterlab.
> 
> This port is needed to update jupyter_server to the 2.x series:
> - devel/py-overrides
> 
> These 2 ports are needed by devel/py-jupyter_events (not in the tree yet):
> - textproc/py-python-json-logger
> - devel/py-test-console-scripts
> 
> These ports will be needed by jupyterlab 4.x:
> - devel/py-hatch-jupyter-builder
> - devel/py-async-lru
> - textproc/py-json5
> 
> ok to import these so I can move forward with updating the jupyter stack?

Ports look good and, with exception of devel/py-test-console-scripts,
build. py-test-console-scripts needs the diff below. Without it, it does
not build.

diff --git devel/py-test-console-scripts/Makefile devel/py-test-console-scripts/Makefile
index f7c59eaf6eb..096a6e643c1 100644
--- devel/py-test-console-scripts/Makefile
+++ devel/py-test-console-scripts/Makefile
@@ -15,6 +15,6 @@ FLAVORS =		python3
 FLAVOR =		python3

 MODPY_PI =		Yes
-MODPY_PYBUILD =		setuptools
+MODPY_PYBUILD =		setuptools_scm

 .include <bsd.port.mk>



Would it make sense to rename rename py-python-json-logger to
py-json-logger? Seems a bit nicer, and is done by several other python
modules. Easily addressed by renaming the directory and applying the
diff below. It is up to you...

diff --git textproc/py-python-json-logger/Makefile textproc/py-python-json-logger/Makefile
index fa94b873aa2..a2602f45551 100644
--- textproc/py-python-json-logger/Makefile
+++ textproc/py-python-json-logger/Makefile
@@ -2,7 +2,7 @@ COMMENT =		Python library adding a json log formatter
 
 MODPY_EGG_VERSION =	2.0.7
 DISTNAME =		python-json-logger-${MODPY_EGG_VERSION}
-PKGNAME =		py-${DISTNAME}
+PKGNAME =		${DISTNAME:S/python-/py-/}
 
 CATEGORIES =		textproc
 

With the first diff, OK bket@ for importing all ports.