layout.tex.erbの分割

[kmuto]

(元の実装からだいぶ変えたので概要を変更)

layout.tex.erbのdocument環境内のerbコードをプリアンプルでの事前定義+マクロ参照 に置き換える。

• バージョン3の機能とし、バージョン2以下をconfig.ylmで指定しているプロジェクトでは旧来のlayout.tex.erbを使うようにして互換性を保持する
• review-initに--latex-templateオプションを追加し、templates/latex/テンプレート名 からstyファイル群をコピーするようにする。zipやURI指定も考える? デフォルトはreview-jsbook

templates/latex/review-jsbook に置くようにしてみる。review-initで無指定だったときにはこれらがsty/内にコピーされるというイメージ。

ユーザー置き換えやすいようにフェーズで分割してみたが、かえって多すぎてややこしくなってしまっている気もする。

layout.tex.erbは抽象化し、__define.sty、__coverpage.tex、__frontmatter.tex、__mainmatter.tex、__backmatter.texを読み込むようにしている。

[kmuto]

• review_versionを見て、「3以上」だった場合は新形式、nilまたは未満だった場合（既存物）は旧式のlayout.tex.erbを使うようにした
• review-init -tオプションを追加し、templates/latex/テンプレート名 の内容をsty/にコピーするようにした

[kmuto]

ifthenパッケージを使って、layout.tex.erbのプリアンプル部分を定義に書き換えてみました。
document内もいちおうできそうではあるかな…

視認性は低め。特に3項演算子はわかりにくいので、支援メソッドを作ったほうがよさそう。

[kmuto]

cover、titlepageをsty領域に移動を試みたけど、TeX力が低いためなぜこうなるのかわからん…

! Missing \endcsname inserted.
<to be read again> 
                   \textwidth 
l.390 \reviewcoverpagecont

[kmuto]

document環境内は全部マクロ化してみた。

• coverのincludegraphicsのオプションを展開しようとすると、オプション文字列の\textwidthで\endcsnameになってしまう？
• colophonがうまくいかない。何かテキストを入れると! LaTeX Error: Missing \begin{document}.になってしまう。

[kmuto]

全部をプリアンプル側に移動しました。

機能互換性は保てていると思うのですが、自信はあまりありません。実稼働を受けないとわからないところも多そう。

• マクロ名はこんなかんじでよさそう?
• define領域(erb書いてるところ)はpdfmaker.rbの1メソッドで賄ってしまったほうがよさそう？
• sty定義部分は1ファイルにする? 複数にする? (今はreviewmacro.styの内容もマージして1つにしてる)
• ユーザのconfig.ymlのtexstyleパラメータでは複数のファイルを指定させる? それとも1ファイルにしてそのファイルからusepackageする?

[kmuto]

なお、

coverのincludegraphicsのオプションを展開しようとすると、オプション文字列の\textwidth\endcsnameになってしまう？

展開を遅らせるためにexpandafterを使う必要があった。

colophonがうまくいかない。何かテキストを入れると! LaTeX Error: Missing \begin{document}.になってしまう。

単にdef忘れてた…。

[kmuto]

• 大枠はこれでいいのか
• パラメータ定義は本当にこの形でいいのか（あとあと禍根にならないか）
• パラメータ定義、環境コマンド定義 はこのままlayout.tex.erbにあるのは大きすぎるので分割したいがどうするのがいいか（環境コマンド定義からは最後のtexstyle読み込み以外erbコードは除去済み）
• document環境内はこれで網羅できてそうか。erbコードは除去済み
• テンプレート選択のロジックはこれでいいのか

[kmuto]

https://docs.google.com/document/d/17VUrns_U89s6Svy9nNNSqFcA3HJHTydK_aQDf2mWCBA/edit?usp=sharing

[takahashim]

いったんmasterをマージした方がよさそう（rebaseしたらconflictしたので）

[kmuto]

いったんmasterをマージした方がよさそう（rebaseしたらconflictしたので）

やってみます

[kmuto]

masterを取り込みました。

[takahashim]

おお、テストも通った。ありがとうございます！

• このままsquashしてマージする
• 「textbf -> reviewbold みたいな変更」と「テンプレート変更」を切り分けて、2つのコミットにしてマージする

のどっちかがよいかと思うんですが、どちらがよさそうですか？ @kmuto

（あとで大幅見直しをするなら後者の方が楽かな…と思わないでもないです。）

[kmuto]

textbf→reviewboldは結局テンプレート変更とも絡んでくるので、私としては分離する意味はないかなと思いました。revertすることもなさそうですし。

templates/latex/config.erbの中身やtemplates/latex/review-jsbook内については考慮の余地はありますが、骨格部分はこれでreview_version:2.0系資産に影響を出さずに3.0系展開を進めやすいのではと考えています。

[takahashim]

了解です、ではそのままsquash and mergeしますね

[takahashim]

@kmuto これのreview-jlreqがどうにもビルドできなさそうなのですが、動くサンプル等はありますか？

[kmuto]

あら。夜に見てみます。