whiteleaf7
commented
5 years ago macOS だとファイル名ちょっと長い程度じゃ全く問題ないので塩対応部分… 調べてみます
Closed jam7 closed 5 years ago
whiteleaf7
commented
5 years ago macOS だとファイル名ちょっと長い程度じゃ全く問題ないので塩対応部分… 調べてみます
whiteleaf7
commented
5 years ago From: /Users/whiteleaf/github/narou/lib/extension.rb @ line 40 File.write: 変換中:1
変換中:1
26: def File.write(path, string, *options, mode: nil) 変換中:1
27: return super if mode 変換中:1
28: 変換中:1
29: dirpath = File.dirname(path) 変換中:1
30: temp_path = File.join(dirpath, SecureRandom.hex(15)) 変換中:1
31: if File.extname(path) == ".yaml" && File.basename(dirpath) != Downloader::SECTION_SAVE_DIR_NAME :1
32: backup = "#{path}.backup" 変換中:1
33: end 変換中:1
34: 変換中:1
35: res = super(temp_path, string, *options) 変換中:1
36: if backup 変換中:1
37: super(backup, string, *options) 変換中:1
38: end 変換中:1
39: binding.pry 変換中:1
=> 40: File.rename(temp_path, path) 変換中:1
41: res 変換中:1
42: end 変換中:1
変換中:1
[1] pry(File)> path
=> "/Users/whiteleaf/github/narou/小説データ/小説家になろう/26/n2656ev 本能寺から始める信長との天下統一 《４２００万アクセス超え》☆戦国時代歴史改変タイ/長いよ.txt"
特に問題ないような気がするが
jam7
commented
5 years ago 特に問題ないような気がするが
setting.iniの変更が反映されない、というのはこちらのミスでした。こちらでも反映されていました。
aki1-asr
commented
5 years ago 自分も #220 のn6339doで同様のエラーが発生いたしました。 (Windows => Linuxへ環境を移行した際に発生)
自分の場合 filename-length-limit をtxtファイル名にも反映させるよう下記のパッチを当てて使用しておりますが、可能でしたらmergeしていただければ助かります。
--- narou.rb.org 2019-08-29 16:14:12.807045200 +0900
+++ narou.rb 2019-08-29 19:55:01.088471338 +0900
@@ -235,7 +235,7 @@
novel_setting.novel_title.presence || novel_data["title"],
true
)
- "[#{author}] #{title}#{ext}"
+ Helper.truncate_path("[#{author}] #{title}#{ext}")
end
end# 補足: ファイル名の最大長
自分の環境の場合CentOS7でXFSを使用しているため、最大ファイル名長が255byteとなっており、下記の通りオーバーしてしまったようです。
irb(main):005:0> title = '[緋色優希（旧 トワイライト）] おっさんのリメイク冒険日記 ～オートキャンプから始まる異世界満喫ライフ～（旧題 おっさんがオートキャンプしてたら、何故か異世界でキャンプする羽目になった）.txt'
irb(main):006:0> title.size
=> 98
irb(main):007:0> title.bytesize
=> 278@aki1-asr
自分の場合 filename-length-limit をtxtファイル名にも反映させるよう下記のパッチを当てて使用しておりますが、可能でしたらmergeしていただければ助かります。
確認します
whiteleaf7
commented
5 years ago https://jbbs.shitaraba.net/bbs/read.cgi/computer/44668/1511245701/192 現時点では間違いなく仕様ですが（バグではなく実装時点では意識的に出力ファイル名には適用しなかった）、
対策するとしたら title-length-limit 的なのを新しく作ることを検討します
という考えは改めようかなと思いました。 つまり filename-length-limit を出力電書ファイル名にも効かせる = https://github.com/whiteleaf7/narou/issues/325#issuecomment-526143988 の適用をする
なぜ出力ファイル名に適用しなかったのかは、出力ファイル名は別途制御する方法があったからですが、filename-length-limit を適用せざるを得ない環境では、出力ファイル名も同様の問題が発生する可能性が高いので、出力ファイル名制御の設定の手間がたしかに省けるな、と思い直しました。
whiteleaf7
commented
5 years ago @jam7 @aki1-asr version 3.4.4 released
aki1-asr
commented
5 years ago @whiteleaf7 まさか1日でご対応いただけるとは思いませんでした。ありがとうございます。 updateして確認してみましたが、パッチなしで正常にConvertできました。
whiteleaf7
commented
5 years ago デフォルト５０文字だからいままで出力してたファイル名が変わって栞が消えるみたいな ことはあるかもですねとか今更思ったけど設定で１００文字とか２００文字とかに増やしてくだされ…
aki1-asr
commented
5 years ago @whiteleaf7
すいません、リリース後で申し訳ないのですけども、もう1点オーバーする部分がありました。
バックアップする際のファイル名生成処理についてもタイトルをそのまま使用しているため、おそらく下記の部分にも Helper.truncate_path() が必要かなと思います。
https://github.com/whiteleaf7/narou/blob/develop/lib/command/backup.rb#L34
# 実際n6339doでbackupしてみたら Errno::ENAMETOOLONG が発生しました
尚、現役で使われているファイルシステムの場合殆どが255文字 or 255byteが上限のようです(※)。
UTF-8, UTF-16では長くとも4byteであるため、50文字上限の場合ワーストケースでも200byteとなり、backup時の19byte( _%Y%m%d%H%M%S.zip )を追加してもエラーになることは無さそうです。
(絵文字50文字のタイトルとかでない限り全部4byteなんてありえないですが・・・)
whiteleaf7
commented
5 years ago バックアップに関しては影響軽微という事でそのうち直しますね
whiteleaf7
commented
5 years ago やっぱり filename-length-limit で共有設定は厳しかったかもしれない 当初想定していた別の設定名に分けようかな
whiteleaf7
commented
5 years ago 厳しい理由:
いままで出力ファイル名は５０文字以上も許容していた。 （「文字」数換算なので、バイト換算では許容範囲内の作品はかなりあると思われる）
ファイル名の長さの計算は、出力ファイル名に限れば作者名部分も含むので、意外と５０文字に引っかかりやすかった。 それが強制５０文字で端折られると、出力ファイル名が変わる作品が多い。 →別のファイルが増える。栞も変わってしまう（後方互換がない）
いままでの設定にあわせて filename-length-limit を増やそうとする →出力ファイル名はいいかもしれないが、各話保存ファイルの名前も増えるため、再DLが発生してしまう
以上の理由から、今回のアプデはなかったこととして、別設定名に変えます
いつもお世話になっています。また、エラーが出るパターンがありましたので報告です。
https://ncode.syosetu.com/n2656ev/
これを変換すると以下のエラーとなります。フォルダ名の段階ではタイトルが省略されているのですが、.txtにするところで、タイトルが省略されなくなってしまうようでした。著者名が長いのも関係があるのかもしれません。
以前、類似の#220では、setting.iniのnovel_titleを短くすることで回避できていましたが、今回はnovel_titleやoutput_filenameを弄っても、rename先のファイル名が変わらずエラー内容に変化ありませんでした。
また、#220の際に伺った、database.yaml中のfile_titleを縮めてフォルダー名も合わせて変更するというのをやってみましたが、確かにフォルダ名は変わったのですが、ファイル名が長すぎるようで効果ありませんでした。
database.yaml中の、titleとfile_titleを縮めることで、なんとか回避しています。