jlreq派生クラスとスタイル

[kmuto]

いちおうsyntax-bookがコンパイルできるもの。

• 完成度はまだ低いです。
• reviewnoteおよびコラムの環境でオプション[]を使ってキャプションを指定する都合上、jsbookスタイルのほうも変更しています。
• 本当はコードの前後アキ固定のためにreviewlistblockでキャプションとreviewlistを囲みたい。
• LoadClassWithOptionsにしてもクラスオプションを引き継いでくれるというわけではないみたい。jlreq採用で肝心のfont指定などをしようとすると怒られる…。

とりあえず実用からはまだほど遠いので、masterからjlreqは削除したほうがよさそうに思っています。ただ消した場合に派生ブランチのこっちのファイルはどう影響受けるんでしょうね。

[takahashim]

@kmuto これですが、jlreq関連はさておき、latexbuilderの変更部分とそれに関連するところはcherry-pickしてマージしますか？
jlreq関連もまだexperimentalでよければそのまま突っ込んでもそんなに影響ないかと思います

[kmuto]

そうですね、jlreqもexperimentalでこういう方法があるよという提示にできればと。

[kmuto]

しかしちょっと今週はコミットを分けたりする時間がとれそうもないです… やれそうなのは使用ドキュメントを少し書いてそのままマージくらいでしょうか。

[takahashim]

む、なるほど、では私のほうでいじってみます

[takahashim]

とりあえず 86556be の@<uchar>のやつと 273d6f4 のcaptionblockを変更するやつは別PRで取り込みます。

[kmuto]

README.mdを書いて、syntax-bookが通るようにしたのでひとまず実験用としては足りそう。

styにお悩みコメントが多すぎる気はしているのでissue側に振ったほうがいいかなぁ…。

[kmuto]

ファイルの持たせ方をreview-jsbookと合わせました。
いちおうこれで「実験版として使えなくはない」ものになったかと思います。

[yuw]

現状だとドライバまわりやluatexでの単位zwなどの扱いでエラーが出てしまいます．
手許では次のdiffでビルドできるようになっています．
TeXのディストリビューションはTeX Live 2017 frozenなのですが，
他のディストリビューションだったりすると不要だったりしますか？

diff --git a/templates/latex/layout.tex.erb b/templates/latex/layout.tex.erb
index 59ae254f..1a1e1782 100644
--- a/templates/latex/layout.tex.erb
+++ b/templates/latex/layout.tex.erb
@@ -1,4 +1,8 @@
+<%- if @config['texcommand']=='lualatex' -%>
+\documentclass[<%= @documentclassoption %>]{<%= @documentclass %>}
+<%- else -%>
 \documentclass[dvipdfmx,<%= @documentclassoption %>]{<%= @documentclass %>}
+<%- end -%>
 <%= latex_config %>
 <%- if @config['texstyle'] -%>
 <%-   [@config['texstyle']].flatten.each do |x| -%>
diff --git a/templates/latex/review-jlreq/review-base.sty b/templates/latex/review-jlreq/review-base.sty
index f1f3e481..c94a2b26 100644
--- a/templates/latex/review-jlreq/review-base.sty
+++ b/templates/latex/review-jlreq/review-base.sty
@@ -76,11 +76,13 @@
 \fi
 \ifdefined\review@coverimage
   \def\reviewcoverpagecont{%
-\thispagestyle{empty}
-\begin{center}
-\expandafter\includegraphics\expandafter[\review@coverimageoption]{\review@coverimage}% includefullpagegraphicsを導入すべき?
-\end{center}
-\clearpage
+    \thispagestyle{empty}
+    \null\vfill
+    \centerline{\vbox to 0pt{\vss\hbox to 0pt{\hss
+      \expandafter\includegraphics\expandafter[\review@coverimageoption]{\review@coverimage}% includefullpagegraphicsを導入すべき?
+      \hss}\vss}}
+    \vfill\null
+    \clearpage
   }
 \fi
 
@@ -91,7 +93,12 @@
        \begin{titlepage}
        \thispagestyle{empty}
        \begin{center}%
