On Mon, 2012-07-02 at 13:33 -0400, Simo Sorce wrote:
On Mon, 2012-07-02 at 12:56 -0400, Stephen Gallagher wrote:
>
> Fixed (as mentioned above, I tied everything to
> ad_get_common_options())
>
>
> New patches since review:
>
> Patch 0008 is the same as sent earlier, Patch 0009 adds the AD
> provider
> into the contrib spec (forgotten earlier).
>
Manual code review looks fine now.
Although I haven't actually tested the code myself I would give an ACK
here.
Pushed to master.