tDiary

On 16/06/2010 05:26, SHIBATA Hiroshi wrote:
> svn はただのミラーになってしまったので、tdiary-svn@... に配信する
> コミットログは github から送信する下の内容に変更したいと思うのですがどうでしょうか。
(snip)
> Changed paths:
>   M Rakefile
> 
> Log Message:
> -----------
> added task spec:core
> ----

さっそくの対応ありがとうございます。もちろん賛成です！

> github のコミットメールはトピックブランチも対象としてしまうのですが
> tdiary/tdiary-core の下に存在するものなら送信されても良いかなと思っています。

いま進行している開発の話題を共有できて、かつ「これさっさとマージしよう
よー」みたいなきっかけになると思うので、こちらも賛成です。

かずひこ

ただただしです。

2010/6/16 SHIBATA Hiroshi <shibata.hiroshi@...>:
> svn はただのミラーになってしまったので、tdiary-svn@... に配信する
> コミットログは github から送信する下の内容に変更したいと思うのですがどうでしょうか。

いいんじゃないでしょうか。

ただ、tdiary-svn@...は未登録アドレスからのメールを
蹴ると思うので、コミットログのfromに注意する必要があるかも。

-- 
ただただし - ツッコミは、短く鋭く愛を込めて♪
http://tdtds.jp/
@tdtds

柴田です。

> ただ、tdiary-svn@...は未登録アドレスからのメールを
> 蹴ると思うので、コミットログのfromに注意する必要があるかも。

from は noreply@... なんですが、
こちらから一方的に tdiary-svn に登録できますか?

もしくはホワイトリスト登録とか。

# ML の管理権限は持っていませんでした
-- 
SHIBATA Hiroshi mailto:shibata.hiroshi@...
URL:http://www.hsbt.org/

ただただしです。

2010/6/16 SHIBATA Hiroshi <shibata.hiroshi@...>:
>> ただ、tdiary-svn@...は未登録アドレスからのメールを
>> 蹴ると思うので、コミットログのfromに注意する必要があるかも。
>
> from は noreply@... なんですが、
> こちらから一方的に tdiary-svn に登録できますか?

noreply@...を登録しました。

> # ML の管理権限は持っていませんでした

shibata.hiroshi@...の管理者に追加してみた
けど、どうかな?

-- 
ただただし - ツッコミは、短く鋭く愛を込めて♪
http://tdtds.jp/
@tdtds

柴田です。

すでにテストメールとして最後の push 結果が配信されていますが
github の notify を tdiary-svn に流すようにしました。

tdiary-svn という名前はさておき、気になる変更点があったら
github のページで変更点を追いかけるようにして下さい。
1Clickの手間をかけてしまい申し訳ありませんがよろしくお願いします。

-- 
SHIBATA Hiroshi mailto:shibata.hiroshi@...
URL:http://www.hsbt.org/

須藤です。

In <AANLkTin9JPQXn4EzaRO1eutuBhpuXXzObuIp9voCnl3Y@...>
  "Re: [tDiary-devel] github のコミットメッセージ" on Thu, 17 Jun 2010 21:52:50 +0900,
  SHIBATA Hiroshi <shibata.hiroshi@...> wrote:

> すでにテストメールとして最後の push 結果が配信されていますが
> github の notify を tdiary-svn に流すようにしました。
> 
> tdiary-svn という名前はさておき、気になる変更点があったら
> github のページで変更点を追いかけるようにして下さい。
> 1Clickの手間をかけてしまい申し訳ありませんがよろしくお願いします。

gitでpushしたときにdiff入りのコミットメールを送るプログラムが
ある(*1)のですが、それをGitHubのPost-Receive hookで起動する
Webサービスがあります。

(*1) http://www.clear-code.com/blog/2010/5/18.html

それを使うと↓のようにdiff入りのメールが送られるようになりま
す。
  http://sourceforge.jp/projects/groonga/lists/archive/commit/2010-June/thread.html

もしよかったら設定しますが、いかがでしょうか。

-- 
須藤 功平 <kou@...>
株式会社クリアコード <http://www.clear-code.com/> (03-6231-7270)

