Re: [Boost-bugs] [Boost C++ Libraries] #2435: execution_monitor.ipp not portable on slightly older FreeBSD machines

Subject: Re: [Boost-bugs] [Boost C++ Libraries] #2435: execution_monitor.ipp not portable on slightly older FreeBSD machines
From: Boost C++ Libraries (noreply_at_[hidden])
Date: 2009-03-25 13:19:00

#2435: execution_monitor.ipp not portable on slightly older FreeBSD machines
--------------------------------------+-------------------------------------
  Reporter:  pisymbol_at_[hidden]       |       Owner:  rogeeff
      Type:  Bugs                     |      Status:  new    
 Milestone:  Boost 1.37.0             |   Component:  test   
   Version:  Boost Development Trunk  |    Severity:  Problem
Resolution:                           |    Keywords:         
--------------------------------------+-------------------------------------

Comment(by pisymbol_at_[hidden]):

 It is not and I am synced with trunk.  The other thing is that the patch
 has a bug in it as attached (I don't know how this happened) but as per
 #1838 I want to check against anything less than FreeBSD 7.1 release which
 should be:

 #ifdef __FreeBSD_version < 701000

 Source: http://www.freebsd.org/doc/en/books/porters-handbook/freebsd-

 versions.html

 Other than that, I believe my patch is correct.  One thing is I made it
 per signal test configurable for maximum flexibility at the cost of #ifdef
 ugliness.  The other patch just skipped the tests altogether.  I leave it
 up to test to do the right thing here.  I believe my patch as it stands
 does not effect other platforms but a code review would be nice (since I
 myself found one bug!).

-- 
Ticket URL: <https://svn.boost.org/trac/boost/ticket/2435#comment:3>
Boost C++ Libraries <http://www.boost.org/>
Boost provides free peer-reviewed portable C++ source libraries.