[SSSD] Fixed typo

Jakub Hrozek jhrozek at redhat.com
Wed Mar 6 11:41:39 UTC 2013


On Mon, Mar 04, 2013 at 05:07:37PM +0100, Ondrej Kos wrote:
> On 03/04/2013 04:55 PM, Thorsten Scherf wrote:
> >On [Mon, 04.03.2013 11:28], Ondrej Kos wrote:
> >>On 02/28/2013 02:01 PM, Thorsten Scherf wrote:
> >>>On [Thu, 28.02.2013 11:25], Sumit Bose wrote:
> >>>>On Thu, Feb 28, 2013 at 10:46:10AM +0100, Jakub Hrozek wrote:
> >>>>>On Thu, Feb 28, 2013 at 09:36:13AM +0100, Sumit Bose wrote:
> >>>>>>On Wed, Feb 27, 2013 at 09:51:16PM +0100, Thorsten Scherf wrote:
> >>>>>>>
> >>>>>>
> >>>>>>Hi Thorsten,
> >>>>>>
> >>>>>>thank you for the patch and bringing the attention to the *.doxy.in
> >>>>>>files. Those files are autogenerated by doxygen where this typo and
> >>>>>>others are already fixed upstream. Instead of fixing the typos
> >>>>>>individually I wonder if it would be better to run doxygen -u on all
> >>>>>>*.doxy.in files? But I'm not sure if files created by a newer version
> >>>>>>can be handled by older versions as well. E.g. make docs is used in
> >>>>>the
> >>>>>>RHEL-6 spec file which still uses doxygen-1.6.1. Does anyone know
> >>>>>if it
> >>>>>>will just ignore new options or will it fail?
> >>>>>>
> >>>>>>bye,
> >>>>>>Sumit
> >>>>>
> >>>>>I remember from fixing another related issue that old doxygen releases
> >>>>>would complain about an unknown option but would not fail.
> >>>>>
> >>>>>So I think we can refresh the files, yes. Can you open a ticket?
> >>>>
> >>>>I've created https://fedorahosted.org/sssd/ticket/1819 .
> >>>>
> >>>>Thorsten, are you interested in providing a new patch here?
> >>>
> >>>Patch attached.
> >>>
> >>>The configs have been update based on doxygen-1.8.3.
> >>>
> >>>Cheers,
> >>>Thorsten
> >>>
> >>Hi Thorsten,
> >>
> >>I tested the documentation build with few versions of doxygen, and
> >>everything works fine. Could you please just add the trac link to the
> >>patch commit message?
> >
> >Correct patch attached this time.
> >
> >Cheers,
> >Thorsten
> 
> Thanks for the patch, Thorsten
> 
> Ack

Works for me on F18, el6 and el5, too.

Ack++ and pushed to master.



More information about the sssd-devel mailing list