BIND 10 #3107: [kean] ./configure report versions of dependencies

BIND 10 Development do-not-reply at isc.org
Wed Nov 13 07:39:32 UTC 2013


#3107: [kean] ./configure report versions of dependencies
-------------------------------------+-------------------------------------
            Reporter:  jreed         |                        Owner:  kean
                Type:  task          |                       Status:
            Priority:  medium        |  reviewing
           Component:  build system  |                    Milestone:
            Keywords:                |  Sprint-20131015
           Sensitive:  0             |                   Resolution:
         Sub-Project:  Core          |                 CVSS Scoring:
Estimated Difficulty:  5             |              Defect Severity:  N/A
         Total Hours:  0             |  Feature Depending on Ticket:
                                     |          Add Hours to Ticket:  0
                                     |                    Internal?:  0
-------------------------------------+-------------------------------------
Changes (by muks):

 * owner:  UnAssigned => kean


Comment:

 Hi Kean

 Replying to [comment:18 kean]:
 > Replying to [comment:17 muks]:
 >
 > > * The following cannot work:
 > > {{{
 > > +CXX_VERSION=`$CXX-V 2> /dev/null | head -1`
 > > }}}
 > >
 > > This implies it was not tested.
 > No, it means it wasn't tested for that compiler. It was tested on two
 completely separate platforms and things worked, it just happened that
 neither of those used SunStudio. Sorry for the typo. It has been
 corrected.

 I don't understand why you claim the above. The whole section is only for
 that Solaris compiler, as you can see:
 {{{
 # SunStudio compiler requires special compiler options for boost
 # (http://blogs.sun.com/sga/entry/boost_mini_howto)
 if test "$SUNCXX" = "yes"; then
 CXX_VERSION=`$CXX-V 2> /dev/null | head -1`
 CXXFLAGS="$CXXFLAGS -library=stlport4 -features=tmplife
 -features=tmplrefstatic"
 MULTITHREADING_FLAG="-mt"
 fi
 }}}

 The quoted line was not tested. Would you refute that?

 Commit `cc480a64481dbce14348038cfa254578fc51623b` is missing the `[3107]`
 prefix. See [[GitTips]] for a suggestion on how to automatically prefix
 commit log messages with the ticket number.

 Other than that, the current state of the branch seems fine to me. I have
 pushed one additional commit to the branch. Please check it. If that is
 OK, you can go ahead and merge it to `master`.

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


More information about the bind10-tickets mailing list