ldap-auth article patch

Toby Burress kurin at delete.org
Wed Dec 16 05:47:09 UTC 2009


Resending this, as I guess it didn't make the list.

On Mon, Nov 09, 2009 at 08:26:10PM -0500, Toby Burress wrote:
> On Mon, Nov 09, 2009 at 07:39:52PM -0500, Tom Rhodes wrote:
> > Patch modified, please verify the original meaning by giving
> > my version a once over.
> 
> It looks like this was modified from my original ticket instead of
> the second patch I sent.  Either one is fine (they both catch the
> dimins.com -> example.org stuff, and yours additionally expands
> contractions).  However, in your patch there are two parenthetical
> statements which aren't closed.
> 
> 107c107
> < +# binds, there's probably some standard account you use for this.
> ---
> > +# binds, there's probably some standard account you use for this).
> 112c112
> < +# UID elsewhere in the tree; we don't want to return those.
> ---
> > +# UID elsewhere in the tree; we don't want to return those).
> 158a159
> 
> Other than that it looks fine.



More information about the freebsd-doc mailing list