Message ID | 1429881789-16060-1-git-send-email-nhorman@tuxdriver.com (mailing list archive) |
---|---|
State | Rejected, archived |
Headers | show |
> -----Original Message----- > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Neil Horman > Sent: Friday, April 24, 2015 2:23 PM > To: dev@dpdk.org > Subject: [dpdk-dev] [PATCH] docs: convert from using inkscape to > ImageMagick tools > > Requiring inkscape to convert svg files to png/pdf files, while functional > is difficult for ephemeral build environments (i.e. build systems in which > a new build root is setup for each build), as inkscape and its > dependencies typically totals over 128Mb of code. Instead, lets use > ImageMagick tools. It gives you the same output in a much more compact > tool suite (512k). Hi Neil, The patch works but the pdf output is different. See the following images for before and after the patch: http://imgur.com/GgdXSnF http://imgur.com/e9AvTza It looks like a dpi issue but I'm not sure what exactly. Omitting the -units/-density options renders the images at the right size but the resolution looks slightly worse. John --
diff --git a/mk/rte.sdkdoc.mk b/mk/rte.sdkdoc.mk index 9952f25..b03bca5 100644 --- a/mk/rte.sdkdoc.mk +++ b/mk/rte.sdkdoc.mk @@ -47,7 +47,7 @@ endif ifeq '$V' '0' RTE_SPHINX_VERBOSE := -q RTE_PDFLATEX_VERBOSE := --interaction=batchmode -RTE_INKSCAPE_VERBOSE := >/dev/null 2>&1 +RTE_CONVERT_VERBOSE := >/dev/null 2>&1 endif RTE_PDF_DPI ?= 300 @@ -116,4 +116,5 @@ guides-%: $(foreach guide, $(RTE_GUIDES), $(foreach img, $(wildcard $(guide)img/*.svg), \ $(eval guides-pdf-$(notdir $(guide:/=)): $(img:svg=pdf)))) %.pdf: %.svg - $(Q)inkscape -d $(RTE_PDF_DPI) -D -f $< -A $@ $(RTE_INKSCAPE_VERBOSE) + $(Q)convert -units PixelsPerInch $< -density 300 $@ $(RTE_CONVERT_VERBOSE) +# $(Q)inkscape -d $(RTE_PDF_DPI) -D -f $< -A $@ $(RTE_INKSCAPE_VERBOSE)
Requiring inkscape to convert svg files to png/pdf files, while functional is difficult for ephemeral build environments (i.e. build systems in which a new build root is setup for each build), as inkscape and its dependencies typically totals over 128Mb of code. Instead, lets use ImageMagick tools. It gives you the same output in a much more compact tool suite (512k). Signed-off-by: Neil Horman <nhorman@tuxdriver.com> --- mk/rte.sdkdoc.mk | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-)