runtime_error is not a member of std

[tfussell]

runtime_error is not a member of std

[Malvineous]

Also just got a compile error in the test code (not with CMake, but with autoconf, so not sure whether this file is included in the CMake tests?)

diff --git a/tests/helpers/temporary_file.hpp b/tests/helpers/temporary_file.hpp
index 481a7f9..110b978 100644
--- a/tests/helpers/temporary_file.hpp
+++ b/tests/helpers/temporary_file.hpp
@@ -4,7 +4,7 @@
 #include <cstdio>
 #include <string>

-#include <detail/include_windows.hpp>
+#include <xlnt/detail/include_windows.hpp>

 #include "path_helper.hpp"

[tfussell]

Not sure how those slipped by me. I'll update them this evening. Also, thanks for giving it a try. Always nice to hear that someone is using the library.

[tfussell]

The missing include in manifest.cpp has been added in a2919df. For the second one, instead of changing tests/helpers/temporary_file.hpp, you can also add /source as an include directory when configuring with autoconf so it can find the source/detail headers. My using source as an include directory is somewhat unusual so I might move those headers eventually.

[Malvineous]

Thanks for the explanation! I figure tests are a bit unusual anyway in that they need more access to the code than you might make available through the public API, so having the source directory as an include path is probably not that unusual! (In my own code I do things like #include "../source/blah.hpp" which is perhaps even worse!)