Skip to content

Try to use pkgconf-pypi for blas dependency#543

Draft
arcondello wants to merge 1 commit into
dwavesystems:mainfrom
arcondello:pkgconf-blas
Draft

Try to use pkgconf-pypi for blas dependency#543
arcondello wants to merge 1 commit into
dwavesystems:mainfrom
arcondello:pkgconf-blas

Conversation

@arcondello
Copy link
Copy Markdown
Member

Try to use https://github.com/pypackaging-native/pkgconf-pypi for scipy-openblas configuration

AI Generation Disclosure

No AI used.

@arcondello arcondello force-pushed the pkgconf-blas branch 2 times, most recently from f40fde5 to 9af2921 Compare May 5, 2026 21:17
@arcondello arcondello added the enhancement New feature or request label May 5, 2026
@arcondello
Copy link
Copy Markdown
Member Author

arcondello commented May 5, 2026

Hmm, unfortunately the pkgconf doesn't play nicely with the lack of universal wheel for scipy-openblas. This means that our x86-64 MacOS build fails.

Some options off the top of my head:

  • Drop the scipy-openblas build for MacOS x86-64. Honestly how many of those users do we still have?
  • Just drop this PR entirely and keep the status quo.
  • I think we could swap the order of the delocate-merge depending on whether we're trying to do an arm or x86-64 build. But yeah, that seems like a mess.
  • Rather than merging the wheels, we might be able to install them into individual environments, dump the pkgconf files somewhere, and then append the pkgconf path.

@arcondello
Copy link
Copy Markdown
Member Author

arcondello commented May 8, 2026

Ok current version just doesn't do a BLAS build for macos x86-64.

I gotta say, with the pkgconf-pypi change pypackaging-native/pkgconf-pypi#105, I am inclined to abandon this. At least until/unless meson or meson-python adds native support pypackaging-native/pkgconf-pypi#98 (comment).

Additional discussion in pypackaging-native/pkgconf-pypi#110

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

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant