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

tmpdirをTeXビルドパスに利用する #556

Merged
merged 3 commits into from Apr 16, 2016
Merged

tmpdirをTeXビルドパスに利用する #556

merged 3 commits into from Apr 16, 2016

Conversation

kmuto
Copy link
Owner

@kmuto kmuto commented Apr 14, 2016

cf. #549
/book-pdf にするのもできるけれども、とりあえず -pdf- で実装してみました。
begin-ensureで囲んで例外にかかわらず削除するようにしています。

取り込むにはpdfmaker_cmdのtestの削除が必要ですね。

@takahashim
Copy link
Collaborator

begin-ensureで囲むのはこれで良さそうというか、やっぱりこうなりますよね…。
build_pathメソッドの方は、ディレクトリの生成まではしないでパスの名前を作るだけにして、パスはデバッグ時とそうではない時で変える、というのがよいかと。

これって直接修正をこのブランチに追加してもよいですか?

@kmuto
Copy link
Owner Author

kmuto commented Apr 15, 2016

どうぞー

@takahashim
Copy link
Collaborator

build_pathメソッド、やっぱりディレクトリをこの中で生成しないと駄目そうですね…
というわけでその方向で修正してみました

@kmuto
Copy link
Owner Author

kmuto commented Apr 15, 2016

👍

@kmuto kmuto merged commit be01dba into master Apr 16, 2016
@kmuto kmuto deleted the pdftmpdir branch April 16, 2016 04:55
takahashim added a commit that referenced this pull request Apr 17, 2016
takahashim added a commit that referenced this pull request Apr 17, 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.

None yet

2 participants