Add note about glog not currently compiling against gflags 2.1. - Also fix a typo in pointer to EIGENSPARSE documentation. Change-Id: I1448fcf3680a65362c6f9b2d8a5ab0259da92364
diff --git a/docs/source/building.rst b/docs/source/building.rst index f739f38..f77fe43 100644 --- a/docs/source/building.rst +++ b/docs/source/building.rst
@@ -31,7 +31,7 @@ .. NOTE :: Ceres can also use Eigen as a sparse linear algebra - library. Please see the documentation for ``-DEIGENSPARSE`` for` + library. Please see the documentation for ``EIGENSPARSE`` for more details. - `CMake <http://www.cmake.org>`_ 2.8.0 or later. @@ -52,6 +52,14 @@ Android due to the various performance and functionality compromises in** ``miniglog``. + .. NOTE :: + + If you are compiling ``glog`` from source, please note that currently, + the unit tests for ``glog`` (which are enabled by default) do not compile + against a default build of ``gflags`` 2.1 as the gflags namespace changed + from ``google::`` to ``gflags::``. A patch to fix this is available from + `here <https://code.google.com/p/google-glog/issues/detail?id=194>`_. + - `Google Flags <http://code.google.com/p/gflags>`_. Needed to build examples and tests.