
CI request: build PEP 656 (https://www.python.org/dev/peps/pep-0656/) compliant "musllinux" wheels. This will allow easy installation in Alpine Docker images. GitHub issue: https://github.com/numpy/numpy/issues/20089

On Mon, Oct 11, 2021 at 9:52 AM Laurie O <laurie_opperman@hotmail.com> wrote:
Thanks for asking Laurie. I'd say we may perhaps want to do this, but only if and when it becomes much easier - for example once we have migrated all our wheel build infra to GitHub Actions and cibuildwheel. For now I'm inclined to say that https://github.com/alpine-wheels has NumPy wheels you can use, and that Alpine is a niche platform, so this is quite low-prio for us. In general, providing wheels for every platform under the sun is not sustainable - see https://mail.python.org/archives/list/numpy-discussion@python.org/thread/46H... for more context. Cheers, Ralf

On 11/10/21 2:22 pm, Ralf Gommers wrote:
I am a little worried about releasing wheels untested that will then come back to use as issues with Nans, error states, or inaccurate results. We should have some verification that the platform passes the test suite, preferably a specific run in the main CI. Matti

On Mon, Oct 11, 2021 at 2:10 PM Matti Picus <matti.picus@gmail.com> wrote:
I agree. Platforms where no maintainer is able to build/test locally to resolve issues are already annoying, and not having regular CI is worse. That's why I'm not sure we want to provide Alpine wheels even if our wheel build machinery setup improves. In the end it becomes a maintainer bandwidth issue. Cheers, Ralf

Hi, It will probably soon be trivial to do a manylinux Alpine build / test on multibuild. There's a work on progress PR here, https://github.com/matthew-brett/multibuild/pull/430 that only requires someone to build a trivial test container, in order to get merged. It's also generally very easy to test locally with a minimal Docker setup. Cheers, Matthew On Mon, Oct 11, 2021 at 2:01 PM Ralf Gommers <ralf.gommers@gmail.com> wrote:

On 11/10/21 4:38 pm, Matthew Brett wrote:
A PR to https://github.com/multi-build/docker-images to add Apline images with a complement of python versions would be welcome. The motivation for not using the muslinux containers provided by pypa/manylinux is to check that the wheel built on the pypa/manylinux containers will work in the full distro. Matti

Hi, On Mon, Oct 11, 2021 at 3:05 PM Matti Picus <matti.picus@gmail.com> wrote:
My guess is that that would be less than 90 minutes of work for someone with some Docker experience. It basically involves some edits to the existing build script for Pythons on the current Intel test images. See this directory: https://github.com/multi-build/docker-images/tree/focal/docker and the PR linked above for details. Please do ask for help if stuck. Cheers, Matthew

On Tue, Oct 12, 2021, at 10:13, Matthew Brett wrote:
If you haven't got to it, I can do it.
Filed a PR for this last night: https://github.com/multi-build/docker-images/pull/20 Best regards, Stéfan

On Mon, Oct 11, 2021 at 2:55 PM Ralf Gommers <ralf.gommers@gmail.com> wrote:
I stand corrected, it turns out Thomas Fan had exactly such a plan:) https://github.com/numpy/numpy/pull/20102 Cheers, Ralf

On Mon, Oct 11, 2021 at 9:52 AM Laurie O <laurie_opperman@hotmail.com> wrote:
Thanks for asking Laurie. I'd say we may perhaps want to do this, but only if and when it becomes much easier - for example once we have migrated all our wheel build infra to GitHub Actions and cibuildwheel. For now I'm inclined to say that https://github.com/alpine-wheels has NumPy wheels you can use, and that Alpine is a niche platform, so this is quite low-prio for us. In general, providing wheels for every platform under the sun is not sustainable - see https://mail.python.org/archives/list/numpy-discussion@python.org/thread/46H... for more context. Cheers, Ralf

On 11/10/21 2:22 pm, Ralf Gommers wrote:
I am a little worried about releasing wheels untested that will then come back to use as issues with Nans, error states, or inaccurate results. We should have some verification that the platform passes the test suite, preferably a specific run in the main CI. Matti

On Mon, Oct 11, 2021 at 2:10 PM Matti Picus <matti.picus@gmail.com> wrote:
I agree. Platforms where no maintainer is able to build/test locally to resolve issues are already annoying, and not having regular CI is worse. That's why I'm not sure we want to provide Alpine wheels even if our wheel build machinery setup improves. In the end it becomes a maintainer bandwidth issue. Cheers, Ralf

Hi, It will probably soon be trivial to do a manylinux Alpine build / test on multibuild. There's a work on progress PR here, https://github.com/matthew-brett/multibuild/pull/430 that only requires someone to build a trivial test container, in order to get merged. It's also generally very easy to test locally with a minimal Docker setup. Cheers, Matthew On Mon, Oct 11, 2021 at 2:01 PM Ralf Gommers <ralf.gommers@gmail.com> wrote:

On 11/10/21 4:38 pm, Matthew Brett wrote:
A PR to https://github.com/multi-build/docker-images to add Apline images with a complement of python versions would be welcome. The motivation for not using the muslinux containers provided by pypa/manylinux is to check that the wheel built on the pypa/manylinux containers will work in the full distro. Matti

Hi, On Mon, Oct 11, 2021 at 3:05 PM Matti Picus <matti.picus@gmail.com> wrote:
My guess is that that would be less than 90 minutes of work for someone with some Docker experience. It basically involves some edits to the existing build script for Pythons on the current Intel test images. See this directory: https://github.com/multi-build/docker-images/tree/focal/docker and the PR linked above for details. Please do ask for help if stuck. Cheers, Matthew

On Tue, Oct 12, 2021, at 10:13, Matthew Brett wrote:
If you haven't got to it, I can do it.
Filed a PR for this last night: https://github.com/multi-build/docker-images/pull/20 Best regards, Stéfan

On Mon, Oct 11, 2021 at 2:55 PM Ralf Gommers <ralf.gommers@gmail.com> wrote:
I stand corrected, it turns out Thomas Fan had exactly such a plan:) https://github.com/numpy/numpy/pull/20102 Cheers, Ralf
participants (6)
-
Andrew Nelson
-
Laurie O
-
Matthew Brett
-
Matti Picus
-
Ralf Gommers
-
Stefan van der Walt