ldap-auth article patch
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.
> < +# binds, there's probably some standard account you use for this.
> > +# binds, there's probably some standard account you use for this).
> < +# UID elsewhere in the tree; we don't want to return those.
> > +# UID elsewhere in the tree; we don't want to return those).
> Other than that it looks fine.
More information about the freebsd-doc