Re: [PATCH v2 1/2] docs-rst: move the rules for svg and graphviz to Makefile.sphinx

[Mauro Carvalho Chehab <mchehab@s-opensource.com>]

Em Wed, 30 Nov 2016 18:03:38 -0700
Jonathan Corbet <corbet@lwn.net> escreveu:

> On Mon, 21 Nov 2016 17:27:03 -0200
> Mauro Carvalho Chehab <mchehab@s-opensource.com> wrote:
> 
> > Not only media documents may have images. So, move the
> > rules for building images from graphviz and from SVG to
> > the Documentation makefile.  
> 
> So I tried applying this one, but...
> 
> > LaTeX Warning: File `{media/typical_media_device}.*' not found on input line 22
> > 4.
> > 
> > ! Unable to load picture or PDF file 'media/typical_media_device.*'.
> > <to be read again> 
> >                    }
> > l.224 ...egraphics{{media/typical_media_device}.*}  
> 
> ...so something's not quite right there still.  Unapplied, for now.
>
> I'm thinking I'd like to button up docs-next for 4.10 pretty soon now;
> don't want end up with more patches than media, after all! :)  So maybe
> this one can be fixed at leisure and go in for 4.11.  I don't think we
> really need it urgently.

Weird, here it worked fine (I almost submitted it by mistake on
the patch series you merged yesterday). Maybe the DOTS or IMAGES
line may be missing some *.svg or *.dot file.

Yet, I agree that the best is not to merge it, specially since Markus 
has already a RFC for a Sphinx extension that would avoid the need of 
doing the conversion via Makefile. If it solves properly, we could merge
his patchset instead.

If something goes wrong with the extension, I'm keeping those two
patches, plus the one suggested by Daniel Vetter that uses find to
seek for the *.svg and *.dot file at the makefile on this branch:

	https://git.linuxtv.org/mchehab/experimental.git/log/?h=svg-images-v2

Thanks,
Mauro