Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(91)

Issue 577900043: Add `-dfont-ps-resdir` option to embed fonts to PDFs later

Can't Edit
Can't Publish+Mail
Start Review
Created:
3 years, 12 months ago by trueroad
Modified:
3 years, 11 months ago
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

Add `-dfont-ps-resdir` option to embed fonts to PDFs later A similar purpose option, `-dfont-export-dir` exports fonts as individual PostScript files for font embedding later. However it cannot embed CID fonts to PDFs with Ghostscript 9.26 and later. This `-dfont-ps-resdir` option creates a subset of PostScript resource directory in the directory specified by the option. The directory contains symbolic links to font files that Ghostscript can use to embed the fonts to PDFs later. This method can embed CID fonts, even with Ghostscript 9.26 and later. This `-dfont-ps-resdir` option skips TrueType fonts because garbled characters occur same as `-dfont-export-dir` option. Unlike `-dfont-export-dir` option, it skips OTC (OpenType Collection/CFF) font files because Ghostscript cannot handle them directly. Simple usage: ``` $ lilypond -dfont-ps-resdir=resdir -dgs-never-embed-fonts foo.ly $ gs -q -dBATCH -dNOPAUSE -sDEVICE=pdfwrite \ -I resdir -I resdir/Font \ -sOutputFile=foo.embedded.pdf foo.pdf ``` Note: It is better not to specify the directory that contains the name `Resource` because it has a special meaning when specified with the -I option for Ghostscript.

Patch Set 1 #

Total comments: 16

Patch Set 2 : Fix 80 characters and document #

Unified diffs Side-by-side diffs Delta from patch set Stats (+147 lines, -30 lines) Patch
M Documentation/usage/running.itely View 1 1 chunk +56 lines, -1 line 0 comments Download
M scm/framework-ps.scm View 1 6 chunks +88 lines, -29 lines 0 comments Download
M scm/lily.scm View 1 chunk +3 lines, -0 lines 0 comments Download

Messages

Total messages: 5
lemzwerg
Very nice, thanks! There are some grammar glitches in the documentation – I hope that ...
3 years, 12 months ago (2020-04-28 15:51:33 UTC) #1
Trevor Daniels
Please read my suggested changes carefully and amend, as I've no knowledge of the subject ...
3 years, 12 months ago (2020-04-28 17:37:01 UTC) #2
trueroad
Fix 80 characters and document
3 years, 12 months ago (2020-04-29 04:01:22 UTC) #3
trueroad
Thank you for your reviewing. https://codereview.appspot.com/577900043/diff/577910043/Documentation/usage/running.itely File Documentation/usage/running.itely (right): https://codereview.appspot.com/577900043/diff/577910043/Documentation/usage/running.itely#newcode652 Documentation/usage/running.itely:652: This is useful when ...
3 years, 12 months ago (2020-04-29 04:12:01 UTC) #4
trueroad
3 years, 11 months ago (2020-05-02 09:55:23 UTC) #5
I've pushed to staging.

commit 2ff485fd858d55263ca46db431793943f8c993f1
Date:   Mon Apr 27 23:52:53 2020 +0900

Issue5947/2: Doc: Add `-dfont-ps-resdir` option to running.itely

commit 3ada4bbab00f1ad91bbf78ab4c63324b4e539d10
Date:   Mon Apr 27 23:52:33 2020 +0900

Issue5947/1: Add `-dfont-ps-resdir` option to embed fonts to PDFs later
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b