Boost logo

Boost :

Subject: Re: [boost] Permission to proceed?
From: Stephen Kelly (steveire_at_[hidden])
Date: 2013-10-13 04:28:31


On 10/12/2013 05:11 PM, Daniel James wrote:
> On 12 October 2013 14:19, Stephen Kelly <steveire_at_[hidden]> wrote:
>> On 10/11/2013 11:49 PM, Rene Rivera wrote:
>>> In this case we wanted to hold off on trunk commits that had a high
>>> probability of breaking many things while we worked on the beta. As people
>>> needed to get test cycled as quickly as possible.
>> You mean 'high risk', not 'high possibility'. No breakage from those
>> patches yet :).
> Sadly, that doesn't mean much. The test results haven't updated since
> Thursday and a lot of libraries aren't checked regularly. For one
> example, iostreams is mostly unmaintained, although I occasionally
> review patches. I had a quick look at the changes, and noticed this in
> 'boost/iostreams/chain.hpp':
>
> -#if !BOOST_WORKAROUND(BOOST_MSVC, < 1310)
> +#if !BOOST_WORKAROUND(BOOST_MSVC, == 1310)

That's part of a private/template friend workaround. Elsewhere in boost
that workaround is applied for MSVC 7.1, so I concluded the < 1310 was
a mistake, and it should have been a <= 1310 before.

That's unrelated to the new patches though. That patch is from September.

> Which looks wrong to me. That was on the second screen of the changes,
> and there are a lot more screens after that. The rest of it might be
> fine, but it's unlikely that they'll get checked soon.
>
> Also, I asked you not to make these changes until after the release
> was made here:
>
> http://lists.boost.org/Archives/boost/2013/09/206537.php
>
> Which you seemed to agree to.

Sorry. I didn't see any connection between the release (on the release
branch), and trunk. I asked about that, but didn't get useful
information back, so I asked again for permission to proceed.

I'll stop committing until a few days after the final release.

> We should wait for a little while after
> the release to give people a chance to respond to the release notes.

> You also should notify library maintainers before making non-trivial
> changes to their libraries.

I'm doing that by writing to this list, right?

Thanks,

Steve.


Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk