-
Notifications
You must be signed in to change notification settings - Fork 95
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
property_tree/ptree_serialization.hpp does not include boost/archive/basic_archive.hpp #100
Comments
@urnathan, does this issue still exist? |
@ashtum I'm not in a position to check any more, feel free to close the report, |
Could you please provide some hints on what you were trying to accomplish? I might investigate it myself. |
IIRC the issues I ran into were either That breaks C++20 header-units and clang-modules was the approximation of that that was available to me. Header units should have neither of those behaviours-- the header's sub-include graph should be complete and a DAG. But my memory's somewhat vague about this exact issue. Hope that helps though. |
Here's the second clang-modules issue I found (with this and #99 addressed, I could build ptree_serialization as a submodule).
It refers to some archive/basic_archive details.
'''
-- a/1.77.0/src/boost_1_77_0/boost/property_tree/ptree_serialization.hpp
+++ b/1.77.0/src/boost_1_77_0/boost/property_tree/ptree_serialization.hpp
@@ -10,8 +10,8 @@
#ifndef BOOST_PROPERTY_TREE_PTREE_SERIALIZATION_HPP_INCLUDED
#define BOOST_PROPERTY_TREE_PTREE_SERIALIZATION_HPP_INCLUDED
+#include <boost/archive/basic_archive.hpp>
#include <boost/property_tree/ptree.hpp>
#include <boost/serialization/nvp.hpp>
#include <boost/serialization/collections_save_imp.hpp>
#include <boost/serialization/detail/stack_constructor.hpp>
'''
The text was updated successfully, but these errors were encountered: