[seasar-dev:1270] Re: Seasar2のGitリポジトリ

2013年 10月 16日 (水) 20:38:41 JST

中村(taedium)です。

マージしてみました。

> あ、よく見たら#1と#2はフォークからじゃなくて
> seasarorg/seasar2の別ブランチからですね。

そうなんです。
そっちのほうがお手軽っていう話を何かで読んで、
seasarorg/seasar2の別ブランチからプルリクエストしてみました。

> フォークは大げさ？

自分としては、どちらでもOKです。
（やはりこれについてもどちらがいいか判断しづらいので）

2013年10月16日 19:05 Koichi Kobayashi <koichik ＠ improvement.jp>:
> 小林 (koichik) です。
>
> あ、よく見たら#1と#2はフォークからじゃなくて
> seasarorg/seasar2の別ブランチからですね。
>
> フォークは大げさ？
>
>
> On Wed, 16 Oct 2013 19:02:43 +0900, Koichi Kobayashi <koichik ＠ improvement.jp> wrote:
>
>> 小林 (koichik) です。
>>
>> では、seasarorg/seasar2へ直接コミットして
>> プッシュは避けて、自分のフォークからプルリク、
>> 他のコミッタからLGTMもらったら (リベースではなく)
>> マージするということでやってみましょう。
>>
>> #1と#2のマージよろしくです＞中村さん
>>
>>
>> On Wed, 16 Oct 2013 18:33:37 +0900, Toshihiro Nakamura <toshihiro.nakamura ＠ gmail.com> wrote:
>>
>> > 中村(taedium)です。
>> >
>> > > maven-s2jdbc-gen-pluginは再作成の時、一緒に
>> > > 除外すればよかったですね。無念だ。
>> >
>> > 事前に気づけずにすみません。
>> >
>> > > あと、マージするかリベースするか？
>> > > Nodeはリベースだけど、GitHubでは明らかに
>> > > マージが主流な感じ。
>> > > Domaはマージしてるようですね。
>> > >
>> > > どうしましょう？
>> >
>> > ぜんぜんGitに慣れていなくて、どちらがいいのか自分の中で判断基準がありません。
>> > ですので、おまかせしたいです。
>> >
>> > 2013年10月16日 18:00 Koichi Kobayashi <koichik ＠ improvement.jp>:
>> > > 小林 (koichik) です。
>> > >
>> > > プルリクエスト乙でーす。
>> > > maven-s2jdbc-gen-pluginは再作成の時、一緒に
>> > > 除外すればよかったですね。無念だ。
>> > >
>> > > さて、GitHubどうやって運用していきます？
>> > >
>> > > Nodeなんかはコミッタであっても直接本家の
>> > > リポジトリにはコミットしないで、まずはPR
>> > > して他のコミッタからLGTMもらったらマージ
>> > > するという流れになってます。
>> > > 同じようにします？
>> > >
>> > > あと、マージするかリベースするか？
>> > > Nodeはリベースだけど、GitHubでは明らかに
>> > > マージが主流な感じ。
>> > > Domaはマージしてるようですね。
>> > >
>> > > どうしましょう？
>> > >
>> > >
>> > > On Tue, 15 Oct 2013 09:32:23 +0900, Toshihiro Nakamura <nakamura.toshihiro ＠ isid.co.jp> wrote:
>> > >
>> > >> 中村(taedium)です。
>> > >>
>> > >> あ、解決していたのですね。
>> > >> つい、こちらのメールを読まずに返信してしまいました。。。
>> > >>
>> > >> tools.jarの登録は「ハマリ」ポイントなので仕方ないですよ。
>> > >>
>> > >> On Sat, 12 Oct 2013 20:43:50 +0900
>> > >> Koichi Kobayashi <koichik ＠ improvement.jp> wrote:
>> > >>
>> > >> > 小林 (koichik) です。
>> > >> >
>> > >> > > s2jdbc-genがjavadocのパッケージを参照していて
>> > >> >
>> > >> > 自己解決しました。
>> > >> > 昔のこと過ぎてすっかり忘れてましたが、Installed
>> > >> > JREのところでtools.jarを追加していたのですね。
>> > >> > あーもう忘れすぎてて辛い。。。
>> > >> >
>> > >> >
>> > >> > On Sat, 12 Oct 2013 18:40:14 +0900, Koichi Kobayashi <koichik ＠ improvement.jp> wrote:
>> > >> >
>> > >> > > 小林 (koichik) です．
>> > >> > >
>> > >> > > Gitに移行したSeasar2リポジトリですが、
>> > >> > > S2JDBC-Gen関連のプロジェクトがs2jdbc-gen/
>> > >> > > ディレクトリの下にあって、pom.xmlの期待する
>> > >> > > 構成と合ってなかったので一階層上げました。
>> > >> > >
>> > >> > > 油断してましたが、とっくにメンテされてない
>> > >> > > seasar2-exampleを除外してなかったのは失敗
>> > >> > > だったかも。
>> > >> > > 意図して持ってきたわけじゃなくて、先頭がseasar2と
>> > >> > > マッチしたので除外されなかったという。。。
>> > >> > > 今ならまだ作り直しもあり？
>> > >> > >
>> > >> > >
>> > >> > > 主に中村さん:
>> > >> > >
>> > >> > > s2jdbc-genがjavadocのパッケージを参照していて
>> > >> > > .classpathにはそれ用の設定が入っているのですが、
>> > >> > > Eclipse4.3だと(？)見えなくてエラーになります。
>> > >> > > 対処方法知りませんか？
>> > >> > >
>> > >> > > もうひとつ。
>> > >> > > ちょっと今更ですが、S2-Tigerのテストケース
>> > >> > >
>> > >> > > org.seasar.framework.unit.Seasar2Test
>> > >> > >
>> > >> > > ですが、これが使ってるDDL
>> > >> > >
>> > >> > > s2-tiger/src/test/resources/data/demo.script
>> > >> > >
>> > >> > > ではEMPテーブルの主キーEMPNOはNUMERIC(4)なのに、
>> > >> > > データを投入する
>> > >> > >
>> > >> > > s2-tiger/src/test/resources/org/seasar/framework/unit/Seasar2Test$AutoPreparingTest.xls
>> > >> > >
>> > >> > > などではEMPNOに5桁の値が使われています。
>> > >> > > ちょっと見直してもらっていいですか？
>> > >> > >
>> > >> > >
>> > >> > > --
>> > >> > > {
>> > >> > >   name: "Koichi Kobayashi",
>> > >> > >   mail: "koichik ＠ improvement.jp",
>> > >> > >   blog: "http://d.hatena.ne.jp/koichik/",
>> > >> > >   twitter: "@koichik"
>> > >> > > }
>> > >> > >
>> > >> > >
>> > >> > > _______________________________________________
>> > >> > > Seasar-dev mailing list
>> > >> > > Seasar-dev ＠ ml.seasar.org
>> > >> > > https://ml.seasar.org/mailman/listinfo/seasar-dev
>> > >> >
>> > >> >
>> > >> > --
>> > >> > {
>> > >> >   name: "Koichi Kobayashi",
>> > >> >   mail: "koichik ＠ improvement.jp",
>> > >> >   blog: "http://d.hatena.ne.jp/koichik/",
>> > >> >   twitter: "@koichik"
>> > >> > }
>> > >> >
>> > >> > _______________________________________________
>> > >> > Seasar-dev mailing list
>> > >> > Seasar-dev ＠ ml.seasar.org
>> > >> > https://ml.seasar.org/mailman/listinfo/seasar-dev
>> > >>
>> > >> --
>> > >> Toshihiro Nakamura <nakamura.toshihiro ＠ isid.co.jp>
>> > >>
>> > >> _______________________________________________
>> > >> Seasar-dev mailing list
>> > >> Seasar-dev ＠ ml.seasar.org
>> > >> https://ml.seasar.org/mailman/listinfo/seasar-dev
>> > >
>> > >
>> > > --
>> > > {
>> > >   name: "Koichi Kobayashi",
>> > >   mail: "koichik ＠ improvement.jp",
>> > >   blog: "http://d.hatena.ne.jp/koichik/",
>> > >   twitter: "@koichik"
>> > > }
>> > >
>> > > _______________________________________________
>> > > Seasar-dev mailing list
>> > > Seasar-dev ＠ ml.seasar.org
>> > > https://ml.seasar.org/mailman/listinfo/seasar-dev
>> >
>> >
>> >
>> > --
>> > Toshihiro Nakamura
>> > _______________________________________________
>> > Seasar-dev mailing list
>> > Seasar-dev ＠ ml.seasar.org
>> > https://ml.seasar.org/mailman/listinfo/seasar-dev
>>
>>
>> --
>> {
>>   name: "Koichi Kobayashi",
>>   mail: "koichik ＠ improvement.jp",
>>   blog: "http://d.hatena.ne.jp/koichik/",
>>   twitter: "@koichik"
>> }
>>
>> _______________________________________________
>> Seasar-dev mailing list
>> Seasar-dev ＠ ml.seasar.org
>> https://ml.seasar.org/mailman/listinfo/seasar-dev
>
>
> --
> {
>   name: "Koichi Kobayashi",
>   mail: "koichik ＠ improvement.jp",
>   blog: "http://d.hatena.ne.jp/koichik/",
>   twitter: "@koichik"
> }
>
> _______________________________________________
> Seasar-dev mailing list
> Seasar-dev ＠ ml.seasar.org
> https://ml.seasar.org/mailman/listinfo/seasar-dev


-- 
Toshihiro Nakamura