commit 5d00bf40f597e6b4542972fb81b3e1c93ea0cb73
author Sameer Agarwal <sameeragarwal@google.com> Mon Jun 24 13:24:35 2013 -0700
committer Sameer Agarwal <sameeragarwal@google.com> Mon Jun 24 13:24:35 2013 -0700
tree 312a4d47dec48e35decd341f33a3623133e5387d
parent c367b12eebe2657457822225d7f8db54b195f34e

Fix the broken build.

1. When protocol buffers support is enabled CompressedRowSparseMatrix
   has a missing virtual method.
2. When SuiteSparse is missing, covariance_test tries to run the
   large scale covariance computation test.

Thanks to Alex Stewart for reporting #1.

Change-Id: I4238c966036362175e31749595ea8bb6f12a696c

internal/ceres/covariance_test.cc

diff --git a/internal/ceres/covariance_test.cc b/internal/ceres/covariance_test.cc
index bd0a0e2..f9c5cd4 100644
--- a/internal/ceres/covariance_test.cc
+++ b/internal/ceres/covariance_test.cc
@@ -398,8 +398,11 @@
   };
 
   Covariance::Options options;
+
+#ifndef CERES_NO_SUITESPARSE
   options.use_dense_linear_algebra = false;
   ComputeAndCompareCovarianceBlocks(options, expected_covariance);
+#endif
 
   options.use_dense_linear_algebra = true;
   ComputeAndCompareCovarianceBlocks(options, expected_covariance);
@@ -441,8 +444,10 @@
   Covariance::Options options;
   options.num_threads = 4;
 
+#ifndef CERES_NO_SUITESPARSE
   options.use_dense_linear_algebra = false;
   ComputeAndCompareCovarianceBlocks(options, expected_covariance);
+#endif
 
   options.use_dense_linear_algebra = true;
   ComputeAndCompareCovarianceBlocks(options, expected_covariance);
@@ -484,8 +489,11 @@
   };
 
   Covariance::Options options;
+
+#ifndef CERES_NO_SUITESPARSE
   options.use_dense_linear_algebra = false;
   ComputeAndCompareCovarianceBlocks(options, expected_covariance);
+#endif
 
   options.use_dense_linear_algebra = true;
   ComputeAndCompareCovarianceBlocks(options, expected_covariance);
@@ -533,10 +541,12 @@
    -0.00122,  -0.00122,  -0.00149,  -0.00298,   0.00000,   0.03457
   };
 
-
   Covariance::Options options;
+
+#ifndef CERES_NO_SUITESPARSE
   options.use_dense_linear_algebra = false;
   ComputeAndCompareCovarianceBlocks(options, expected_covariance);
+#endif
 
   options.use_dense_linear_algebra = true;
   ComputeAndCompareCovarianceBlocks(options, expected_covariance);