Subject: Re: [boost] [test] Trunk broken: What happened to test_exec_monitor?
From: Eric Niebler (eric_at_[hidden])
Date: 2011-10-04 16:52:27
On 10/4/2011 1:29 PM, Gennadiy Rozental wrote:
> Eric Niebler <eric <at> boostpro.com> writes:
>> On 10/4/2011 10:04 AM, Dave Abrahams wrote:
>>> on Mon Oct 03 2011, Gennadiy Rozental <rogeeff-AT-gmail.com> wrote:
>>>> I thought release is based on release branch.
>>> It is. But as you've seen over the years, it causes an unworkable
>>> amount of upset and alarm when large numbers of failures appear on
>>> the trunk all at once, and people who would otherwise be dealing with
>>> release issues now have trunk issues to worry about.
>> It also interferes with those of us who use trunk health to know when
>> it's safe to merge fixes to the release branch. We're basically flying
>> blind right now.
> Do you have any particular library in mind?
All of them? I use trunk health to determine when to merge to release
for all of my libraries. And that's the current suggested development
model for all Boost libraries, AFAIK.
>> I'd like to second the call to revert these changes.
> And while we at it, let's revert all the changes to Boost.Build, so I can run my
> own regression test against trunk version of it.
I wasn't aware there was a problem with Boost.Build on trunk. What are
you referring to? Link pls.
> I already reverted test_exec_monitor for now. I can reinstate other deprecated
> interfaces, but let's see if there is actually a problem (aside boost.math).
I think there is. All of my tests that use Boost.Test are crashing at
runtime with any version of gcc. I don't use any deprecated Boost.Test
interfaces, AFAIK. And my tests also fail to compile with msvc 9 and 8
due to this:
..\..\..\boost/test/utils/lazy_ostream.hpp(61) : warning C4181:
qualifier applied to reference type; ignored
c:\boost\org\trunk\libs\xpressive\test\./test.hpp(104) : see
reference to class template instantiation
evType,T,StorageT>' being compiled
PrevType=const boost::unit_test::lazy_ostream &,
StorageT=const std::string &
Gennadiy, you've heard enough people complain now. I implore you: please
revert all your changes, and seriously consider testing them
exhaustively on a branch against *all* of Boost before re-committing
them. And on more compilers than just msvc-10.
-- Eric Niebler BoostPro Computing http://www.boostpro.com
Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk