Boost logo

Boost :

Subject: Re: [boost] [1.46] Last few merges needed?
From: Rene Rivera (grafikrobot_at_[hidden])
Date: 2011-01-29 16:55:45


On 1/29/2011 12:38 PM, Joachim Faulhaber wrote:
> Dear release managers,
>
> as discussed before, I've done my homework related the bug that I've
> detected, which is described in ticket #5135:
> https://svn.boost.org/trac/boost/ticket/5135. See details below.
>
> 2011/1/27 Joachim Faulhaber<afojgo_at_[hidden]>:
>> 2011/1/27 Rene Rivera<grafikrobot_at_[hidden]>:
>>> On 1/27/2011 10:48 AM, Joachim Faulhaber wrote:
>>>> Such last minutes surprises are always not very welcome, I know ... I
>>>> won't take your decision personal ;)
>>>
>>> After a brief talk on IRC.. How about if you..
>>>
>>> 1. Add a test to show the problem in trunk. Passing with the fix in it of
>>> course.
>
> 1.1: I've fixed the code (see patchfile)
> Index: is_interval.hpp
> ===================================================================
> --- is_interval.hpp (revision 68546)
> +++ is_interval.hpp (working copy)
> @@ -26,7 +26,7 @@
> {
> typedef is_interval<Type> type;
> BOOST_STATIC_CONSTANT(bool,
> - value = (interval_bound_type<Type>::value<
> interval_bounds::undefined));
> + value = ((interval_bound_type<Type>::value)<
> interval_bounds::undefined));
> };
>
>
> @@ -34,7 +34,7 @@
> {
> typedef has_static_bounds<Type> type;
> BOOST_STATIC_CONSTANT(bool,
> - value = (interval_bound_type<Type>::value<
> interval_bounds::dynamic));
> + value = ((interval_bound_type<Type>::value)<
> interval_bounds::dynamic));
> };
>
> template<class Type> struct has_dynamic_bounds
> ===================================================================
>
> 1.2 I've added two test to the trunk that test the bug fix:
>
> (1.2.1) A minimal test containing the bug and it's fix
> http://svn.boost.org/svn/boost/trunk/libs/icl/test/fix_include_after_thread_/fix_include_after_thread.cpp
>
> (1.2.2) A test with includes of ICL-files that caused the error before
> when preceeded by boost/thread.hpp
> http://svn.boost.org/svn/boost/trunk/libs/icl/test/fix_icl_after_thread_/fix_icl_after_thread.cpp
>
> The tests ran successfully for 2 nights.
>
>>> 2. Wait for the beta to get done.
> Has happened tonight.
>
>>> 3. Then you can ask again to make this change after the beta is out. And
>>> we'll likely say yes.
>
> I have prepared a merge that does the fix and adds the two new tests.
> Do I have permission now to commit the merge to the release branch?

I'm going to say OK. One worry I have is that some of the release
toolsets haven't run that test yet on the trunk :-( But there's nothing
you can do about that. And they are the older set of compilers anyway
(i.e. msvc8).

And thanks for being patient with us :-)

PS. It seems some toolset should be marked as N/A as they are failing
most of the ICL lib tests.

PPS. Yes, I'm breaking my own no-work-in-the-weekends rule :-(

-- 
-- Grafik - Don't Assume Anything
-- Redshift Software, Inc. - http://redshift-software.com
-- rrivera/acm.org (msn) - grafik/redshift-software.com
-- 102708583/icq - grafikrobot/aim,yahoo,skype,efnet,gmail

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