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

[Tomas Babej]

On 06/19/2013 06:13 PM, Dmitri Pal wrote:
> On 06/19/2013 10:46 AM, Simo Sorce wrote:
>> ----- 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.
>>
> So if I want an advise about Solaris 11 client configuration would it
> look like this?
>
> ipa-advise config --client --distro=solaris  --version=11
>
> or
>
> ipa-advise client-config-solrais-11
>
>

The latter.

Tomas