Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix CMake warnings concerning dlib_{INCLUDE_DIRS,LIBRARIES} #28

Merged
merged 2 commits into from
May 4, 2021

Conversation

ptosi
Copy link

@ptosi ptosi commented Apr 28, 2021

Silence the following CMake warnings:

CMake Warning at /usr/lib/x86_64-linux-gnu/cmake/dlib/dlibConfig.cmake:42 (message):
  The variable 'dlib_INCLUDE_DIRS' is deprecated! Instead, simply use
  target_link_libraries(your_app dlib::dlib).  See
  http://dlib.net/examples/CMakeLists.txt.html for an example.

and

CMake Warning at /usr/lib/x86_64-linux-gnu/cmake/dlib/dlibConfig.cmake:42 (message):
  The variable 'dlib_LIBRARIES' is deprecated! Instead, simply use
  target_link_libraries(your_app dlib::dlib).  See
  http://dlib.net/examples/CMakeLists.txt.html for an example.

ptosi added 2 commits April 28, 2021 10:39
Silence the following CMake warning:

    CMake Warning at /usr/lib/x86_64-linux-gnu/cmake/dlib/dlibConfig.cmake:42 (message):
      The variable 'dlib_INCLUDE_DIRS' is deprecated! Instead, simply use
      target_link_libraries(your_app dlib::dlib).  See
      http://dlib.net/examples/CMakeLists.txt.html for an example.
Silence the following CMake warning:

    CMake Warning at /usr/lib/x86_64-linux-gnu/cmake/dlib/dlibConfig.cmake:42 (message):
      The variable 'dlib_LIBRARIES' is deprecated! Instead, simply use
      target_link_libraries(your_app dlib::dlib).  See
      http://dlib.net/examples/CMakeLists.txt.html for an example.
@ptosi
Copy link
Author

ptosi commented Apr 28, 2021

Note: this might actually duplicate #22 ...

@severin-lemaignan severin-lemaignan merged commit ce6f7c9 into severin-lemaignan:master May 4, 2021
@severin-lemaignan
Copy link
Owner

Merged, thanks!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants