[Freeipa-devel] [PATCH 0072] Provide ipa-client-advise tool

[Simo Sorce]

----- Original Message -----
> On 06/19/2013 03:03 PM, Alexander Bokovoy wrote:
> > On Wed, 19 Jun 2013, Jan Cholasta wrote:
> >> On 19.6.2013 14:47, Dmitri Pal wrote:
> >>> On 06/19/2013 08:02 AM, Tomas Babej wrote:
> >>>> Do you have something particular in mind?
> >>>>
> >>>> Tomas
> >>>>
> >>>> _______________________________________________
> >>>> Freeipa-devel mailing list
> >>>> Freeipa-devel at redhat.com
> >>>> https://www.redhat.com/mailman/listinfo/freeipa-devel
> >>>
> >>>
> >>> ipa-config-advisor ?
> >>>
> >>
> >> IMO we should stick to a verb in the name, so ipa-config-advise.
> > Then it is better to be simpler, ipa-advise is a nice name.
> 
> Isn't that too simple? Are you trying to create an all knowing Siri-like
> advisor for IPA? If I am a user, I would really not know what "ipa-advise"
> means and what advise could it give to me.
> 
> # ipa-advise "what pair of socks should I take for today?"
> 
> ipa-config-advise was better IMHO.

then as soon as you need to 'advise' on something that is not config related it becomes akward, also ipa-config-advise is much longer to type and 'config' doesn't really add much.

As for the user 'man ipa-advise' will neatly explain what it will advise about, I think that is sufficient.
Nobody will expect 'ipa'-advise to provide info about non-ipa related stuff anyway.

As for the actual command line options I do wonder as well why you need a --setup or --about option at all.

ipa-advise 'topic' is sufficient imo.

options that may make sense are things like --verbose so that you can have a small excerpt with the short form and a much longer text with --verbose if necessary. Although maybe we should just reference man pages for longer text and not try to create a new manpage substitute, we certainly should always provide all the content in man pages first.

Simo.

-- 
Simo Sorce * Red Hat, Inc. * New York