CMake support

[sergiud]

This series of patches adds CMake support to glog and fixes several compilation issues.

My primary goal was to be able to compile glog on Windows using Visual Studio without any hassle. The modifications have been tested using VC 8.0 through VC 14.0 CTP 6, cygwin gcc 4.9.2, Intel C++ Compiler 15.0, Apple Clang, and on Arch Linux using gcc 4.9.2 and clang 3.6 (both static and shared builds).

CMakeLists.txt also generates config files, which allow to use glog in CMake projects easily:

cmake_minimum_required (VERSION 2.8.11)
project (myproject)

find_package (glog 0.3.4)

add_executable (myproject myproject.cpp)
target_link_libraries (myproject glog)

To run the unit tests, execute:

make test

when using the Makefiles generator.

[ukai]

on ubuntu/trusty at commit 82823b0

$ ./configure && make
...
In file included from src/demangle.cc:38:0:
src/demangle.h:80:27: error: expected initializer before 'Demangle'
bool GOOGLE_GLOG_DLL_DECL Demangle(const char mangled, char *out, int out_size);
^
make: ** [src/libglog_la-demangle.lo] Error 1

[ukai]

same error at commit 0caa067

[sergiud]

It's commit ab0a2da that causes the problem. The demangle.h header is missing an #include "config.h". I'll fix that.

[sergiud]

Actually, GOOGLE_GLOG_DLL_DECL was missing. I fixed that.

[ukai]

still failing with autoconf at commit 44a22f8

$ ./configure && make
...
libtool: compile: g++ -DHAVE_CONFIG_H -I. -I./src -I./src -Wall -Wwrite-strings
-Woverloaded-virtual -Wno-sign-compare -DNO_FRAME_POINTER -DNDEBUG -g -O2 -MT s
rc/libglog_la-logging.lo -MD -MP -MF src/.deps/libglog_la-logging.Tpo -c src/log
ging.cc -fPIC -DPIC -o src/.libs/libglog_la-logging.o
In file included from src/utilities.h:82:0,
from src/logging.cc:32:
src/glog/logging.h:290:12: error: 'GOOGLE_GLOG_DLL_DECL' does not name a type
extern GOOGLE_GLOG_DLL_DECL type FLAGS_##name;
^

[sergiud]

I'm not seeing this error on Arch Linux. GOOGLE_GLOG_DLL_DECL is clearly defined in config.h.in. See sergiud@0938ff4#diff-d23f5b7090d56af546252d03336564a3R182.

[sergiud]

I can reproduce the problem on Ubuntu 12.04. Will look into it.

[sergiud]

Can you please check again?

[gregoire-astruc]

Will this be approved and merged? This is definitely something I'm looking for.

[eelstork]

Looks promising; also interested in this.

[haosdent]

+1

[curtpm]

+1

[haosdent]

@hausdorff glog support cmake now :-)

[eelstork]

Good work!

[jakirkham]

When will this be included in a release?