杉本涼
sugry****@gmail*****
2014年 8月 9日 (土) 11:20:36 JST
杉本です。 buildpack-groongaのheroku用のビルド最新バージョン検出↓の改良が終わりました。 https://github.com/groonga/heroku-buildpack-groonga/blob/master/bin/compile なんか変なところがあったら、アドバイスしてもらえるとうれしいです。 2014年8月5日 22:44 Kouhei Sutou <kou****@clear*****>: > 須藤です。 > > In <CAH9Y****@mail*****> > "[groonga-dev,02571] Re: Groongaのbuildpackについて" on Tue, 5 Aug 2014 > 08:20:30 +0900, > 杉本涼 <sugry****@gmail*****> wrote: > > > GitHub APIは、GitHubトークンがなくても使えるみたいです。(取得するだけなら) > > おぉ、そうなんですね。 > > > git push したので↓おかしいところがあったらアドバイスしてもらえるとうれしいです。 > > > > > https://github.com/groonga/heroku-buildpack-groonga/compare/1ead240fa8...99f0ee1d15 > > みてみました。 > > require "octokit"がないので動かないかなぁと思いました。 > > そもそも、require "octokit"ができるかというと、できないと思 > うんですよね。buildpackが動く環境にoctokit gemがインストール > されているとは限らないので。 > > なので、標準ライブラリーだけで実現しないといけないですねぇ。 > > あと、気になったのはここらへんです。 > > * tags[0].name.delete("v")は行儀が悪いなぁ > * タグを打ってからHeroku用Groongaをビルドするまでの間に > デプロイすると失敗しそう > > delete("v")の方ですけど、このコードは「v」という文字を全部削 > 除するっていう意味のコードになるんですが、ここでやりたいこと > は「最初のvを削除する」ですよね？必要以上の処理をしようとし > ているので行儀が悪いなぁと感じました。 > > tags[0].name.sub(/\Av/, "") > > とかの方がより「意図を表したコード」になって適切じゃないかと > 思いました。 > > > もうひとつの失敗しそうの方ですが、新しいバージョンの方から > Heroku用のビルドがあるかを確認していって、最初に見つかったバー > ジョンを使う、みたいにするといいんじゃないかと思いました。 > そうすれば、タグを打ったとしてもHeroku用のビルドがなければそ > のバージョンはスキップするので失敗しなそうです。 > > > という感じのことを思ったんですが、もう一歩改良してもらえると > うれしいです！ > > -- > 須藤 功平 <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/ > > _______________________________________________ > groonga-dev mailing list > groon****@lists***** > http://lists.sourceforge.jp/mailman/listinfo/groonga-dev >
