Skip to content

[coin-or-*] Disable finding mkl, always use lapack.pc#47823

Merged
BillyONeal merged 1 commit intomicrosoft:masterfrom
BillyONeal:fix-coin-or
Oct 15, 2025
Merged

[coin-or-*] Disable finding mkl, always use lapack.pc#47823
BillyONeal merged 1 commit intomicrosoft:masterfrom
BillyONeal:fix-coin-or

Conversation

@BillyONeal
Copy link
Member

This removes explicit detection in coin-or-buildtools that started picking up intel-mkl.lib after #47633

Related #47764

This removes explicit detection in coin-or-buildtools that started picking up intel-mkl.lib after microsoft#47633

Related microsoft#47764
@BillyONeal
Copy link
Member Author

@Neumann-A Do you think this is reasonable or ... ?

@Neumann-A
Copy link
Contributor

I dont mind if it is generally looking for lapack/blas some other way (e.g. pkgconfig)

@BillyONeal
Copy link
Member Author

I dont mind if it is generally looking for lapack/blas some other way (e.g. pkgconfig)

After the removed chunk it has:

dnl If none of the above worked, check whether lapack.pc blas.pc exists and
dnl links. We check for both to ensure that blas lib also appears on link line
dnl in case someone wants to use Blas functions but tests only for Lapack.
dnl We skip this if int64
  if test "$lapack_keep_looking" = yes ; then
    AC_COIN_CHK_MOD_EXISTS([lapack],[lapack blas],
           [lapack_what="generic module (lapack.pc blas.pc)"
            AC_COIN_TRY_LINK([dsyev],[],[lapack],
              [coin_has_lapack=yes
               lapack_keep_looking=no
               lapack_pcfiles="lapack blas"],
              [AC_MSG_WARN([lapack.pc and blas.pc present, but could not find dsyev when trying to link with LAPACK.])],no)
           ])
  fi

@BillyONeal BillyONeal enabled auto-merge (squash) October 15, 2025 22:25
@BillyONeal BillyONeal merged commit dc025b5 into microsoft:master Oct 15, 2025
18 checks passed
@BillyONeal BillyONeal deleted the fix-coin-or branch October 16, 2025 00:12
@BillyONeal BillyONeal mentioned this pull request Oct 16, 2025
7 tasks
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