Kouhei Sutou
kou****@clear*****
2014年 8月 24日 (日) 22:50:06 JST
須藤です。 In <CAH9Y0y6D=Ca6sO****@mail*****> "[groonga-dev,02684] Re: お願い: Heroku用GroongaのMessagePack対応" on Sun, 24 Aug 2014 21:31:43 +0900, 杉本涼 <sugry****@gmail*****> wrote: > Heroku用GroongaのMessagePack対応やりました。 > 遅くなってすみません！ いえいえ、まったく遅くないです。 ありがとうございます！助かりました！ curlのオプション、-Oより--remote-nameの方がわかりやすくてい いですね。Groongaの方も--remote-nameを使うようにしました。 そういえば、この間、畑ケさんがcurlはデフォルトだと404のときも 終了ステータスが0になってエラーに気づきにくいと書いていました ねぇ。Heroku用Groongaのビルドスクリプトでも--failオプションを 有効にしておいた方がURLが変わったときに気づきやすいかもしれな いですねぇ。 他のコードも見てみました。少し思ったところがあるのでコメント しますね。参考までに！ build_msgpackの最後でmsgpack_prefixをreturnしているところです が、groonga_base_nameのようにmsgpack_prefixもメソッドとして定 義するのはどうかなぁと思いました。そうすれば "--with-message-pack=#{build_msgpack}", のところを "--with-message-pack=#{msgpack_prefix}", と書けるようになるので、--with-message-packにどういう値を指 定しているかがわかりやすくなるんじゃないかと思いました！ -- 須藤 功平 <kou****@clear*****> 株式会社クリアコード <http://www.clear-code.com/> Groongaベースの全文検索システムを総合サポート: http://groonga.org/ja/support/ パッチ採用 - プログラミングが楽しい人向けの採用プロセス: http://www.clear-code.com/recruitment/ コードリーダー育成支援 - 自然とリーダブルコードを書くチームへ: http://www.clear-code.com/services/code-reader/
