Windows環境でEPUBのコンテンツ生成パスがおかしくなる

[kmuto]

#1008 (comment)

[kmuto]

状況からするとこのあたり。

    def export_zip_rubyzip(epubfile)
      Dir.chdir(tmpdir) do
        Zip::OutputStream.open(epubfile) do |epub|
          root_pathname = Pathname.new(tmpdir)
          epub.put_next_entry('mimetype', nil, nil, Zip::Entry::STORED)
          epub << 'application/epub+zip'

          export_zip_rubyzip_addpath(epub, File.join(tmpdir, 'META-INF'), root_pathname)
          export_zip_rubyzip_addpath(epub, File.join(tmpdir, 'OEBPS'), root_pathname)
          if @config['zip_addpath'].present?
            export_zip_rubyzip_addpath(epub, File.join(tmpdir, @config['zip_addpath']), root_pathname)
          end
        end
      end
    end

    def export_zip_rubyzip_addpath(epub, dirname, rootdir)
      Dir[File.join(dirname, '**', '**')].each do |path|
        next if File.directory?(path)
        relpath = Pathname.new(path).relative_path_from(rootdir)

chdirで入ってカレントになっているはずなのに、relative_path_fromでなぜか../..みたいになってしまう？

ただ、手元のWindows+Ruby 2.5では問題ないので、バージョンなのか環境なのかよくわかりません。

[mitsuo0114]

Windows7で再現されました。

    def export_zip_rubyzip_addpath(epub, dirname, rootdir)
      Dir[File.join(dirname, '**', '**')].each do |path|
        next if File.directory?(path)
        relpath = Pathname.new(path).relative_path_from(rootdir)

ここのpathとrootdir、アウトプットのrelpathを確認したところ、以下のようにWindowsのパスのショートネームとロングネームが混ざっており、relpathが共通パスを検出できなくなっているようです。

path:
C:/Users/takahiro.a.ono/AppData/Local/Temp/d20190522-13176-62lky7/OEBPS/titlepage.xhtml

rootdir
C:/Users/TAKAHI~1.ONO/AppData/Local/Temp/d20190522-13176-62lky7

relpath
../../../../../takahiro.a.ono/AppData/Local/Temp/d20190522-13176-62lky7/OEBPS/titlepage.xhtml

ruby上の解決案としては$SAFE = 1を設定するのがありそうですが、tmpdirの位置を変更するなども候補としては良さそうです。
https://docs.ruby-lang.org/ja/latest/method/Dir/s/tmpdir.html

[kmuto]

うーん、つらい…

本来Ruby Windows版のほうでカバーしてほしい事象なので、Re:VIEW側でOS分岐でなんとかするというのはあまりやりたくない気持ちです。

テンポラリな回避策としては、debug:trueにすれば一時フォルダの位置がプロジェクトフォルダ下になるので、発生しなくなる？

本質的な解決のほうは…tmpdirの位置を変えるというのは気乗りしないですね。Windowsだったら呼び出し前後でSAFE=1とする？（ということってできたっけかな…？ 別のWindows Ruby実装ではまた違うみたいな可能性もありそうな。

[mitsuo0114]

mitsuo0114@478f17f

tmpdirからDirで走査するときにショート=>ロング変換されてマッチングできないのが原因なので
tmpdirを生成する段階でロング変換をしてみました。
直し方は微妙ですが一応これで動きました。一つの修正候補としていただければ。

[kmuto]

ありがとうございます！
そもそもこのtmpdirのロジックがよくないのでいずれリファクタリングしようと思いますが、
ひとまずいただいたパッチを入れました（new_tmpdirは1行化しました）。