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

CmakeLists.txt: use LIB_RESOLV instead of resolv. #10972

Merged
merged 1 commit into from Sep 7, 2016

Conversation

wjwithagen
Copy link
Contributor

  • FreeBSD does not have a separate libresolv, but keeps it in
    libc

Signed-off-by: Willem Jan Withagen wjw@digiware.nl

 - FreeBSD does not have a separate libresolv, but keeps it in
   libc

Signed-off-by: Willem Jan Withagen <wjw@digiware.nl>
@tchaikov tchaikov self-assigned this Sep 3, 2016
@wjwithagen
Copy link
Contributor Author

@tchaikov
Pushed an extra commit to fix the Python problem.
I can drop that once a fix in master is available

@tchaikov
Copy link
Contributor

tchaikov commented Sep 5, 2016

jenkins, retest this please

@wjwithagen
Copy link
Contributor Author

@tchaikov

The following tests FAILED:
    147 - unittest_http_manager (Timeout)

And I'm having this also in other jenkins runs that are totally not related.

find_package(Python3Interp 3 REQUIRED)
find_package(Python3Libs 3 REQUIRED)
find_package(Python3Interp 3 QUIET)
find_package(Python3Libs 3 QUIET)
Copy link
Contributor

Choose a reason for hiding this comment

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

could you drop this commit? it breaks the build if python3 is missing, see src/pybind/CMakeLists.txt.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@tchaikov
I think it is the other way around.
With QUIET the absence of python3 stuff is sillently ignored.

If I test with REQUIRED, things seriously blow up in my face on some Jenkins runs where there is python3 to be installed. And things really stop during CMake processing.
So I added this to actually get some testing. (Now it stops on the last test due to timeout, and unrelated to this commit.)
But perhaps somebody fixed the python stuff in the mean time.
I'll drop this extra commit.

@tchaikov tchaikov merged commit 3942ac4 into ceph:master Sep 7, 2016
@wjwithagen wjwithagen deleted the wip-wjw-cmake-resolv branch January 23, 2019 12:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants