アカウント名:
パスワード:
問題のパッチは、56ファイル13万行で4.6MBありますが、日立のファイバチャネルボード用のドライバ(hfcldd)を新規追加するもののようです。基本的に「複数のパッチに分割」なんてできそうにないと思いますが、いったいどうしろというのでしょうか?…
対応機種が多いのか、hfcl_detect.c と hfcl_detect_fx.c で3万行取られてるのが大きい感じですので、「対応機種の少ないhfclddドライバを新規追加」と「hfclddに対して対応機種を増やすパッチ」という形なら分割できそうですが…無駄に手間をかけるだけだよなぁ。
対応機種が多いために冗長なソースになっているなら、最初にプロトタイプとして単一機種対応の簡易版を投稿して、-nextあたりにマージしてもらって、対応機種ごとに、機能追加ごとにパッチを追加していくのが本筋かと。いきなり社内で開発終わった巨大なパッチをひとまとめに投稿しようというのが、そもそもの誤り。そんな巨大なものだイブの人がレビューできるわけないじゃないか。いくら社内で開発終わってて問題ないから入れてくれったって、そんなの通用しないよ。
より多くのコメントがこの議論にあるかもしれませんが、JavaScriptが有効ではない環境を使用している場合、クラシックなコメントシステム(D1)に設定を変更する必要があります。
最初のバージョンは常に打ち捨てられる。
「論理的に複数のパッチに分割」できない時はどうするの? (スコア:3, 参考になる)
問題のパッチは、56ファイル13万行で4.6MBありますが、
日立のファイバチャネルボード用のドライバ(hfcldd)を新規追加するもののようです。
基本的に「複数のパッチに分割」なんてできそうにないと思いますが、いったいどうしろというのでしょうか?…
対応機種が多いのか、hfcl_detect.c と hfcl_detect_fx.c で3万行取られてるのが大きい感じですので、
「対応機種の少ないhfclddドライバを新規追加」と「hfclddに対して対応機種を増やすパッチ」という形なら分割できそうですが…無駄に手間をかけるだけだよなぁ。
Re:「論理的に複数のパッチに分割」できない時はどうするの? (スコア:3, すばらしい洞察)
対応機種が多いために冗長なソースになっているなら、最初にプロトタイプとして単一機種対応の簡易版を投稿して、-nextあたりにマージしてもらって、対応機種ごとに、機能追加ごとにパッチを追加していくのが本筋かと。
いきなり社内で開発終わった巨大なパッチをひとまとめに投稿しようというのが、そもそもの誤り。
そんな巨大なものだイブの人がレビューできるわけないじゃないか。いくら社内で開発終わってて問題ないから入れてくれったって、そんなの通用しないよ。