On 2017/03/22 20:05:36 +0900, Akira Yokosawa wrote: > On 2017/03/22 07:54:24 +0900, Akira Yokosawa wrote: >> On 2017/03/21 15:41:20 -0700, Paul E. McKenney wrote: >>> On Wed, Mar 22, 2017 at 07:04:45AM +0900, Akira Yokosawa wrote: >>>> On 2017/03/22 3:35, Paul E. McKenney wrote: >>>>> On Mon, Mar 20, 2017 at 01:58:15PM +0900, Akira Yokosawa wrote: >>>>>> >From 25fb5aea63049454a5df2c923dac59285fe40720 Mon Sep 17 00:00:00 2001 >>>>>> From: Akira Yokosawa <akiyks@xxxxxxxxx> >>>>>> Date: Mon, 20 Mar 2017 13:10:38 +0900 >>>>>> Subject: [PATCH] Choose 'cmtt' font for code snippets in targets 'msnt' and 'mstx' >>>>>> >>>>>> By specifying "cmtt" (Computer Modern Teletype) font as \ttdefault >>>>>> in verbatim, verbbox, table, tabular, tabulary, and minipage >>>>>> environments, we can improve consistency of looks independent of >>>>>> monospace font choice for main text. cmtt font is more than suitable >>>>>> for listing of code snippet and program output. >>>>>> >>>>>> However, when mixed used with Times Roman (clone) font, cmtt's >>>>>> slightly lower x-height than Times Roman, and its similar thickness >>>>>> as Times Roman, it is not distinctive enough as can be seen in >>>>>> "mslm" target. >>>>>> >>>>>> In this respect, "txtt" and its enhanced version "newtxtt" fonts >>>>>> are more suitable as is described in [1]. >>>>>> >>>>>> This commit enables the font choice of cmtt for code snippets in >>>>>> targets "msnt" and "mstx". >>>>>> >>>>>> Help messages in Makefile are updated accordingly. >>>>>> >>>>>> Any feedback on the font choice is welcome. >>>>>> >>>>>> [1] Excerpt from newtxtt documentation: >>>>>> >>>>>> This package provides an interface to another alternative---the >>>>>> typewriter fonts provided with txfonts, with some enhancements. >>>>>> They have the same widths as cmtt, but are taller, heavier, >>>>>> more geometric and less shapely, with very low contrast, and >>>>>> are more suited to match Roman fonts of height and weight >>>>>> approximating that of Times. >>>>>> >>>>>> Signed-off-by: Akira Yokosawa <akiyks@xxxxxxxxx> >>>>> >>>>> Queued and pushed, thank you! >>>>> >>>>> I did remove some trailing whitespace that this patch added to >>>>> perfbook.tex. Please let me know if this whitespace was needed for >>>>> some reason. >>>> >>>> My bad. And thanks for the fix. >>>> >>>> Is there any config option to flag trailing whitespaces at "git add"? >>> >>> There is! >>> >>> You can create a pre-commit hook, which is a script that lives in your >>> .git/hooks/pre-commit file. I believe that the default hook checks for >>> trailing whitespace. You need to make the script executable. To make >>> the commit fail, make the script exit non-zero. If you want to force the >>> commit despite the script complaining, give "git commit" the "-n" flag. >>> >>> It has been years since I set mine up, so I am probably forgetting >>> something, but there is always: >>> >>> https://git-scm.com/book/en/v2/Customizing-Git-Git-Hooks >> >> Thanks! I'll try it. >> >>> >>>> And please try the modified targets and see if you really want to promote >>>> "msnt" as the default font choice. >>> >>> Now that you mention it, I get this: >>> >>> ! LaTeX Error: File `newtxtt.sty' not found. >>> >>> I downloaded this file from: >>> >>> ftp://ftp.dante.de/tex-archive/fonts/newtxtt/tex/newtxtt.sty >>> >>> and it was much happier. > > Well, font packages do not work properly by placing just .sty files. > You need to get the whole font package if you need to manually install > it on older TeX distribution. > > "newtxtt" font package can be downloaded from: > > http://mirrors.ctan.org/install/fonts/newtxtt.tds.zip > > The way to manually install a package can be found at: > > http://tex.stackexchange.com/questions/73016/ > > Method 3 in the answer is your option. (If you really want to do so.) Oops, for font packages, you also need to update font-map database. See: http://tex.stackexchange.com/questions/88423/ I'll prepare a patch to update FAQ-BUILD.txt regarding this issue. But "msnt" as default seems premature. "mslm" would be a moderate option for default. Thoughts? Thanks, Akira > > But you can just try "make mstx" instead. This gives mostly the same > result give or take the shape of "0" (hashed in mstx). > >>> But I bet that I should put it somewhere >>> other than in the main perfbook directory, given that I don't want >>> "git status" to gripe about it forever. Any advice on location? >> >> I thought you are already on Ubuntu 16.04. TeX Live 2015/Debian package >> should have this package. But let me check later. > > On Ubuntu 16.04, installing texlive-fonts-extra should be enough. > > Hope this helps. > > Thanks, Akira > >> >>> >>> One thing I am not all that happy with is that it is hard for me >>> to distinguish the inline fixed-width font from the normal font. >>> Or is that the fault of my setup? >> >> You might be accustomed to the thinner courier font. >> newtxtt font looks thiker than Times Roman, doesn't it? >> >> Thanks, Akira >>> >>> Thanx, Paul >>> >>> Thanx, Paul >>> [snip] -- To unsubscribe from this list: send the line "unsubscribe perfbook" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html