Boost C++ Libraries: Ticket #2435: execution_monitor.ipp not portable on slightly older FreeBSD machines https://svn.boost.org/trac10/ticket/2435 <p> boost/test/impl/execution_monitor.ipp </p> <p> In the main switch statements, execution_monitor.ipp is now check for the optional specific signal codes which not all FreeBSD machines support (namely anything less than 7.0 does not set these codes in the kernel nor defines them in /usr/include/sys/signal.h). </p> <p> The attached patch builds on 6,7 and CURRENT and should be innocuous to other platforms. </p> en-us Boost C++ Libraries /htdocs/site/boost.png https://svn.boost.org/trac10/ticket/2435 Trac 1.4.3 pisymbol@… Sat, 25 Oct 2008 15:00:58 GMT attachment set https://svn.boost.org/trac10/ticket/2435 https://svn.boost.org/trac10/ticket/2435 <ul> <li><strong>attachment</strong> → <span class="trac-field-new">execution_monitor.ipp.patch</span> </li> </ul> <p> Patch against current trunk to fix this issue on FreeBSD </p> Ticket Steven Watanabe Thu, 08 Jan 2009 03:29:42 GMT component changed; owner set https://svn.boost.org/trac10/ticket/2435#comment:1 https://svn.boost.org/trac10/ticket/2435#comment:1 <ul> <li><strong>owner</strong> set to <span class="trac-author">Gennadiy Rozental</span> </li> <li><strong>component</strong> <span class="trac-field-old">Building Boost</span> → <span class="trac-field-new">test</span> </li> </ul> Ticket Vladimir Prus Wed, 25 Mar 2009 07:06:58 GMT <link>https://svn.boost.org/trac10/ticket/2435#comment:2 </link> <guid isPermaLink="false">https://svn.boost.org/trac10/ticket/2435#comment:2</guid> <description> <p> Gennadiy, did you look into this patch? </p> <p> Note that a similar issue was reported in <a class="closed ticket" href="https://svn.boost.org/trac10/ticket/1838" title="#1838: Patches: test/impl/execution_monitor.ipp doesn't compile under FreeBSD (closed: fixed)">#1838</a>, and fixed. However, Alexander (the submitter of this issue), reported that this specific issue is still there for him. </p> <p> Alexander, can you double-check that the patch from <a class="closed ticket" href="https://svn.boost.org/trac10/ticket/1838" title="#1838: Patches: test/impl/execution_monitor.ipp doesn't compile under FreeBSD (closed: fixed)">#1838</a> is present in your source tree? </p> </description> <category>Ticket</category> </item> <item> <author>pisymbol@…</author> <pubDate>Wed, 25 Mar 2009 13:18:59 GMT</pubDate> <title/> <link>https://svn.boost.org/trac10/ticket/2435#comment:3 </link> <guid isPermaLink="false">https://svn.boost.org/trac10/ticket/2435#comment:3</guid> <description> <p> 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 <a class="closed ticket" href="https://svn.boost.org/trac10/ticket/1838" title="#1838: Patches: test/impl/execution_monitor.ipp doesn't compile under FreeBSD (closed: fixed)">#1838</a> I want to check against anything less than FreeBSD 7.1 release which should be: </p> <p> #ifdef <span class="underline">FreeBSD_version &lt; 701000 </span></p> <p> Source: <a class="ext-link" href="http://www.freebsd.org/doc/en/books/porters-handbook/freebsd-versions.html"><span class="icon">​</span>http://www.freebsd.org/doc/en/books/porters-handbook/freebsd-versions.html</a> </p> <p> 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!). </p> </description> <category>Ticket</category> </item> <item> <dc:creator>Gennadiy Rozental</dc:creator> <pubDate>Tue, 09 Jun 2009 05:13:12 GMT</pubDate> <title>status changed; resolution set https://svn.boost.org/trac10/ticket/2435#comment:4 https://svn.boost.org/trac10/ticket/2435#comment:4 <ul> <li><strong>status</strong> <span class="trac-field-old">new</span> → <span class="trac-field-new">closed</span> </li> <li><strong>resolution</strong> → <span class="trac-field-new">fixed</span> </li> </ul> Ticket