We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
‘‘‘ def build_path "./#{@config["bookname"]}-pdf" end
def generate_pdf(yamlfile) remove_old_file @path = build_path() Dir.mkdir(@path) ‘‘‘ なので、特にdebug: trueのときだと再実行のたびにbookname-pdfフォルダを消さないといけないのが繁雑な感じがします。 ビルドパスをtmpnameにするか、mkdirする前に消去するかをしたほうがよいのではないでしょうか。
ちなみに、review initでできるサンプルがdebug: trueになるのも気持ちが悪いです……。
The text was updated successfully, but these errors were encountered:
これは修正した方が良さそうですね…。
というかもともとのdebugの目的としては、debug: falseの時は中間ファイルは全部消えて欲しいけどdebug: trueの時は(デバッグのために)中間ファイルを全部残して欲しい、というところから始まっていて、
(bookname)-pdf
というのが理想的です。が、debug: falseの時に一時ディレクトリが上手く消せなくて大量に残ることがあるため現状の挙動になった、と記憶しています。この辺を修正するべきですね…。
Sorry, something went wrong.
TeXのコンパイルエラーがあったときに止まっちゃうんですかね。ensure使うか、mktmpdirをブロックで囲んでしまうか…
#556 で実装提案してみました。
merge済み
No branches or pull requests
‘‘‘
def build_path
"./#{@config["bookname"]}-pdf"
end
def generate_pdf(yamlfile)
remove_old_file
@path = build_path()
Dir.mkdir(@path)
‘‘‘
なので、特にdebug: trueのときだと再実行のたびにbookname-pdfフォルダを消さないといけないのが繁雑な感じがします。
ビルドパスをtmpnameにするか、mkdirする前に消去するかをしたほうがよいのではないでしょうか。
ちなみに、review initでできるサンプルがdebug: trueになるのも気持ちが悪いです……。
The text was updated successfully, but these errors were encountered: