On Jul 24, 2010, at 11:38 AM, Robert Ramey wrote:
Belcourt, Kenneth wrote:
On Jul 24, 2010, at 7:34 AM, Matthias Troyer wrote:
Eric Niebler wrote:
Guys, we're trying to get a release out. IIUC, a course of action has already been chosen: Robert is to add back the public default constructors that were causing MPI to fail. What's the status of that? Has it in fact fixed the problem on release?
No, it's still broken on trunk. As such, it represents a regression from the previous release.
Is this true? I thought this has been broken since 1.42?
Robert, please (re)read this email I sent to both the testing mailing list and copied you on directly (Note the date was early June, 2010). This commit broke all MPI tests including the remaining broken Sun MPI tests. Begin forwarded message:
From: "Belcourt, Kenneth" <kbelco@sandia.gov> Date: June 7, 2010 7:43:16 PM MDT To: Running Boost regression tests <boost-testing@lists.boost.org> Cc: Robert Ramey <ramey@rrsd.com> Subject: [boost]Trunk MPI tests broken (Serialization bug)
Hi Robert,
It looks like most all the MPI tests may have broken with this checkin.
https://svn.boost.org/trac/boost/changeset/62358/trunk
or at least one of your checkins that touched the archive version_type.
The error points to ../boost/mpi/datatype.hpp:184:3: error: no matching function for call to ???assertion_failed(mpl_::failed************ boost ::mpi::is_mpi_datatype<boost::archive::version_type>::************)??? but from what I can tell neither this file (nor any other MPI source file) has changed in months.
Once it's fixed on trunk and merged into release, I volunteer to swap all trunk testing over to release for a few days to ensure MPI is working correctly before cutting the release. Just as a side note, the only two release branch testers running MPI tests are showing wholesale failure as well (perhaps caused by a premature or partial merge?).
As far as the serialization library is concerned, the only difference between the two branches is that the trunk includes the publication of a default constructor for a couple of types.
I believe that mathias has changes in the trunk that haven't been merged to release yet.
Okay, that would explain it, thanks. -- Noel