Modify

Opened 8 years ago

Closed 2 years ago

#3897 closed Bugs (fixed)

Test framework does not include <osreldate.h> before testing __FreeBSD_version

Reported by: janm-boost-bug@… Owned by: Gennadiy Rozental
Milestone: Boost 1.60.0 Component: test
Version: Boost 1.42.0 Severity: Problem
Keywords: test freebsd osreldate version Cc:

Description

The file boost/test/impl/execution_monitor.ipp tests FreeBSD_version without including <osreldate.h> first. This causes compilation warnings about preprocessor symbol redefinitions.

Patch to fix the problem:

diff -dur boost_1_42_0.orig/boost/test/impl/execution_monitor.ipp boost_1_42_0/boost/test/impl/execution_monitor.ipp --- boost_1_42_0.orig/boost/test/impl/execution_monitor.ipp 2009-11-28 20:19:18.000000000 +1100 +++ boost_1_42_0/boost/test/impl/execution_monitor.ipp 2010-02-04 21:23:20.301652051 +1100 @@ -155,6 +155,7 @@

# include <setjmp.h>

# if defined(FreeBSD)

+# include <osreldate.h>

# ifndef SIGPOLL # define SIGPOLL SIGIO

Attachments (0)

Change History (5)

comment:1 Changed 7 years ago by Steven Watanabe

Resolution: fixed
Status: newclosed

(In [62780]) #include <osreldate.h> on FreeBSD. Fixes #3897

comment:2 Changed 4 years ago by chmeeedalf+boost@…

Can somebody please merge this into the branches? It's not in as of 1.54-Beta1 that I can see, and it causes Boost to fail to build on non-x86 FreeBSD.

comment:3 Changed 4 years ago by anonymous

Resolution: fixed
Status: closedreopened

comment:4 Changed 4 years ago by anonymous

Reopen for merge request.

comment:5 Changed 2 years ago by Raffi Enficiaud

Milestone: Boost 1.43.0Boost 1.60.0
Resolution: fixed
Status: reopenedclosed

Milestone set to 1.60, but fix appeared in 1.59.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Gennadiy Rozental.
The resolution will be deleted.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.