柴田です。

> gitでpushしたときにdiff入りのコミットメールを送るプログラムが
> ある(*1)のですが、それをGitHubのPost-Receive hookで起動する
> Webサービスがあります。

申し出ありがとうございます。
ところで、Webサービスとは clear-code さんの中で動かしているものですよね?

私は主に変える側の人なので現状でも大丈夫なのですが、diff 付きメールの方が
嬉しいという人がいるのであれば、須藤さんにお願いしようと思います。

どうでしょうか > かずひこさんかな?
-- 
SHIBATA Hiroshi mailto:shibata.hiroshi@...
URL:http://www.hsbt.org/

On 21/06/2010 14:01, SHIBATA Hiroshi wrote:
>> gitでpushしたときにdiff入りのコミットメールを送るプログラムが
>> ある(*1)のですが、それをGitHubのPost-Receive hookで起動する
>> Webサービスがあります。
> 
> 申し出ありがとうございます。
> ところで、Webサービスとは clear-code さんの中で動かしているものですよね?
> 
> 私は主に変える側の人なので現状でも大丈夫なのですが、diff 付きメールの方が
> 嬉しいという人がいるのであれば、須藤さんにお願いしようと思います。

もちろん、diff付きメールの方が断然嬉しいです。
「gitになったおかげでコミットメールから意味のあるログが消えたりdiffがな
くなってコードレビューが面倒になりました」というのは、個人的には「デメ
リット＞＞メリット」でまったく嬉しくなかったので。

ちなみに、もし次回VCSを変更するときがあれば（二度あることは三度あるかも
しれませんから）、そのへんの利便性を損ねずに移行できる目処がついてからに
して欲しいと思っています。

かずひこ

須藤です。

In <AANLkTik49vHkPh6RCqqQdIpL-BwSwgpkYAuueDma_ux5@...>
  "Re: [tDiary-devel] github のコミットメッセージ" on Mon, 21 Jun 2010 21:01:58 +0900,
  SHIBATA Hiroshi <shibata.hiroshi@...> wrote:

>> gitでpushしたときにdiff入りのコミットメールを送るプログラムが
>> ある(*1)のですが、それをGitHubのPost-Receive hookで起動する
>> Webサービスがあります。
> 
> 申し出ありがとうございます。
> ところで、Webサービスとは clear-code さんの中で動かしているものですよね?

サービスを動かしているホストのドメイン名はclear-code.comにな
ります。

> 私は主に変える側の人なので現状でも大丈夫なのですが、diff 付きメールの方が
> 嬉しいという人がいるのであれば、須藤さんにお願いしようと思います。

それでは、こっちのメールにすることが決まったら、設定方法など
は直接柴田さん（とたださん？）に送りますね。決まったら教えて
ください。

-- 
須藤 功平 <kou@...>
株式会社クリアコード <http://www.clear-code.com/> (03-6231-7270)

柴田です。

> それでは、こっちのメールにすることが決まったら、設定方法など
> は直接柴田さん（とたださん？）に送りますね。決まったら教えて
> ください。

diff 付きメールにすることにしましたので設定をよろしくお願いします。
github 側の設定情報をお待ちしています。

# たださんにも送信お願いします。

-- 
SHIBATA Hiroshi mailto:shibata.hiroshi@...
URL:http://www.hsbt.org/

まちゅです。
＃ 反応の早さにビックリ

> 上記については、migrate スクリプトで走らせても良いと思いますが、今後作成されるで
> あろう plugin (特に contrib)についてはどうしましょう。
>
> 開発の手引みたいな部分で運用回避?

HerokuやGAE/JRubyなどのいろんな環境上に対応することも考慮すると、
pluginが直接PStoreを扱うのではなく
kdmsnrさんのtDiary会議案であげられていた

> * プラグインから使うIOの抽象化

で対応する方がいいのかなと思っています。

-- 
MATSUOKA Kohei < http://www.machu.jp/diary/ >

ただただしです。

(今日は仕事なもので)あまり深く考えられませんが:

