BIND 10 #2475: Split libdhcp++ and libdhcp_srv into separate folders.

BIND 10 Development do-not-reply at isc.org
Fri Nov 16 14:18:25 UTC 2012


#2475: Split libdhcp++ and libdhcp_srv into separate folders.
-------------------------------------+-------------------------------------
                   Reporter:         |                 Owner:  marcin
  marcin                             |                Status:  reviewing
                       Type:  task   |             Milestone:  Sprint-
                   Priority:         |  DHCP-20121129
  medium                             |            Resolution:
                  Component:  dhcp   |             Sensitive:  0
                   Keywords:         |           Sub-Project:  DHCP
            Defect Severity:  N/A    |  Estimated Difficulty:  0
Feature Depending on Ticket:         |           Total Hours:  0
        Add Hours to Ticket:  0      |
                  Internal?:  0      |
-------------------------------------+-------------------------------------
Changes (by stephen):

 * owner:  stephen => marcin


Comment:

 > The copyright headers should be updated to 2012:
 Updated.

 > at this point I found that many other files in src/lib still have wrong
 order of includes. Is the reason for this that you aimed to fix includes
 order in only those files that you had to touch anyway when you were
 changing the path to dhcpsrv headers?
 That's what I originally did, but in response to your comments, as well as
 the files you mention I thought that I may as well complete the job for
 lib/dhcp, lib/dhcpsrv, bin/dhcp4 and bin/dhcp6 (and the various "test"
 directories). The order of inclusion of header files now follows the
 coding guidelines, being in the order:

 1. config.h
 1. BIND 10 header files.
 1. Package header files. (e.g. Boost, MySQL, GTest)
 1. C++ header files.
 1. C header files.

 I've also altered the {{{#include "xxx.h"}}} forms of the directive to
 {{{#include <directory/xxx.h>}}}, again following the coding guidelines.
 In addition, I checked that the "#include" sentinels conform to the format
 {{{#ifndef FILENAME_H}}}.

 Suggested !ChangeLog entry:
 {{{
 506.    [bug]           stephen
         Split the DHCP source files into two directories, each with its
 own Makefile.
         This properly solves the problem whereby a "make" operation with
 multiple
         threads could fail because of dependencies between the two
 libraries in the
         same directory.
         (Trac #2475, git xxx)
 }}}

-- 
Ticket URL: <http://bind10.isc.org/ticket/2475#comment:7>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development


More information about the bind10-tickets mailing list