Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] migrate to uplatex #541

Merged
merged 11 commits into from
Apr 17, 2016
Merged

[WIP] migrate to uplatex #541

merged 11 commits into from
Apr 17, 2016

Conversation

kmuto
Copy link
Owner

@kmuto kmuto commented Apr 12, 2016

uplatex化する変更。
#536

  • デフォルトをuplatex、--kanjiオプションなしに。documentclass jsbookのオプションにuplatex追加
  • hyperrefをuplatexでの利用方式に
  • METACHARSからネイティブ表現可能なものを除去
  • deluxeクラスの利用は外してみた

紙面の見栄えがplatex版と異なっている(柱まわりなど)。
platexに戻したいというときにはMETACHARSとlayoutが困りそう。

@kmuto
Copy link
Owner Author

kmuto commented Apr 12, 2016

ちなみにuplatexの縦書きはイバラの道なかんじがする…

@takahashim
Copy link
Collaborator

こちらもとりあえずrubocopがエラーを出すやつだけ直しておきますね

@takahashim
Copy link
Collaborator

conflictしてたのでmasterをmergeしました

@takahashim
Copy link
Collaborator

METACHARS改め@metacharsの処理はあんまりスマートではないですが、簡単にはならなさそうですし、そもそも後ろ向きの互換性のための機能なのでこれで良いかなあ、という気がしております。

@kmuto
Copy link
Owner Author

kmuto commented Apr 16, 2016

ではひとまずコードとしてはOKとして、uplatex化してしまってよいかな…

以前のplatexで使うには をWikiかdocに書いておくべきか。

When an (old) content doesn't have such attributes, Re:VIEW failed.
@kmuto
Copy link
Owner Author

kmuto commented Apr 16, 2016

おっと、その設定ちょうど今追加していたところでした。

@kmuto kmuto closed this Apr 16, 2016
@kmuto kmuto reopened this Apr 16, 2016
@takahashim
Copy link
Collaborator

takahashim commented Apr 16, 2016

被ってたらすみません、1.7用文書で試したらいきなり死んだので、Configureのデフォルト値を追加しました。
これを修正した上で、

  • review_versionを追加
  • texdocumentclassの2番めの要素にuplatexを追加 ←こちらはLaTeXのエラーを見ても原因不明になりそうなので要注意

でビルドできました。

@kmuto
Copy link
Owner Author

kmuto commented Apr 16, 2016

ドキュメント doc/pdfmaker.ja.md を入れてみました。

余裕があればフォントマップ指定まわりも説明入れておきたいですね。

@takahashim
Copy link
Collaborator

現状でマージしましょうか?

@kmuto kmuto merged commit cf88e65 into master Apr 17, 2016
@kmuto kmuto deleted the use_uplatex branch April 17, 2016 00:54
@kdmsnr kdmsnr added this to the 2.0.0 milestone Apr 29, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants