Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add CMake option USE_BUNDLED_BLAS_WIN #1940

Merged
merged 4 commits into from
Nov 29, 2023
Merged

Add CMake option USE_BUNDLED_BLAS_WIN #1940

merged 4 commits into from
Nov 29, 2023

Conversation

wshanks
Copy link
Contributor

@wshanks wshanks commented Sep 15, 2023

Summary

Adds a CMake option USE_BUNDLED_BLAS_WIN to control whether the bundled BLAS library is used on Windows

Details and comments

This option controls whether CMake uses the bundled OpenBLAS library when building on Windows (defaults to True). Having this option allows a user to build qiskit-aer against a different system BLAS without needing to specify a full path using the AER_BLAS_LIB_PATH option.

Having this option helps with building qiskit-aer with conda by letting CMake follow its standard library search process.

This option controls whether CMake uses the bundled OpenBLAS library
when building on Windows (defaults to True). Having this option allows a
user to build qiskit-aer against a different system BLAS without needing
to specify a full path using the `AER_BLAS_LIB_PATH` option.
@wshanks
Copy link
Contributor Author

wshanks commented Nov 28, 2023

@doichanj What can I do to move this forward? It was passing before but ran out of heap space after you merged in main. I also wasn't sure about the name I chose for the option.

Also, off topic, but I would be interested in any feedback on #1923. With that issue resolved and the PR here, I could have full platform support for qiskit-aer on conda-forge.

@doichanj doichanj added the automerge This PR will automatically merge once its CI has passed label Nov 29, 2023
@mergify mergify bot merged commit 17edfa3 into Qiskit:main Nov 29, 2023
33 checks passed
@wshanks wshanks deleted the win-blas branch December 18, 2023 16:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge This PR will automatically merge once its CI has passed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants