Skip to content

CI set rpath before building numpy in BLIS/FlexiBLAS jobs - #260

Merged
lesteve merged 2 commits into
joblib:masterfrom
virchan:issues/259_CI_BLIS/FlexiBLAS_jobs_fail_with_numpy
Sep 30, 2026
Merged

lesteve merged 2 commits into
joblib:masterfrom
virchan:issues/259_CI_BLIS/FlexiBLAS_jobs_fail_with_numpy

Conversation

@virchan

@virchan virchan commented Sep 30, 2026

Copy link
Copy Markdown
Contributor

Fixes #259.

@virchan
virchan marked this pull request as ready for review September 30, 2026 05:19
Comment thread continuous_integration/install_blis.sh Outdated

PKG_CONFIG_PATH=$ABS_PATH/numpy/ pip install . -v --no-build-isolation -Csetup-args=-Dblas=blis

export CFLAGS=-I$ABS_PATH/BLIS_install/include/blis

@DeaMariaLeon DeaMariaLeon Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NIT: maybe move CFLAGS under LDFLAGS ? just to keep things tidy? (it doesn't change behaviour of course).

EDIT: and maybe do the same in continuous_integration/install_flexiblas.sh ?

@DeaMariaLeon

Copy link
Copy Markdown
Contributor

For completeness: here is the pr that seems to have changed the behaviour on meson-python mesonbuild/meson-python#788

@lesteve

lesteve commented Sep 30, 2026

Copy link
Copy Markdown
Member

I haven't read the full discussion but it seems like there is mesonbuild/meson-python#912. I wonder if this is related or a separate thing ...

@lesteve

lesteve commented Sep 30, 2026

Copy link
Copy Markdown
Member

OK CI is green, let's merge this to get the CI greener! Still a non-deterministic issue IIUC.

Thanks for the PR @virchan!

@lesteve
lesteve merged commit 69d69f5 into joblib:master Sep 30, 2026
26 checks passed
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.

CI BLIS/FlexiBLAS jobs fail with numpy

3 participants