Skip to content

fix bug: cannot found gfortran lib in openblas #1980

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

Closed
wants to merge 1 commit into from

Conversation

dzhwinter
Copy link
Contributor

No description provided.

@dzhwinter dzhwinter requested review from a user, helinwang, gangliao and typhoonzero May 3, 2017 07:11
@dzhwinter
Copy link
Contributor Author

百度内部开发机,编译openblas找不到gfortran的bug,fix

@gangliao
Copy link
Contributor

gangliao commented May 3, 2017

related to #1958 这里其实已经去掉gfortran了

@dzhwinter
Copy link
Contributor Author

加入向后兼容一下?同学都反映过gfortran这个问题。后续去掉gfortran就确实不需要了,类似的还有swig。依赖太多了

@gangliao
Copy link
Contributor

gangliao commented May 3, 2017

@dzhwinter 好呢

@dzhwinter dzhwinter removed the request for review from a user May 3, 2017 07:46
@@ -37,11 +37,18 @@ IF(NOT ${CBLAS_FOUND})
string(REGEX MATCHALL "[0-9]+" Fortran_VERSION ${CMAKE_Fortran_COMPILER_VERSION})
list(GET Fortran_VERSION 0 Fortran_MAJOR)
list(GET Fortran_VERSION 1 Fortran_MINOR)
set(GCC_INCLUDE_PATH ${CMAKE_C_COMPILER})
Copy link
Contributor

Choose a reason for hiding this comment

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

这里不要叫GCC_INCLUDE_PATH是否好点?

@dzhwinter dzhwinter closed this May 3, 2017
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