All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mauro Carvalho Chehab <mchehab@s-opensource.com>
To: Jonathan Corbet <corbet@lwn.net>
Cc: Markus Heiser <markus.heiser@darmarIT.de>,
	ksummit-discuss@lists.linuxfoundation.org,
	Linux Doc Mailing List <linux-doc@vger.kernel.org>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	Sakari Ailus <sakari.ailus@linux.intel.com>,
	Mauro Carvalho Chehab <mchehab@kernel.org>
Subject: Re: [Ksummit-discuss] [PATCH 0/9] Get rid of bitmap images
Date: Wed, 30 Nov 2016 07:29:15 -0200	[thread overview]
Message-ID: <20161130072915.09d1147b@vento.lan> (raw)
In-Reply-To: <20161128180946.516c64d6@lwn.net>

Em Mon, 28 Nov 2016 18:09:46 -0700
Jonathan Corbet <corbet@lwn.net> escreveu:

> On Sun, 20 Nov 2016 14:08:31 -0200
> Mauro Carvalho Chehab <mchehab@s-opensource.com> wrote:
> 
> > The goal of this patch series is to get rid of PNG images, using either graphviz
> > or SVG for images.  
> 
> So the pdfdocs build fails with an error when I apply this...
> 
> > xelatex -interaction=batchmode 'media.tex'
> > This is XeTeX, Version 3.14159265-2.6-0.99993 (TeX Live 2016/dev) (preloaded format=xelatex)
> >  restricted \write18 enabled.
> > entering extended mode
> > Makefile:58: recipe for target 'media.pdf' failed
> > make[2]: *** [media.pdf] Error 1
> > Documentation/Makefile.sphinx:83: recipe for target 'pdfdocs' failed
> > make[1]: *** [pdfdocs] Error 2
> > Makefile:1442: recipe for target 'pdfdocs' failed
> > make: *** [pdfdocs] Error 2  
> 
> There's nothing more useful than that in the log as far as I can tell.

> I take it you don't see this?

No, I didn't see.

Calling it in interactive mode with:

$ make -C Documentation/output/latex/ media.pdf PDFLATEX=xelatex

allowed to identify the issue. The problem is that I forgot to
change the file extension from .svg to .* on one of the figure includes.

The enclosed patch fixes it.

I'll fold it with the patch that converted and send a new version
of this patch series.

Thanks for noticing it!
Mauro


diff --git a/Documentation/media/uapi/v4l/field-order.rst b/Documentation/media/uapi/v4l/field-order.rst
index 27fccd3cb9b4..e05fb1041363 100644
--- a/Documentation/media/uapi/v4l/field-order.rst
+++ b/Documentation/media/uapi/v4l/field-order.rst
@@ -151,7 +151,7 @@ Field Order, Top Field First Transmitted
 Field Order, Bottom Field First Transmitted
 ===========================================
 
-.. figure::  fieldseq_bt.svg
+.. figure::  fieldseq_bt.*
     :alt:    fieldseq_bt.pdf / fieldseq_bt.svg
     :align:  center
 

WARNING: multiple messages have this Message-ID (diff)
From: Mauro Carvalho Chehab <mchehab@s-opensource.com>
To: Jonathan Corbet <corbet@lwn.net>
Cc: Linux Doc Mailing List <linux-doc@vger.kernel.org>,
	Mauro Carvalho Chehab <mchehab@infradead.org>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	ksummit-discuss@lists.linuxfoundation.org,
	Sakari Ailus <sakari.ailus@linux.intel.com>,
	Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
	Markus Heiser <markus.heiser@darmarIT.de>,
	Mauro Carvalho Chehab <mchehab@kernel.org>
Subject: Re: [PATCH 0/9] Get rid of bitmap images
Date: Wed, 30 Nov 2016 07:29:15 -0200	[thread overview]
Message-ID: <20161130072915.09d1147b@vento.lan> (raw)
In-Reply-To: <20161128180946.516c64d6@lwn.net>

Em Mon, 28 Nov 2016 18:09:46 -0700
Jonathan Corbet <corbet@lwn.net> escreveu:

> On Sun, 20 Nov 2016 14:08:31 -0200
> Mauro Carvalho Chehab <mchehab@s-opensource.com> wrote:
> 
> > The goal of this patch series is to get rid of PNG images, using either graphviz
> > or SVG for images.  
> 
> So the pdfdocs build fails with an error when I apply this...
> 
> > xelatex -interaction=batchmode 'media.tex'
> > This is XeTeX, Version 3.14159265-2.6-0.99993 (TeX Live 2016/dev) (preloaded format=xelatex)
> >  restricted \write18 enabled.
> > entering extended mode
> > Makefile:58: recipe for target 'media.pdf' failed
> > make[2]: *** [media.pdf] Error 1
> > Documentation/Makefile.sphinx:83: recipe for target 'pdfdocs' failed
> > make[1]: *** [pdfdocs] Error 2
> > Makefile:1442: recipe for target 'pdfdocs' failed
> > make: *** [pdfdocs] Error 2  
> 
> There's nothing more useful than that in the log as far as I can tell.

