update ipykernel to 6.6.0, make debugpy optional

[tornaria]

This new standard package contains precompiled binaries, which are dynamically linked to a particular libc.

In particular this causes some problems building sage with the musl libc.

AFAICT, this was added in #31280 just because it was a new dependency for ipykernel.

OTOH, debugpy was made optional in ipython/ipykernel#767 and this is included in ipykernel 6.4.0 and up (current version 6.6.0).

Is it possible to

• either upgrade ipykernel or add the patch to make debugpy optional;
• and then either remove or make optional debugpy in sagemath
  before 9.5, please?

Depends on #32930

CC: @mkoeppe

Component: packages: standard

Author: Gonzalo Tornaría

Branch/Commit: 715d1ea

Reviewer: Matthias Koeppe

Issue created by migration from https://trac.sagemath.org/ticket/33020

[mkoeppe]

comment:1

It is strange that ipykernel still declares the dependency on debugpy as an install_requires (https://github.com/ipython/ipykernel/blob/main/setup.py#L66).

[mkoeppe]

comment:2

I'd say for Sage 9.5, let's add PR 767 as a patch, patch out the install_requires (otherwise there will be a warning when installing the ipykernel package.), and move the dependency on debugpy from ipykernel to jupyterlab

[tornaria]

comment:3

Note arch pkg for debugpy removes all these precompiled libs and compiles the one for linux 64 bit (see https://github.com/archlinux/svntogit-community/blob/packages/python-debugpy/trunk/PKGBUILD).

It doesn't even seem difficult, as it is a single file. However, making sure this works for all supported architectures may be nontrivial, but otoh I expect the current package doesn't really work in all supported architectures anyway.

[tornaria]

Commit: 15cf50e

[tornaria]

New commits:

15cf50e

update ipykernel and make debugpy optional

[tornaria]

Branch: u/tornaria/33020

[tornaria]

comment:5

That commit works for me, it includes updating ipykernel to 6.6.0 and patching out setup.py and pyproject.toml.

If you prefer to stay at 6.3.1 and add PR 767 as a patch, that'd be easy to do (I was already testing this when I saw your comment).

[mkoeppe]

Dependencies: #32930

[mkoeppe]

Changed branch from u/tornaria/33020 to u/mkoeppe/33020

[mkoeppe]

comment:8

I've put this on top of #32930

---

Last 10 new commits:

6347f98	build/pkgs/prompt_toolkit: Update to 3.0.22
28e535d	build/pkgs/widgetsnbextension: Update to 3.5.2
63078ac	build/pkgs/terminado: Update to 0.12.1
22de5dc	build/pkgs/notebook: Update to 6.4.6
a0fe18b	build/pkgs/nbclient: Update to 0.5.4
ef3fc7f	build/pkgs/jupyter_client: Update to 7.1.0
a593bc9	build/pkgs/jupyter_core: Update to 4.9.1
66c16b1	build/pkgs/importlib_metadata: Update to 4.8.2
cdfde89	Merge #32930
af1a6b0	update ipykernel and make debugpy optional

[mkoeppe]

Changed commit from 15cf50e to af1a6b0

[mkoeppe]

Author: Gonzalo Tornaría

[sagetrac-git]

Changed commit from af1a6b0 to 715277c

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

715277c

update ipykernel and make debugpy optional

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

715d1ea

build/pkgs/ipykernel/checksums.ini: Use files.pythonhosted.org URL

[sagetrac-git]

Changed commit from 715277c to 715d1ea

[mkoeppe]

comment:13

The pypi.io URL didn't seem to work, so I changed it to one that works.

[mkoeppe]

Reviewer: Matthias Koeppe

[vbraun]

Changed branch from u/mkoeppe/33020 to 715d1ea