2010/6/13 Kazuhiko <kazuhiko@...>:
> では早速と、1.9系に移行して久しい私の日記でこの処理を削ってみると、いき
> なりこんなエラーが出ました。
>
> incompatible encoding regexp match (UTF-8 regexp with ASCII-8BIT string)
> (Encoding::CompatibilityError)
(snip)
> data_dirのrecent_trackbacksを削除したら、もちろんエラーが出なくなりまし
> た。まだ深追いしていませんが、ひとまず報告しておきます。

これは単に、1.9移行後いちどもtrackBackを受信してないということ
では? 動作時に1.9向けに変換しても保存していないから、ファイルは
ずっと1.8用のままです。

>> UTF-8化のplugin/90migrate.rbでの @conf.tdiary_version のように、
>> 移行フラグを用いて関連ファイルを初回のみ書き換えるのが
>> よいと思ったのですが、独自プラグインが使うPStoreデータは
>> アップデートする人に手動で削除してもらう必要があります。
>>
>> キャッシュのように消してもいいデータだと
>> 影響は少ないのですが、消すと復旧できないデータを
>> PStoreに保存している場合は影響が大きそうですね。
>>
>> なにかいいアイデアはないでしょうか。
>
> migrateが走るタイミングで、ありえそうなPStoreデータを全部かたっぱしか
> ら、上記のパッチみたいな感じでencodingを変更してしまうしかないのではない
> かと思います。

読み込み時にそれが1.8で保存されたものかどうかがわかって、
かつ勝手に保存してもいいなら、変換してすぐ保存してしまう
ことで「見つけ次第自動変換」も可能かも知れない(未検証)。

-- 
ただただし - ツッコミは、短く鋭く愛を込めて♪
http://tdtds.jp/
@tdtds

まちゅです。

> 読み込み時にそれが1.8で保存されたものかどうかがわかって、
> かつ勝手に保存してもいいなら、変換してすぐ保存してしまう
> ことで「見つけ次第自動変換」も可能かも知れない(未検証)。

「見つけ次第自動変換」方式でパッチ（とテスト）を書いてみました。
http://github.com/machu/tdiary-core/compare/master...auto_convert_1.9
＃ さっそく、GitHubでトピックブランチを使ってみています

以下、基本的な動きです。

(1) PStoreトランザクションの開始時に1.8保存データかどうかを判定
   * 判定は:__ruby_versionキーの有無で実施
   * :__ruby_versionキーは呼び出し元からは隠す
     （想定しないキーが存在すると動かないプラグインが存在するため）
(2) 1.8保存データの場合は、トランザクションをいったん抜けて自動変換モードに移行
   * 自動変換モードではかずひこさん方式(Marshal::load)でエンコーディングをUTF-8に変換
   * トランザクションのread_onlyオプションに関わらずに強制保存
   * 変換後に、最初のトランザクションをもう一度実行する


いちおう、私の日記では問題なく動いています。
最初の1回目は10万回のProc#callが発生しますが、
2回目からは呼ばれなくなります。

パッチは以下の2点が気になるところです。添削歓迎します。

(1) :__ruby_versionキーの有無だけで判定しているので、ASCII-8BITデータが混入する恐れがある。
 * Ruby1.9から1.8にダウングレードする場合
 * Ruby1.9でcompatible.rbをロードせずにPStoreに書き込むスクリプトを動かした場合
    （デフォルトのencodingがUTF-8であれば大丈夫ですが）
(2) @force_convert_8bit_dataをフラグのように使っているので、ソースの見通しが悪い

-- 
MATSUOKA Kohei < http://www.machu.jp/diary/ >

柴田です。

> 「見つけ次第自動変換」方式でパッチ（とテスト）を書いてみました。
> http://github.com/machu/tdiary-core/compare/master...auto_convert_1.9
> ＃ さっそく、GitHubでトピックブランチを使ってみています

> (2) @force_convert_8bit_dataをフラグのように使っているので、ソースの見通しが悪い

フラグのように使わないで動かすようにパッチを書いてみました。

http://github.com/hsbt/tdiary-core/compare/080a...8a17

あと、machu さんの spec は最初は落ちてしまいました。
深追いはしていないのですが、オーバーライドした PStore.dump が
spec で should する前に呼ばれているようで、最初から table[:__ruby_version] に
値が入っていました。

