Re: [PATCH 3/3] [media] DocBook: Add rules to auto-generate some media docbook

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 01 Jun 2011 07:52:33 -0300 Mauro Carvalho Chehab wrote:

> Hi Randy,
> 
> Em 26-05-2011 00:10, Randy Dunlap escreveu:
> > On Wed, 25 May 2011 12:26:42 -0300 Mauro Carvalho Chehab wrote:
> > 
> >> Auto-generate the videodev2.h.xml,frontend.h.xml and the indexes.
> >>
> >> Some logic at the Makefile helps us to identify when a symbol is missing,
> >> like for example:
> >>
> >> Error: no ID for constraint linkend: V4L2-PIX-FMT-JPGL.
> > 
> > 
> > a.  Still get that message..  is that OK?
> > 
> > b.  In the generated index.html file, "media" is listed first, but it should be
> > listed in alphabetical order, not first.
> > 
> > c.  The generated files are (hidden) in .tmpmedia/
> > 
> > d.  The link from the top-level index.html file to "media" is to
> > media/index.html, but the file is actually in .tmpmedia/media/index.html
> > 
> > e.  patches 1/3 and 2/3 are OK.
> > 
> > 
> > Please build docs with and without using "O=builddir" and test that.
> > 
> > I'm looking over the generated output now and will let you know if I see
> > any other problems.
> 
> Fixed the pointed issues. I also moved the media-specific stuff into another
> Makefile. I opted to include it into the DocBook Makefile, as otherwise, I would
> need to duplicate the xml conversion stuff into the media/Makefile.

Sounds good.  Thanks.

> Patch is enclosed.

It will probably be a few days before I test this again,
so just merge it when you are ready...

---
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux