Index | Thread | Search

From:
Theo Buehler <tb@theobuehler.org>
Subject:
Re: [new] spyder 6.1.x new deps
To:
Daniel Dickman <didickman@gmail.com>, foxy@free.fr
Cc:
ports@openbsd.org
Date:
Sun, 19 Oct 2025 00:41:02 +0200

Download raw body.

Thread
On Sat, Oct 18, 2025 at 05:42:28PM -0400, Daniel Dickman wrote:
> Hello, please see attached for new Python ports required in order to 
> update spyder to the 6.1.x series.
> 
> The 2 new ports are:

These look generally ok, although they both have some failing tests.

> 1) textproc/py-ipython_pygments_lexers

test_ipython_pygments_lexers.py:27: AssertionError
=========================== short test summary info ============================
FAILED test_ipython_pygments_lexers.py::test_pygments_entry_points[IPython console session]
FAILED test_ipython_pygments_lexers.py::test_pygments_entry_points[IPython]

> 2) devel/py-python-lsp-ruff

tests/test_ruff_lint.py:185: AssertionError
------------------------------ Captured log call -------------------------------
ERROR    pylsp_ruff.plugin:plugin.py:571 Ruff returned <MagicMock name='Popen().returncode' id='13646936975904'> != 0
=========================== short test summary info ============================
FAILED tests/test_ruff_lint.py::test_ruff_config_param - AssertionError: asse...
FAILED tests/test_ruff_lint.py::test_ruff_settings - AssertionError: assert [...
========================= 2 failed, 11 passed in 2.26s =========================

> The latter port needs these 3 additional ports:
> 1) devel/py-cattrs

Is there a problem with py-immutables? It would be nice to have tests
that work.

ok regardless

> 2) devel/py-lsprotocol

ok

> 3) devel/ruff

I assume that you chose the rather old version 4.4 since it's the last
version that builds without some cargo fiddling?

Laurent Cheylus maintains an up-to-date version here (14.1 now):

https://github.com/jasperla/openbsd-wip/tree/master/devel/ruff

I think we should look into importing that instead. Ideally, Laurent would
maintain that in the ports tree since it's about the same deal as devel/uv.

If you do wish to move forward with ruff 4.4, I'd suggest you set

SEPARATE_BUILD =	Yes
MODCARGO_INSTALL_TARGET_PATHS = crates/ruff

instead of your custem do-install target.