yayugu / net-radio-archive

MIT License
188 stars 29 forks source link

設定した単語が含まれていたらシムリンク作成 #28

Closed soramugi closed 8 years ago

soramugi commented 8 years ago

新年明けましておめでとうございます。

年末に佐倉綾音にドはまりしまして、佐倉綾音のラジオを毎回より分けるのも面倒なので指定した単語が含まれるファイル名の場合には別途シムリンクを張るようにしてみました。 おかげで佐倉綾音がより効率的に摂取出来るので大変便利になるかと思われます。こちらの変更を追加して欲しいと思うのですがどうでしょうか。

シムリンク張る場所が"#{Settings.archive_dir}/0_selections"とかにしてますが別の場所が良さそうとか、ここ処理変えた方が良さそうとかあれば何なりと。

yayugu commented 8 years ago

いいっすね~

soramugi commented 8 years ago

warpzone commented 8 years ago

この変更によって、設定した単語がファイル名に含まれていない場合 Main::move_to_archive_dir の 戻り値が常に nil になってしまい、DB に記録されている state が failed になってしまいます。

今回追加された処理を、 0_latest 下へシンボリックリンクを張る既存処理より前に行うようにすれば この問題は解決しますが、いかがでしょうか。

yayugu commented 8 years ago

マジで。それはまずい

yayugu commented 8 years ago

おそくなりましたがdevelop(個人用開発ブランチ)の方では修正。数日中にmasterに反映させます

warpzone commented 8 years ago

lib/radiru/recording.rb の方も修正お願いします!

yayugu commented 8 years ago

直しました。ただらじる使っていないので動くかはわからないっす

yayugu commented 8 years ago

今後はらじるも使って確認するようにしますね

warpzone commented 8 years ago

ありがとうございます。 こちらで、らじるらじるでも正常に動いていることを確認しました!