[mlpack] 15/53: Fix failing HDF5 tests (handle old Armadillo bug that was probably my fault anyway).

[Barak A. Pearlmutter]

This is an automated email from the git hooks/post-receive script.

bap pushed a commit to branch master
in repository mlpack.

commit 5aa60657013edaf758f02c04f615468ee218cfb7
Author: Ryan Curtin <ryan at ratml.org>
Date:   Wed Oct 12 22:34:41 2016 -0400

    Fix failing HDF5 tests (handle old Armadillo bug that was probably my fault anyway).
---
 src/mlpack/tests/load_save_test.cpp | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/src/mlpack/tests/load_save_test.cpp b/src/mlpack/tests/load_save_test.cpp
index 9cf7cb6..5c9a0b8 100644
--- a/src/mlpack/tests/load_save_test.cpp
+++ b/src/mlpack/tests/load_save_test.cpp
@@ -518,6 +518,12 @@ BOOST_AUTO_TEST_CASE(SavePGMBinaryTest)
   remove("test_file.pgm");
 }
 
+// Don't perform any HDF5 tests on Armadillo 4.300-4.400 (inclusive).  A bug
+// causes loading to fail.
+#if ((ARMA_VERSION_MAJOR == 4) && \
+        (ARMA_VERSION_MINOR < 300 || ARMA_VERSION_MINOR > 400)) || \
+    (ARMA_VERSION_MAJOR >= 5)
+
 #if defined(ARMA_USE_HDF5)
 /**
  * Make sure load as HDF5 is successful.
@@ -650,6 +656,8 @@ BOOST_AUTO_TEST_CASE(NoHDF5Test)
 }
 #endif
 
+#endif
+
 /**
  * Test normalization of labels.
  */

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/debian-science/packages/mlpack.git