-       \mbox{} \vskip5zw
+       \mbox{}%
+       \ifx\review@jlreq@driver\review@jlreq@driver@luatex
+         \vskip5\zw
+       \else
+         \vskip5zw
+       \fi
        \reviewtitlefont%
        {\Huge\review@booktitlename\par}%
        \ifdefined\review@subtitlename
@@ -105,8 +112,14 @@
        \review@titlepageauthors
        \end{tabular}\par}%
        \vfill
-       {\large\review@date \review@intn@edition\hspace{2zw}\review@intn@publishedby\par}%
-       \vskip4zw\mbox{}
+       {\large\review@date \review@intn@edition%
+         \ifx\review@jlreq@driver\review@jlreq@driver@luatex
+           \hspace{2\zw}%
+         \else
+           \hspace{2zw}%
+         \fi
+         \review@intn@publishedby\par}%
+       \vskip4\zw\mbox{}
        \end{center}%
        \end{titlepage}\clearpage
     }
diff --git a/templates/latex/review-jlreq/review-jlreq.cls b/templates/latex/review-jlreq/review-jlreq.cls
index d105b207..97f844e6 100644
--- a/templates/latex/review-jlreq/review-jlreq.cls
+++ b/templates/latex/review-jlreq/review-jlreq.cls
@@ -89,6 +89,7 @@
 
 % エンジンとドライバの情報。jlreq-trimmarksが定義されていればそっちから持ってくる。
 % 定義されていなければjlreqから持ってくる
+\def\review@jlreq@driver@luatex{luatex}
 \def\review@jlreq@engine{}
 \def\review@jlreq@driver{}
 \ifdefined\jlreq@trimmarks@engine
@@ -106,7 +107,7 @@
   \ifx o\jlreq@trimmarks@driver\def\review@jlreq@driver{dviout}\fi
 \else% jlreqから持ってくる
   \ifx l\jlreq@engine
-    \def\review@jlreq@driver{lualatex}
+    \def\review@jlreq@driver{luatex}
   \else
     \def\review@jlreq@driver{dvipdfmx}
   \fi
@@ -129,7 +130,10 @@ setpagesize=false]{hyperref}[2016/06/24 v6.83q]
 \RequirePackage[T1]{fontenc}
 \RequirePackage{textcomp}%T1/TS1
 \RequirePackage{lmodern}%\ttdefault: lmtt
+\def\review@jlreq@driver@luatex{luatex}
+\ifx\review@jlreq@driver\review@jlreq@driver@luatex\else
 \RequirePackage[\review@jlreq@driver]{pxjahyper}[2012/05/27 v0.2]
+\fi
 \RequirePackage{tikz}
 \usetikzlibrary{calc}
 \RequirePackage{multirow}

[kmuto]

ありがとうございます、手元ではuplatexだけで試していたので気付いてませんでした…（TeXLive2016だとそもそもlualatex+jlreqがダメ）。

もともとdvipdfmxオプションがlayout明示なのがだいぶ気持ちが悪いので、@documentclassoption側の処理に持っていきたい気がしています。

[yuw]

もともとdvipdfmxオプションがlayout明示なのがだいぶ気持ちが悪いので、@documentclassoption側の処理に持っていきたい気がしています。

大賛成です！
グローバルオプションを決め打ちでは無理が出てしまいますしね．
必要ならconfig.ymlで完全に制御できるとよいと思います．

[kmuto]

こんな判定だと荒っぽいですかね…

      if @texcompiler && @texcompiler =~ /platex/ &&
         @documentclassoption !~ /dvipdfmx/
        @documentclassoption = "dvipdfmx,#{@documentclassoption}"
      end

利点

• 旧来データのconfig.ymlをいじらなくていい

欠点

• uplatexでdvipdfmxを使いたくないという場合に困る（のが実際あるのか）
• *platex な名前でPDF直接書けるような名前のが登場すると困る

lualatexを使っている人はまだ少なく、ドキュメント記載されてればわかるのではという感じもするので、「dvicommandパラメータの値がdvipdfmx以外に設定されている（たとえばnullとか）なら入れない」という方法もあります。