Re: [Boost-mpi] [EXTERNAL] Develop VS master

Subject: Re: [Boost-mpi] [EXTERNAL] Develop VS master
From: Alain Miniussi (Alain.Miniussi_at_[hidden])
Date: 2017-07-17 04:33:29

Hi Noel,

On 17/07/2017 03:11, Belcourt, Kenneth wrote:
> Hi Alain,
>
> On Jul 15, 2017, at 7:28 AM, Alain Miniussi via Boost-mpi 
> <boost-mpi_at_[hidden] <mailto:boost-mpi_at_[hidden]>> wrote:
>>
>> It passes with:
>>
>>        - Intel compiler  17.0.2
>>        - library IMPI 5.0.3.048
>>
>>
>> On 15/07/2017 14:08, Alain Miniussi via Boost-mpi wrote:
>>> https://github.com/boostorg/mpi/pull/49
>
> I just ran graph, graph-parallel, and mpi tests with latest pull of 
> master.  All the tests passed provided they’re built with std=c++11. 
>  This is on my Mac with OpenMPI 1.8.4 and clang, gcc and intel-17.  Is 
> the failure you mention platform or toolset specific?
It is not platform specific, but goes unnoticed if you have a 
distribution boost installed as the compiler will use the 
/usr/include/boost/serialization/detail/get_data.hpp.
I have it passing without the std=c++11.

Alain

>  I’ve been gone for quite a while so thanks for working to get things 
> fixed up.
>
> Noel
>
>>>
>>>
>>> On 15/07/2017 14:05, Alain Miniussi via Boost-mpi wrote:
>>>> Hi,
>>>>
>>>> I just realized that people with a platform boost installed had 
>>>> their 1.64 distrib building for the wrong reasons... (as long as a 
>>>> "correct" get_data is included, it can come from /usr...).
>>>>
>>>> So, considering:
>>>>
>>>> a) Since the 1.65 process is ongoing, huge merges should be avoided.
>>>> b) The lib needs to live on, develop should be merge quickly so 
>>>> that we can get some work done.
>>>> c) The 1.65 should build for good reason.
>>>> d) current fixes for the get_data issue are in merges against develop.
>>>>
>>>> What about the following proposal:
>>>>
>>>> I will manually merge the minimal subset of 
>>>> https://github.com/boostorg/mpi/pull/47 
>>>> <https://github.com/boostorg/mpi/pull/47> that get master passing 
>>>> the test even when no boost header are present in /usr/include.
>>>>
>>>> It's in branch get_data_fix for those interested. Any issue/build 
>>>> report is welcomed.
>>>>
>>>> Then, short after 1.65 is out, develop will be merged into master 
>>>> unless someone has issues with it.
>>>>
>>>> People can already have a look at 
>>>> https://github.com/boostorg/mpi/pull/46 
>>>> <https://github.com/boostorg/mpi/pull/46> and 
>>>> https://github.com/boostorg/mpi/pull/48 
>>>> <https://github.com/boostorg/mpi/pull/48>.
>>>>
>>>> I'll answer to the todo list question in another mail.
>>>>
>>>> Regards.
>>>>
>>>>
>>>> _______________________________________________
>>>> Boost-mpi mailing list
>>>> Boost-mpi_at_[hidden] <mailto:Boost-mpi_at_[hidden]>
>>>> https://lists.boost.org/mailman/listinfo.cgi/boost-mpi
>>>
>>> _______________________________________________
>>> Boost-mpi mailing list
>>> Boost-mpi_at_[hidden] <mailto:Boost-mpi_at_[hidden]>
>>> https://lists.boost.org/mailman/listinfo.cgi/boost-mpi
>>
>> _______________________________________________
>> Boost-mpi mailing list
>> Boost-mpi_at_[hidden] <mailto:Boost-mpi_at_[hidden]>
>> https://lists.boost.org/mailman/listinfo.cgi/boost-mpi
>