Subject: Re: [Boost-bugs] [Boost C++ Libraries] #9301: Error when attempting to boot strap boost
From: Boost C++ Libraries (noreply_at_[hidden])
Date: 2014-04-03 04:29:33
#9301: Error when attempting to boot strap boost
--------------------------------------+----------------------------
Reporter: Ben Key <BKey76@â¦> | Owner:
Type: Bugs | Status: new
Milestone: Boost 1.55.0 | Component: Building Boost
Version: Boost Development Trunk | Severity: Showstopper
Resolution: | Keywords:
--------------------------------------+----------------------------
Comment (by Ben Key):
I think you misunderstand what I was saying. I was not saying that
including Windows.h is wrong. I am saying that assuming that including
Windows.h will automatically cause winioctl.h to be included on every
compiler in existence is wrong. And referring to Microsoft documentation
that is specific to Microsoft Visual Studio does not disprove my point.
When Boost is compiled using MinGW there is a compilation error due to the
use of a constant that is not defined by any of the directly included
header files. You get away with not including the header file in which the
constant is defined on one compiler, but Boost is a library that supports
many compilers and many operating systems.
On one very commonly used compiler including Windows.h does not cause
winioctl.h to be included. The solution is simple, add one line of code to
Boost. Do not say that "Microsoft says thus and we only care about
compilers that do as Microsoft says."
My suggested change is to include winioctl.h in addition to Windows.h
since not all versions of Windows.h in existence cause winioctl.h to be
included.
I would understand your resistance to this change if I was referring to a
compiler that no one uses, but MinGW is a heavily used version of GCC.
-- Ticket URL: <https://svn.boost.org/trac/boost/ticket/9301#comment:2> Boost C++ Libraries <http://www.boost.org/> Boost provides free peer-reviewed portable C++ source libraries.
This archive was generated by hypermail 2.1.7 : 2017-02-16 18:50:15 UTC