BIND 10 #365: clang++ support
BIND 10 Development
do-not-reply at isc.org
Wed Oct 27 23:06:17 UTC 2010
#365: clang++ support
-------------------------------+--------------------------------------------
Reporter: jinmei | Owner: jinmei
Type: enhancement | Status: reviewing
Priority: major | Milestone: y2 12 month milestone
Component: build system | Resolution:
Keywords: | Sensitive: 0
Estimatedhours: 0.0 | Hours: 0
Billable: 1 | Totalhours: 0
Internal: 0 |
-------------------------------+--------------------------------------------
Comment(by jinmei):
Replying to [comment:8 jelte]:
> Should we also print the compiler we chose in the configure overview
output?
>
Good suggestion. Done in r3381.
> About the -Wno-error, in trac327, all asio code should have been
localized to one location (we ran into unused warnings there as well), I'm
assuming this is a similar problem, so if that is merged, we *should* be
able to revert the changes for asio-specific problems here.
>
Right.
> There's a few tests within specific rdata tests where EXPECT_TRUE(a > b)
is used now, there is also an EXPECT_GT that should test the > operator.
>
That makes sense, but it's beyond the scope of this ticket. I'll create a
separate ticket for this.
> I can confirm that it still works with gcc, but I have no working clang
version at this moment (the older version i have crashes, and the latest
svn version does not compile here), so I cannot confirm whether it works
for me too. However, I see no problems with the changes, and except for
the trivial comments above, I think it can be merged.
>
Okay, thanks for the review. I'll hold for a while due to the additional
change (r3381) but unless I hear something I'll soon merge it to trunk.
--
Ticket URL: <http://bind10.isc.org/ticket/365#comment:9>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development
More information about the bind10-tickets
mailing list