[release][poly_collection] Permission to merge to release

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

[release][poly_collection] Permission to merge to release

Boost - Dev mailing list
Hello,

I'm asking for permission to merge the following commit chain into release:

https://github.com/boostorg/poly_collection/commit/3609895bd01e5add5971345a3e7d2d220e654020
https://github.com/boostorg/poly_collection/commit/18b7d88df360001c2c006f021136a95bceacd29b
https://github.com/boostorg/poly_collection/commit/ce7e7aac60747c6fad5e97d2cbc22f814b14d886
https://github.com/boostorg/poly_collection/commit/ab32df47f8811dc7f136a166f3b957a0325fde60
https://github.com/boostorg/poly_collection/commit/df754527917b7be013ce558f8c0286aa7a0cc5b9

Fixes bring compatibility with Clang 3.3 and solve a potential UB
problem related to CWG 232 and
detected at

http://www.boost.org/development/tests/develop/developer/Marshall-UBSAN-poly_collection-clang-darwin-darwin-ubsan-test_construction-variants_.html

I'm waiting for Marshall-UBSAN to cycle through these but all recently
updated test runners are
working fine. Clang 3.3 is already verified to work by
teeks99-02-dc3.3-11-Docker-64on64.

Thank you,

Joaquín M López Muñoz


_______________________________________________
Unsubscribe & other changes: http://lists.boost.org/mailman/listinfo.cgi/boost
Reply | Threaded
Open this post in threaded view
|

Re: [release][poly_collection] Permission to merge to release

Boost - Dev mailing list
On 22 November 2017 at 10:23, Joaquin M López Muñoz via Boost
<[hidden email]> wrote:

> Hello,
>
> I'm asking for permission to merge the following commit chain into release:
>
> https://github.com/boostorg/poly_collection/commit/3609895bd01e5add5971345a3e7d2d220e654020
> https://github.com/boostorg/poly_collection/commit/18b7d88df360001c2c006f021136a95bceacd29b
> https://github.com/boostorg/poly_collection/commit/ce7e7aac60747c6fad5e97d2cbc22f814b14d886
> https://github.com/boostorg/poly_collection/commit/ab32df47f8811dc7f136a166f3b957a0325fde60
> https://github.com/boostorg/poly_collection/commit/df754527917b7be013ce558f8c0286aa7a0cc5b9
>
> Fixes bring compatibility with Clang 3.3 and solve a potential UB problem
> related to CWG 232 and
> detected at
>
> http://www.boost.org/development/tests/develop/developer/Marshall-UBSAN-poly_collection-clang-darwin-darwin-ubsan-test_construction-variants_.html
>
> I'm waiting for Marshall-UBSAN to cycle through these but all recently
> updated test runners are
> working fine. Clang 3.3 is already verified to work by
> teeks99-02-dc3.3-11-Docker-64on64.

The test results for poly_collection aren't working at the moment, so
I'll take your word for it. Go ahead.

_______________________________________________
Unsubscribe & other changes: http://lists.boost.org/mailman/listinfo.cgi/boost
Reply | Threaded
Open this post in threaded view
|

Re: [release][poly_collection] Permission to merge to release

Boost - Dev mailing list
El 22/11/2017 a las 12:23, Daniel James via Boost escribió:

> On 22 November 2017 at 10:23, Joaquin M López Muñoz via Boost
> <[hidden email]> wrote:
>> Hello,
>>
>> I'm asking for permission to merge the following commit chain into release:
>>
>> [...]
>>
>> I'm waiting for Marshall-UBSAN to cycle through these but all recently
>> updated test runners are
>> working fine. Clang 3.3 is already verified to work by
>> teeks99-02-dc3.3-11-Docker-64on64.
> The test results for poly_collection aren't working at the moment, so
> I'll take your word for it. Go ahead.

They are back to life now. Merge done.

Joaquín M López Muñoz


_______________________________________________
Unsubscribe & other changes: http://lists.boost.org/mailman/listinfo.cgi/boost