Skip to content

WIP: build scipy with meson#3649

Closed
rth wants to merge 4 commits intopyodide:mainfrom
rth:meson-scipy
Closed

WIP: build scipy with meson#3649
rth wants to merge 4 commits intopyodide:mainfrom
rth:meson-scipy

Conversation

@rth
Copy link
Copy Markdown
Member

@rth rth commented Mar 10, 2023

Tries to continue @hoodmane 's work in #3037

@rth rth changed the title WIP: builds scipy with meson WIP: build scipy with meson Mar 10, 2023
@rth
Copy link
Copy Markdown
Member Author

rth commented Mar 10, 2023

So there are a few issues that happen here,

Dependency lookup for OpenBLAS with method 'pkgconfig' failed: Pkg-config binary for machine 1 not  
found. Giving up.

as discussed in the parent issue. pkg-config is installed and setting export PKG_CONFIG_LIBDIR=$WASM_PKG_CONFIG_PATH should fix the path, so I don't know why it cannot find it.

  1. WARNING: Unknown CPU family wasm, please report this at https://github.com/mesonbuild/meson/issues/new
    Maybe a false positive, the config looks correct.

  2. ERROR: Compiler /usr/bin/gfortran can not compile programs. depending on what one puts in the config of tools/emscripten.meson.cross.

Initially, I felt the quickest might be to monkeypatch the compiler checks in meson but since it's run in a separate process by meson-python we can't easily modify that code at runtime, short of installing a modified version of meson.

So we would indeed likely need to make current checks pass with our compiler wrappers, as Hood was suggesting.

I'll likely not continue this PR in the near future. This was mostly so we have a separate PR on scipy with meson, independent from scipy. Feel free to take it over.

@rth
Copy link
Copy Markdown
Member Author

rth commented Mar 14, 2023

BTW, it looks like the last version of scikit-image also migrated to meson. So maybe that would be an easier package to start, since it only has Cython with a little C.

@ryanking13
Copy link
Copy Markdown
Member

ryanking13 commented Mar 15, 2023

There are too many build systems in the world 😂

@rth
Copy link
Copy Markdown
Member Author

rth commented Mar 15, 2023

Indeed :) But with distutils deprecations, many of Python scientific packages (or packages with C extensions) will move to meson. So that's one we likely can't avoid.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants