[LTP] [PATCH v3 3/3] Add Doxygen configuration

Cyril Hrubis chrubis@suse.cz
Thu Jul 19 15:44:03 CEST 2018


Hi!
> >> > Not that this is a bad thing quite contrary actually, but it's quite
> >> > confusing that this is a 3/3 of this patchset.
> >> >
> >> > Haven you send this by a mistake?
> >> 
> >> Nope, but maybe I should have included a covering letter. I mentioned
> >> that it is not part of the patch set in the notes on the first
> >> commit. I just included it so that you can easily verify the
> >> documentation comments if you want to do that.
> >
> > Ah, I missed that note in the first commit.
> >
> > Hmm, if we decide to go that way we should probably get this in with all
> > the bells and whistles in a separate patchset. I.e. add doxygen target
> > to the doc/Makefile, convert the headers we have and introduce the
> > config.
> 
> Are you saying that you want me to submit the patch with the
> documentation in the kernel format, then submit another patch set
> updating all the documentation later?

Actually I do not care that much about the format at this point, as far
as we do not parse it automatically anything human-readable will do...

-- 
Cyril Hrubis
chrubis@suse.cz


More information about the ltp mailing list