BIND 10 #3107: [kean] ./configure report versions of dependencies
BIND 10 Development
do-not-reply at isc.org
Mon Sep 16 13:44:11 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-20130917
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
-------------------------------------+-------------------------------------
Comment (by 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.
> You chose an approach against the coding guidelines and stated that you
It wasn't so much against the coding quidelines as an explicit action but
rather a reflexive one. I use "filename.h" instead of <filename.h>
reflexively, because that is what 10 or so years of autoconf headaches has
taught me is the path of least resistance. You pointed out it was against
the guidelines, so I fixed it.
--
Ticket URL: <https://bind10.isc.org/ticket/3107#comment:18>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development
More information about the bind10-tickets
mailing list