Commit ad317726 authored by kumagi's avatar kumagi Committed by Facebook Github Bot

find_package(gflags CONFIG QUIET) didn't works in Ubuntu.

Summary:
in Linux environment, `find_package(gflass)` searches `Findgflags.cmake`.
And it cannot find  https://github.com/facebook/folly/blob/master/CMake/FindGFlags.cmake , because linux is case-sensitive environment in filenames and cause error.

```
CMake Error at CMake/folly-deps.cmake:34 (message):
  Unable to determine the target name for the GFlags package.
Call Stack (most recent call first):
  CMakeLists.txt:65 (include)
```

For linux programmers, this line should find `FindGFlags.cmake` file which exists in the same directory.
Closes https://github.com/facebook/folly/pull/805

Reviewed By: yfeldblum

Differential Revision: D7505908

Pulled By: Orvid

fbshipit-source-id: c30097a25569d293df9a1bfecbee84cece260306
parent 66405fc6
...@@ -22,7 +22,7 @@ list(APPEND FOLLY_LINK_LIBRARIES ${DOUBLE_CONVERSION_LIBRARY}) ...@@ -22,7 +22,7 @@ list(APPEND FOLLY_LINK_LIBRARIES ${DOUBLE_CONVERSION_LIBRARY})
list(APPEND FOLLY_INCLUDE_DIRECTORIES ${DOUBLE_CONVERSION_INCLUDE_DIR}) list(APPEND FOLLY_INCLUDE_DIRECTORIES ${DOUBLE_CONVERSION_INCLUDE_DIR})
set(FOLLY_HAVE_LIBGFLAGS OFF) set(FOLLY_HAVE_LIBGFLAGS OFF)
find_package(gflags CONFIG QUIET) find_package(GFlags CONFIG QUIET)
if (gflags_FOUND) if (gflags_FOUND)
message(STATUS "Found gflags from package config") message(STATUS "Found gflags from package config")
set(FOLLY_HAVE_LIBGFLAGS ON) set(FOLLY_HAVE_LIBGFLAGS ON)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment