public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
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@vger.kernel.org
Subject: Re: [PATCH v2 0/6] Better document Sphinx install instructions
Date: Mon, 17 Jul 2017 18:55:20 -0300	[thread overview]
Message-ID: <20170717185520.39e640f7@vento.lan> (raw)
In-Reply-To: <20170717152633.5cdeff38@lwn.net>

Em Mon, 17 Jul 2017 15:26:33 -0600
Jonathan Corbet <corbet@lwn.net> escreveu:

> On Mon, 17 Jul 2017 18:23:05 -0300
> Mauro Carvalho Chehab <mchehab@s-opensource.com> wrote:
> 
> > Btw, I'm submitting in a few a second version of the patches with
> > add a script to check Sphinx dependencies.  
> 
> I figured I wasn't going to look at that until you get up to v42 or so -
> maybe another day or two...? :)

Just sent ;)

There are two patches that I didn't send via e-mail:

	https://git.linuxtv.org/mchehab/experimental.git/commit/?h=sphinx_install_v2&id=4bd9e1ea9ddabce144ee4c84549449563f0b8416
	https://git.linuxtv.org/mchehab/experimental.git/commit/?h=sphinx_install_v2&id=6b9596e432709f7a7def85873c8ccbb837fdf4a2

The problem is that SVG files often have lines bigger than 998 chars,
with is the limit that e-mails usually accept (and VGER enforces).

We need some sort of script to break long lines. I remember that I
wrote one once, but need to seek for it, as it is not at the
usual places.

Those two patches are required for PDF build to work without errors
on Ubuntu 17.04. The problem there is that ImageMagick is a way
more pedantic there than on other distros, and don't support
several atributes that inkscape add. I had to use some tools to
filter out those arguments (svgo, scour) and even manually replace
font-family there, for it to properly recognize the SVG contents.

What's funny is that ImageMagick version on Ubuntu is newer
than the one in Fedora 26:

	ubuntu17.04$ dpkg -l imagemagick
	Desired=Unknown/Install/Remove/Purge/Hold
	| Status=Not/Inst/Conf-files/Unpacked/halF-conf/Half-inst/trig-aWait/Trig-pend
	|/ Err?=(none)/Reinst-required (Status,Err: uppercase=bad)
	||/ Name                    Version          Architecture     Description
	+++-=======================-================-================-====================================================
	ii  imagemagick             8:6.9.7.4+dfsg-3 amd64            image manipulation programs -- binaries

	fedora25$ rpm -q ImageMagick
	ImageMagick-6.9.3.0-6.fc25.x86_64

	fedora26$ rpm -q ImageMagick
	ImageMagick-6.9.3.0-6.fc26.x86_64

Perhaps some regression?

	

Thanks,
Mauro

      reply	other threads:[~2017-07-17 21:55 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-07-14 16:41 [PATCH v2 0/6] Better document Sphinx install instructions Mauro Carvalho Chehab
2017-07-14 16:41 ` [PATCH v2 1/6] changes.rst: Update Sphinx minimal requirements Mauro Carvalho Chehab
2017-07-14 16:41 ` [PATCH v2 2/6] docs-rst: move Sphinx install instructions to sphinx.rst Mauro Carvalho Chehab
2017-07-14 16:41 ` [PATCH v2 3/6] sphinx.rst: explain the usage of virtual environment Mauro Carvalho Chehab
2017-07-14 16:41 ` [PATCH v2 4/6] sphinx.rst: fix unknown reference Mauro Carvalho Chehab
2017-07-14 16:41 ` [PATCH v2 5/6] sphinx.rst: describe the install requirements for kfigure Mauro Carvalho Chehab
2017-07-14 16:41 ` [PATCH v2 6/6] sphinx.rst: better organize the documentation about PDF build Mauro Carvalho Chehab
2017-07-17 19:40 ` [PATCH v2 0/6] Better document Sphinx install instructions Jonathan Corbet
2017-07-17 21:23   ` Mauro Carvalho Chehab
2017-07-17 21:26     ` Jonathan Corbet
2017-07-17 21:55       ` Mauro Carvalho Chehab [this message]

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=20170717185520.39e640f7@vento.lan \
    --to=mchehab@s-opensource.com \
    --cc=corbet@lwn.net \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mchehab@infradead.org \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox