Re: Review Request 39230: Added check for SASL deprecation into configuration phase.

2015-10-13 Thread Till Toenshoff
> On Oct. 12, 2015, 7:37 p.m., James Peach wrote: > > It's marginal, but I think ```AC_LANG_PROGRAM``` is a little clearer that > > ```AC_LANG_SOURCE```. > > > > If you are going to rely on the diagnostic pragmas, an alternative is to > > simply use those to disable the warning in the SASL

Re: Review Request 39230: Added check for SASL deprecation into configuration phase.

2015-10-12 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39230/#review102251 --- Patch looks great! Reviews applied: [39230] All tests passed. -

Re: Review Request 39230: Added check for SASL deprecation into configuration phase.

2015-10-12 Thread James Peach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39230/#review102306 --- Ship it! It's marginal, but I think ```AC_LANG_PROGRAM``` is a