Re: [Boost-bugs] [Boost C++ Libraries] #11908: mpi_process_group.cpp uses undefined class 'std::list'

Subject: Re: [Boost-bugs] [Boost C++ Libraries] #11908: mpi_process_group.cpp uses undefined class 'std::list'
From: Boost C++ Libraries (noreply_at_[hidden])
Date: 2016-01-17 19:56:16

#11908: mpi_process_group.cpp uses undefined class 'std::list'
-------------------------------------+-----------------------------
  Reporter:  terry lyons <tlyons@…>  |      Owner:  jewillco
      Type:  Bugs                    |     Status:  new
 Milestone:  To Be Determined        |  Component:  graph
   Version:  Boost 1.61.0            |   Severity:  Showstopper
Resolution:                          |   Keywords:  include missing
-------------------------------------+-----------------------------

Comment (by terry lyons <tlyons@…>):

 Attempts to build boost for msvc environments with MS MPI fall down in two
 places.

 One reason is that mpi.jam needs modifying to be aware of current path and
 environment conventions (there is also a workaround definition needed for
 an irritating broken Microsoft macro).

 Then there are these problems with graph documented above. Adding an
 include of <list> to mpi_process_group.ipp and added the long list of
 "std::" includes one finds in mpi_process_group.ipp to
 mpi_process_group.cpp including <list> (not tested that these latter
 includes are really needed) fix these errors.

 After both changes are implemented boost builds error free on msMPI v7. I
 wonder how long this has been broken? It seems strange to me as I would
 have thought that making sure boost worked on the MS stack was important
 to many people.

 It would be good to have this properly fixed. I have uploaded the modified
 mpi.jam, mpi_process_group.ipp and mpi_process_group.cpp . I hope they are
 useful.

 I should add - when I say error free, I mean error free, as there are
 plenty of warnings about size_t being converted to int so maybe it still
 fails on some test!

 I realise that this is all written by volunteers and I am very grateful,
 but I am surprised that one finds these 64/32 conversion errors in mature
 code such as boost and hope it does not lead to malicious opportunities
 for denial of service etc. in some loop.

-- 
Ticket URL: <https://svn.boost.org/trac/boost/ticket/11908#comment:1>
Boost C++ Libraries <http://www.boost.org/>
Boost provides free peer-reviewed portable C++ source libraries.