上記のパッチでは便宜上 dump で入れている :__ruby_version を
無効化して動作を確認しています。

# こういう時の spec ってどう書くのがいいんですかね。
-- 
SHIBATA Hiroshi mailto:shibata.hiroshi@...
URL:http://www.hsbt.org/

まちゅです。

>> 「見つけ次第自動変換」方式でパッチ（とテスト）を書いてみました。
>> http://github.com/machu/tdiary-core/compare/master...auto_convert_1.9
>> ＃ さっそく、GitHubでトピックブランチを使ってみています
>
>> (2) @force_convert_8bit_dataをフラグのように使っているので、ソースの見通しが悪い
>
> フラグのように使わないで動かすようにパッチを書いてみました。
>
> http://github.com/hsbt/tdiary-core/compare/080a...8a17

ありがとうございます。
同時アクセスに対応できるように、ファイルのロック処理を入れました。
http://github.com/machu/tdiary-core/compare/8a170266...41c6a378

> あと、machu さんの spec は最初は落ちてしまいました。

ごめんなさい。specファイルをpushするのを忘れていました。
テスト用のPStoreファイル作成後にcompatible.rbをrequireしています。

-- 
MATSUOKA Kohei < http://www.machu.jp/diary/ >

柴田です。

ごめんなさ(ry

先ほどのパッチでは常に load_proc を呼び出していてちっとも改善に
なっていませんでした。
その後 machu さんと色々やって、以下のような形になりました。

http://github.com/hsbt/tdiary-core/compare/b394...auto_convert_1.9

上記で良さそうに見えますが、master に入れてもいいでしょうか。
-- 
SHIBATA Hiroshi mailto:shibata.hiroshi@...
URL:http://www.hsbt.org/

ただただしです。

日本の勝利を祝してコミットしてください!

2010/6/13 SHIBATA Hiroshi <shibata.hiroshi@...>:
> 先ほどのパッチでは常に load_proc を呼び出していてちっとも改善に
> なっていませんでした。
> その後 machu さんと色々やって、以下のような形になりました。
>
> http://github.com/hsbt/tdiary-core/compare/b394...auto_convert_1.9
>
> 上記で良さそうに見えますが、master に入れてもいいでしょうか。

-- 
ただただし - ツッコミは、短く鋭く愛を込めて♪
http://tdtds.jp/
@tdtds

柴田です。

> 日本の勝利を祝してコミットしてください!

master, svn/trunk の双方にコミットしました。

問題を見つけてくれた machu さん、ありがとうございました。
-- 
SHIBATA Hiroshi mailto:shibata.hiroshi@...
URL:http://www.hsbt.org/

On 15/06/2010 00:34, SHIBATA Hiroshi wrote:
>> 日本の勝利を祝してコミットしてください!
> 
> master, svn/trunk の双方にコミットしました。

できれば、

Log Message:
-----------
Merge branch 'master' into svn

↑もっと意図の理解しやすいコミットログにして欲しいのですが、もしsvnではこ
んなのしか無理だけどどこそこに見に行ったらちゃんと書いてあるよとかでした
ら、そのあたりもアナウンスしてくださると幸いです。

それはそれとして、どうせならRuby-1.8 EUC-JP → Ruby-1.9 UTF-8というマイグ
レートも実現できたりしそうでしょうか？

かずひこ

> Log Message:
> -----------
> Merge branch 'master' into svn
>
> ↑もっと意図の理解しやすいコミットログにして欲しいのですが、もしsvnではこ
> んなのしか無理だけどどこそこに見に行ったらちゃんと書いてあるよとかでした
> ら、そのあたりもアナウンスしてくださると幸いです。

git の merge コミットメッセージなので、--no-commit してから
人力でコミットメッセージを作れば解決できますが、
負担が大きいのであまりやりたくないです。

github の方の Service-hook を使って期待通りのことができそうなので
そっちの方向で調べてみます。

-- 
SHIBATA Hiroshi mailto:shibata.hiroshi@...
URL:http://www.hsbt.org/