PDFMaker: reflect imagedir config

[m-shibata]

No description provided.

[m-shibata]

pdfmakerもconfig.ymlのimagedirで画像ディレクトリの参照元を変更できるとうれしいです。
epbumakerやwebmakerを参考に、"./images"でハードコードされていた部分を変更してみました。

[kmuto]

ありがとうございます。そもそもimagedirがちゃんと動いていないのではという報告を受けていたのを思い出しました…
EPUBのほうはただコピーしているだけでちゃんと参照できていないですね。

いただいているパッチのほうでも、実際にはうまく画像を探せないと思うのですが、@m-shibata さんの環境では実行うまくいっていますか？

[kmuto]

とりあえずこのパッチ自体は問題ないのでmergeしますね。

ただし、imagedirパラメータでの変更は現状ちゃんと動かないので、そっちを直さないといけません。
ref #453

[m-shibata]

@kmuto さん、

マージありがとうございます。

いただいているパッチのほうでも、実際にはうまく画像を探せないと思うのですが、@m-shibata さんの環境では実行うまくいっていますか？

はい、Re:VIEW 2.1にこのパッチを適用した状態でのreview-pdfmakerについては、期待通りの動作をしています。またreview-epubmakerをやってみたところ、こちらも期待通りの画像になっていました。

EPUBのほうはただコピーしているだけでちゃんと参照できていないですね。

EPUBの方はちょっとわからないのですが、PDFの方は次のような挙動になっていると認識しています。

• imagedirはコピー元の画像ディレクトリを指定するものである
• imagedirにある画像はpdfmakerによって、BOOK-pdf/images/ディレクトリにコピーされる
• reファイルからTeXファイルへの変換が始まる
• //imageはimage_indexによってパスが決まるが、このときのベースディレクトリは"#{book.basedir}/#{@book.image_dir}"である
• book.basedirは.となる（＝BOOK-pdf）
• book.image_dirであり、これはconfig["image_dir"]であり、初期値はimagesである

よってTeXファイルとしては./images/...をincludegraphicsに設定するので、画像を参照できるのだと思います。

#453 ではEPUBの場合は「ソースの画像パスとターゲットの画像パスが一緒になってしまっている」と記載されていますが、この「ソースの画像パス」も「ターゲットの画像パス」も上記で言うところの./images（BOOK-epub/images）なので、これが一緒になる分には問題ないのではないでしょうか。

追記：「ターゲットの画像パス」は./imagesですが、実際の位置は「BOOK-epub/BOOK-epub/OEBPS/images/」でした。ただ相対パスになっているので参照可能ではあるようです。

(1) ./images_alt/ => imagedirで設定するディレクトリ
(2) ./BOOK-epub/images/ => config['image_dir']として参照されるディレクトリ（ソースの画像パス）
(3) ./BOOK-pdf/images/ => config['image_dir']として参照されるディレクトリ
(4) ./BOOK-epub/BOOK-epub/OEBPS/images/ => EPUBで参照されるディレクトリ（ターゲットの画像パス）

#453 の頃から挙動が変わったのかもしれませんが、現状のimagedirは「ソースのパスのみを指定するべきもの」ではなく、「ソースのパスのコピー元を指定すべきもの」になっている気がしています。