----- Original Message -----
> From: "Hubert Kario via RT" <[email protected]>
> Cc: [email protected]
> Sent: Tuesday, 17 June, 2014 7:24:45 PM
> Subject: Re: [openssl.org #3385] Patch: document -trusted_first option in man 
> pages and help.
> 
> ----- Original Message -----
> > From: "Stephen Henson via RT" <[email protected]>
> > To: [email protected]
> > Cc: [email protected]
> > Sent: Tuesday, 17 June, 2014 2:31:07 PM
> > Subject: [openssl.org #3385] Patch: document -trusted_first option in man
> > pages and help.
> > 
> > On Tue Jun 17 13:38:49 2014, [email protected] wrote:
> > >
> > > Thank you, somehow missed that.
> > >
> > > For now I just removed changes to s_time and ts from patch set.
> > > Fixed pull request: https://github.com/openssl/openssl/pull/138
> > >
> > 
> > The -trusted_first option is handled through a common verify options
> > function
> > call args_verify. Could we avoid duplication in the usage messages by
> > having
> > the relevant usage messages for all the args_verify options printed in one
> > place?
> > 
> > Similarly we could document the relevant options in one manual page and
> > reference them in other pages. A bit like we do with the pass phrase
> > arguments.
> 
> Yes, I've noticed that they are handled by a single function. The man pages
> were not consistent with documenting them, so I opted for the
> easiest-for-user
> version.
> 
> If the current guideline is to explain them in only single place, I can
> document them the same way -purpose, -ignore_critical, -issuer_checks, etc.
> are documented in s_client. I'll also try to clean up other options
> parsed by args_verify() so that the man pages are consistent.

Pull request that implements those changes is available here:
https://github.com/openssl/openssl/pull/141

-- 
Regards,
Hubert Kario
Quality Engineer, QE BaseOS Security team
Email: [email protected]
Web: www.cz.redhat.com
Red Hat Czech s.r.o., Purkyňova 99/71, 612 45, Brno, Czech Republic
______________________________________________________________________
OpenSSL Project                                 http://www.openssl.org
Development Mailing List                       [email protected]
Automated List Manager                           [email protected]

Reply via email to