-
Notifications
You must be signed in to change notification settings - Fork 6.2k
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
[openblas] Fix C/CXX flag customization in openblas #18535
[openblas] Fix C/CXX flag customization in openblas #18535
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please bump the port version. See documentation.
Also please run command vcpkg x-add-version --all
then commit the changes.
Co-authored-by: Alexander Neumann <30894796+Neumann-A@users.noreply.github.com>
@JackBoosY Updated the versions. Thanks. |
Can you please resolve the file conflict? Thanks. |
…joschonb/openblas-portfile-fix
@JackBoosY Done. Would appreciate to get this merged to avoid another conflict. Thanks! |
The previous code was broken when VCPKG_C_FLAGS or VCPKG_CXX_FLAGS were already defined in the triplet config file.