id,summary,reporter,owner,description,type,status,milestone,component,version,severity,resolution,keywords,cc
3868,bimap build fails without serialization library,jdsiiro@…,Matias Capeletto,"Builds including Bimap fail unless the Serialization library is present (even when defining `BOOST_BIMAP_DISABLE_SERIALIZATION`).  This is caused by a reference to a missing header file (`boost/serialization/nvp.hpp`).  A fix appears to be to guard those `#include`s within `#ifndef`s:
{{{
#!diff
Index: detail/set_view_iterator.hpp
===================================================================
--- detail/set_view_iterator.hpp	(revision 2317)
+++ detail/set_view_iterator.hpp	(working copy)
@@ -19,7 +19,9 @@
 #include <boost/config.hpp>
 
 // Boost
-#include <boost/serialization/nvp.hpp>
+#ifndef BOOST_BIMAP_DISABLE_SERIALIZATION
+    #include <boost/serialization/nvp.hpp>
+#endif
 #include <boost/iterator/detail/enable_if.hpp>
 #include <boost/iterator/iterator_adaptor.hpp>
 #include <boost/bimap/relation/support/get_pair_functor.hpp>
Index: detail/map_view_iterator.hpp
===================================================================
--- detail/map_view_iterator.hpp	(revision 2317)
+++ detail/map_view_iterator.hpp	(working copy)
@@ -19,7 +19,9 @@
 #include <boost/config.hpp>
 
 // Boost
-#include <boost/serialization/nvp.hpp>
+#ifndef BOOST_BIMAP_DISABLE_SERIALIZATION
+    #include <boost/serialization/nvp.hpp>
+#endif
 #include <boost/iterator/detail/enable_if.hpp>
 #include <boost/iterator/iterator_adaptor.hpp>
 #include <boost/bimap/relation/support/pair_by.hpp>
Index: relation/mutant_relation.hpp
===================================================================
--- relation/mutant_relation.hpp	(revision 2317)
+++ relation/mutant_relation.hpp	(working copy)
@@ -21,7 +21,9 @@
 #include <boost/mpl/vector.hpp>
 #include <boost/operators.hpp>
 #include <boost/call_traits.hpp>
-#include <boost/serialization/nvp.hpp>
+#ifndef BOOST_BIMAP_DISABLE_SERIALIZATION
+    #include <boost/serialization/nvp.hpp>
+#endif
 
 #include <boost/functional/hash/hash.hpp>
}}}

In addition, this exposes a missing dependence on `boost/throw_exception.hpp` that can be resolved (albeit, non-ideally) through the following:
{{{
#!diff
Index: bimap.hpp
===================================================================
--- bimap.hpp	(revision 2317)
+++ bimap.hpp	(working copy)
@@ -55,6 +55,8 @@
 
 #ifndef BOOST_BIMAP_DISABLE_SERIALIZATION
     #include <boost/serialization/nvp.hpp>
+#else
+    #include <boost/throw_exception.hpp>
 #endif // BOOST_BIMAP_DISABLE_SERIALIZATION
 
 // Boost.Bimap
}}}",Bugs,closed,Boost 1.42.0,bimap,Boost 1.41.0,Problem,fixed,,