Re: [Boost-bugs] [Boost C++ Libraries] #9301: Error when attempting to boot strap boost

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