プロジェクト名にスペースが入るとepubmakerの挙動がおかしくなる

[suzukin]

リリース日にすみません、、スペース入れるなって話かもですが一応ご報告まで。

/XXXX YY
というフォルダー名のプロジェクトがあったとして、

/XXXX YY 配下で review-epubmaker config.yml を実行すると
/XXXX YY と同じ階層に XXXX.zip として生成されます。

[takahashim]

これは「スペース入れるな」というか文字種別チェックを入れた方が良さそうですね…。
明日以降に対応させていただきたく 🙇

[suzukin]

よろしくお願いします！

[kdmsnr]

とりあえず " で囲ってみましたが…… " が含まれるプロジェクトのときに別のところでコケますね。

diff --git a/lib/epubmaker/epubcommon.rb b/lib/epubmaker/epubcommon.rb
index f2feb5d..22b0b5f 100644
--- a/lib/epubmaker/epubcommon.rb
+++ b/lib/epubmaker/epubcommon.rb
@@ -411,8 +411,8 @@ EOT
     end

     def export_zip(tmpdir, epubfile)
-      Dir.chdir(tmpdir) {|d| `#{@producer.params["epubmaker"]["zip_stage1"]} #{epubfile} mimetype` }
-      Dir.chdir(tmpdir) {|d| `#{@producer.params["epubmaker"]["zip_stage2"]} #{epubfile} META-INF OEBPS #{@producer.params["epubmaker"]["zip_addpath"]}` }
+      Dir.chdir(tmpdir) {|d| `#{@producer.params["epubmaker"]["zip_stage1"]} "#{epubfile}" mimetype` }
+      Dir.chdir(tmpdir) {|d| `#{@producer.params["epubmaker"]["zip_stage2"]} "#{epubfile}" META-INF OEBPS #{@producer.params["epubmaker"]["zip_addpath"]}` }
     end

     def legacy_cover_and_title_file(loadfile, writefile)

[kdmsnr]

あ、shellescapeってのがあるんですね……知らなかった。それ使えばよさそう。

[takahashim]

あ、でも、この対応を入れても今後のバージョンで空白文字がNGになる可能性はあるのでご了承ください 🙇 > @suzukin