Skip to content

[libmysql] Fix issue #5046#5218

Closed
PhoebeHui wants to merge 2 commits intomicrosoft:masterfrom
PhoebeHui:master
Closed

[libmysql] Fix issue #5046#5218
PhoebeHui wants to merge 2 commits intomicrosoft:masterfrom
PhoebeHui:master

Conversation

@PhoebeHui
Copy link
Contributor

libmysql installed failed on Linux, ${CMAKE_SYSTEM_PROCESSOR} didn’t get the value here, apply the patch for fix this issue.

The error message likes:
CMAKE_REQUIRED_LIBRARIES is set to:
m;crypt;dl;-lpthread;rt;atomic
For compatibility with CMake 3.11 and below this check is ignoring it.
Call Stack (most recent call first):
configure.cmake:188 (CHECK_INCLUDE_FILES)
CMakeLists.txt:653 (INCLUDE)
This warning is for project developers. Use -Wno-dev to suppress it.

CMake Error at configure.cmake:459 (STRING):
STRING no output variable specified
Call Stack (most recent call first):
CMakeLists.txt:653 (INCLUDE)

@vicroms vicroms self-assigned this Jan 23, 2019
@vicroms
Copy link
Member

vicroms commented Jan 28, 2019

Hi Phoebe,
The real issue we need to solve is why ${CMAKE_SYSTEM_PROCESSOR} is not set to the right value.
Changes must be made in the Linux toolchain so that the right value is detected and set; l will take a look at this PR during the week.

@PhoebeHui
Copy link
Contributor Author

Thank you Victor for pointing out this!

@vicroms
Copy link
Member

vicroms commented Jan 31, 2019

There was some previous work done to address this issue in #4688.

@JackBoosY
Copy link
Contributor

@vicroms , the related PR is #4970.

@vicroms
Copy link
Member

vicroms commented May 16, 2019

Superseded by #6442

@vicroms vicroms closed this May 16, 2019
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.

4 participants