On 09.08.2016 15:13, Florence Blanc-Renaud wrote:
On 08/02/2016 09:11 AM, Stanislav Laznicka wrote:
On 07/19/2016 10:25 AM, Florence Blanc-Renaud wrote:
On 07/15/2016 02:09 PM, Stanislav Laznicka wrote:
https://fedorahosted.org/freeipa/ticket/6013



Hi Stanislav,

thanks for your patch. As CERTFILE is added in the arguments for
install, I would suggest to mention it in the command description. For
instance:

install
  - Install a CA certificate
  This command can be used to install the certificate contained in
CERTFILE as a new CA certificate to IPA.

Flo.

Hi,

Thanks for the notice, I agree that it'd be better to be more verbose
about the CERTFILE argument. Please see the modified patch.


Hi,

thanks for the updated patch. Looks good to me,
ACK

Flo.

Pushed to master: bf6adfe69d2dc1e6cc76d17023f4049e49cfd8ae

--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to