On Fri, Jan 23, 2015 at 10:14:56PM +0100, Jakub Hrozek wrote:
On Fri, Jan 23, 2015 at 12:23:43PM +0100, Pavel Reichl wrote:
>
> On 01/22/2015 05:58 PM, Stephen Gallagher wrote:
> >
> >
> >On Mon, 2015-01-19 at 09:58 +0100, Jakub Hrozek wrote:
> >>On Mon, Jan 19, 2015 at 09:39:41AM +0100, Pavel Reichl wrote:
> >>>>>man page:
> >>>>>Specify AD site client should try to connect to.
> >>>>>Specify AD site to which client should try to connect.
> >>>>>
> >>>>>Which one sounds better?
> >>>>I don't care much, maybe the second one. Should I prepare a new
patch or
> >>>>will you Jakub amend the patch while pushing it? Thanks.
> >>Oops, sorry, this one slipped.
> >>
> >>>Attached patch updates man page with Pavel's second suggestion.
> >>>Can I get the final ack? I would like to prepare scratch build for
customer
> >>>ASAP.
> >>I like the second form as well, I think it should say "the
client".
> >>Could you ping some native speaker to help us out?
> >>
> >>btw go ahead and built the test package, I'm sure the customer is more
> >>interested in functionality rather than a typo in a man page..
> >
> >Slight change to the manpage:
> >
> >"Specify AD site to which client should try to connect. If this option
> >is not provided, the AD site will be auto-discovered."
> >
> Thanks!
> Updated patches attached.
Since the man pages were amended per Stephen's suggestion and the code
was acked by Pavel Brezina earlier, I'm going to push the patches.
* master:
e438fbf102c3d787902504bdae177e84230cbbc9
b22e0da9e644f5eb84ee0c8986979fec3fe7eb56
* sssd-1-12:
6992f203c2b37d130287eae11f3929d0000e6d44
e2f4a87ef4a657d27c3ec544fd75a21eefcf3ce7