> I take it you don't see this?

No, I didn't see.

Calling it in interactive mode with:

$ make -C Documentation/output/latex/ media.pdf PDFLATEX=xelatex

allowed to identify the issue. The problem is that I forgot to
change the file extension from .svg to .* on one of the figure includes.

The enclosed patch fixes it.

I'll fold it with the patch that converted and send a new version
of this patch series.

Thanks for noticing it!
Mauro


diff --git a/Documentation/media/uapi/v4l/field-order.rst b/Documentation/media/uapi/v4l/field-order.rst
index 27fccd3cb9b4..e05fb1041363 100644
--- a/Documentation/media/uapi/v4l/field-order.rst
+++ b/Documentation/media/uapi/v4l/field-order.rst
@@ -151,7 +151,7 @@ Field Order, Top Field First Transmitted
 Field Order, Bottom Field First Transmitted
 ===========================================
 
-.. figure::  fieldseq_bt.svg
+.. figure::  fieldseq_bt.*
     :alt:    fieldseq_bt.pdf / fieldseq_bt.svg
     :align:  center
 

  reply	other threads:[~2016-11-30  9:29 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-11-20 16:08 [Ksummit-discuss] [PATCH 0/9] Get rid of bitmap images Mauro Carvalho Chehab
2016-11-20 16:08 ` Mauro Carvalho Chehab
2016-11-20 16:08 ` [Ksummit-discuss] [PATCH 1/9] [media] convert more media images to SVG Mauro Carvalho Chehab
2016-11-20 16:08   ` Mauro Carvalho Chehab
2016-11-20 16:08 ` [Ksummit-discuss] [PATCH 2/9] [media] svg files: cleanup them Mauro Carvalho Chehab
2016-11-20 16:08 ` [Ksummit-discuss] [PATCH 3/9] [media] docs-rst: nv12mt zigzag images: replace by SVG images Mauro Carvalho Chehab
2016-11-20 16:08   ` Mauro Carvalho Chehab
2016-11-20 16:08 ` [Ksummit-discuss] [PATCH 4/9] [media] docs-rst: convert pipeline to SVG format Mauro Carvalho Chehab
2016-11-20 16:08   ` Mauro Carvalho Chehab
2016-11-20 16:08 ` [Ksummit-discuss] [PATCH 5/9] [media] docs-rst: replace the selection.png by a SVG image Mauro Carvalho Chehab
2016-11-20 16:08   ` Mauro Carvalho Chehab
2016-11-20 16:08 ` [Ksummit-discuss] [PATCH 6/9] [media] docs-rst: replace bayer.png " Mauro Carvalho Chehab
2016-11-20 16:08   ` Mauro Carvalho Chehab
2016-11-20 16:08 ` [Ksummit-discuss] [PATCH 7/9] docs-rst: media: build SVG from graphviz files Mauro Carvalho Chehab
2016-11-20 16:08   ` Mauro Carvalho Chehab
2016-11-20 16:08 ` [Ksummit-discuss] [PATCH 8/9] doc-rst: media/Makefile: reorganize the rules Mauro Carvalho Chehab
2016-11-20 16:08   ` Mauro Carvalho Chehab
2016-11-20 16:08 ` [Ksummit-discuss] [PATCH 9/9] docs-rst: fix media cleandocs target Mauro Carvalho Chehab
2016-11-20 16:08   ` Mauro Carvalho Chehab
2016-11-21 17:44 ` [Ksummit-discuss] [PATCH 0/9] Get rid of bitmap images Jonathan Corbet
2016-11-21 17:44   ` Jonathan Corbet
2016-11-21 19:15   ` [Ksummit-discuss] " Mauro Carvalho Chehab
2016-11-21 19:15     ` Mauro Carvalho Chehab
2016-11-22 13:49     ` [Ksummit-discuss] " Jani Nikula
2016-11-22 15:15       ` Mauro Carvalho Chehab
2016-11-22 15:15         ` Mauro Carvalho Chehab
2016-11-22 15:40         ` Jani Nikula
2016-11-22 15:40           ` Jani Nikula
2016-11-22 16:38           ` Mauro Carvalho Chehab
2016-11-22 16:38             ` Mauro Carvalho Chehab
2016-11-29  1:09 ` Jonathan Corbet
2016-11-29  1:09   ` Jonathan Corbet
2016-11-30  9:29   ` Mauro Carvalho Chehab [this message]
2016-11-30  9:29     ` Mauro Carvalho Chehab

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20161130072915.09d1147b@vento.lan \
    --to=mchehab@s-opensource.com \
    --cc=corbet@lwn.net \
    --cc=ksummit-discuss@lists.linuxfoundation.org \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=markus.heiser@darmarIT.de \
    --cc=mchehab@kernel.org \
    --cc=sakari.ailus@linux.intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.