Skip to content

Commit

Permalink
Add note about glog not currently compiling against gflags 2.1.
Browse files Browse the repository at this point in the history
- Also fix a typo in pointer to EIGENSPARSE documentation.

Change-Id: I1448fcf3680a65362c6f9b2d8a5ab0259da92364
  • Loading branch information
alexsmac committed Jan 6, 2015
1 parent 60cc520 commit 257f6da
Showing 1 changed file with 9 additions and 1 deletion.
10 changes: 9 additions & 1 deletion docs/source/building.rst
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ optional. For details on customizing the build process, see
.. 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.
Expand All @@ -52,6 +52,14 @@ optional. For details on customizing the build process, see
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.

Expand Down

0 comments on commit 257f6da

Please sign in to comment.