=begin
遠藤です。

ext/dbのextconf.rbに、様々なバージョンのBerkley DBを認識させるためのパッチです。

確認ですが、バージョン 3 から 5 を認識させるようになったのですよね。
実際にそれぞれのバージョンでビルドして、test/dbm/ がパスすることを
確認して頂いたということでしょうか。
そういうことであれば、神戸さんを信じて取り込もうと思います。

あと、

• dblib = %w(db db2 db1 db5 db4 db3 dbm gdbm gdbm_compat qdbm)

という不思議な順番でチェックすることに意味はあるでしょうか。

--
Yusuke Endoh [email protected]
=end

=begin

確認ですが、バージョン 3 から 5 を認識させるようになったのですよね。
そうです。

実際にそれぞれのバージョンでビルドして、test/dbm/ がパスすることを
確認して頂いたということでしょうか。
テストまではしていません。ただ、dbmの互換ライブラリとして各バージョンのBerkeley DBを使う分に、
それほど問題が出てくるとも思えません。

そういうことであれば、神戸さんを信じて取り込もうと思います。
と、いうわけで信じてはダメです。;-)

• dblib = %w(db db2 db1 db5 db4 db3 dbm gdbm gdbm_compat qdbm)

という不思議な順番でチェックすることに意味はあるでしょうか。
新しいリリースについては新しいものを優先、でもベースシステムに存在しているdbがあればそれで済ませたい、といった程度の意図ですが、db2やdb1の辺りは元を尊重しただけなので深い意味はありません。

=end

=begin

Berkeley DB は API の互換性とかにちゃんと気を使っている立派な
プロジェクトなんですかね。OpenSSL とか Ruby とか見てるとどうも
疑い深くなってしまいます。
Berkley DBとしては4.6と4.7で非互換があって、/usr/pkgsrc/databased/db46 とかが存在します。

あくまでも「dbm互換の部分」だけに限れば、dbm自体がそんなに非互換が出てくるところは考えにくく、
同一のバージョンのBerklery DBを相手にしている限りは問題は出てこないと思うわけです。

一方で、Rubyの行うテストで出てこないであろう部分としては、実際に作成した foo.db のファイルの
内容が異なるBerkley DBの間で互換があるのかという部分もあるでしょう。

=end

2011年11月12日8:14 Tanaka Akira [email protected]:

ここの部分ですが、have_type("dbm", ["db.h", h], hsearch)} を
加えているのはなんででしょう?

探したところ、pkgsrc の

Both db.h and ndbm.1 must be included to use DBM. Use both headers
in extconf.rb to check the availability of DBM. This makes the package
build on Darwin.

http://pkgsrc.se/[email protected]

というのが元のようで、Darwin では db.h と ndbm.h の両方を include しないと
DBM が使えないという話のようです。

しかし、どうもこれは信じ難いです。
このパッチは extconf.rb で db.h を使って DBM という型を確認するものですが、
dbm.c では db.h を include するようにはなっていません。
そして、それでも dbm がコンパイルできる以上 DBM という型は使えているはずです。

この部分を外して現在の Darwin でも問題ないようなら、
外してしまいたいんですが、どうですかね。
どなたか試していただけないでしょうか。

1.9.3 なら以下のパッチで外れます。

% svn diff --diff-cmd diff -x '-u -p'
Index: ext/dbm/extconf.rb¶

--- ext/dbm/extconf.rb (revision 33774)
+++ ext/dbm/extconf.rb (working copy)
@@ -39,8 +39,7 @@ def headers.db_check(db)
db_prefix ||= ""

if (have_library(db, db_prefix+"dbm_open") ||

have_func(db_prefix+"dbm_open")) and

•  hdr = self.fetch(db, ["ndbm.h"]).find {|h| have_type("DBM", h,
  

hsearch)} or

•  hdr = self.fetch(db, ["ndbm.h"]).find {|h| have_type("DBM",
  

["db.h", h], hsearch)}

•  hdr = self.fetch(db, ["ndbm.h"]).find {|h| have_type("DBM", h, hsearch)}
  

  have_func(db_prefix+"dbm_clearerr") unless have_gdbm
  $defs << hsearch if hsearch
  $defs << '-DDBM_HDR="<'+hdr+'>"'

trunk はいろいろと変わっていますが、以下のパッチで相当する部分を外せます。

% svn diff --diff-cmd diff -x '-u -p'
Index: ext/dbm/extconf.rb¶

--- ext/dbm/extconf.rb (revision 33774)
+++ ext/dbm/extconf.rb (working copy)
@@ -50,7 +50,7 @@ def headers.db_check2(db, hdr)
have_library("gdbm") or return false
end

• if (have_type("DBM", hdr, hsearch) || have_type("DBM", ["db.h",
  hdr], hsearch)) and

• if have_type("DBM", hdr, hsearch) and
  (db == 'libc' ? have_func('dbm_open("", 0, 0)', hdr, hsearch) :
  have_library(db, 'dbm_open("", 0, 0)', hdr, hsearch)) and
  have_func('dbm_clearerr((DBM *)0)', hdr, hsearch)
  --
  [田中 哲][たなか あきら][Tanaka Akira]

2011年11月16日9:55 Tanaka Akira [email protected]:

2011年11月12日8:14 Tanaka Akira [email protected]:

ここの部分ですが、have_type("dbm", ["db.h", h], hsearch)} を
加えているのはなんででしょう?

探したところ、pkgsrc の

Both db.h and ndbm.1 must be included to use DBM. Use both headers
in extconf.rb to check the availability of DBM. This makes the package
build on Darwin.

http://pkgsrc.se/[email protected]

というのが元のようで、Darwin では db.h と ndbm.h の両方を include しないと
DBM が使えないという話のようです。

しかし、どうもこれは信じ難いです。
このパッチは extconf.rb で db.h を使って DBM という型を確認するものですが、
dbm.c では db.h を include するようにはなっていません。
そして、それでも dbm がコンパイルできる以上 DBM という型は使えているはずです。

この部分を外して現在の Darwin でも問題ないようなら、
外してしまいたいんですが、どうですかね。
どなたか試していただけないでしょうか。

Lionでは問題なく通るようです。もういれちゃって、誰かが
ぎゃっと言ってから考えるでいいんじゃないでしょうか。
2.0でるころにはSnow Leopard サポートしなくても怒られない予感がするし

2011年11月17日12:17 KOSAKI Motohiro [email protected]:

Lionでは問題なく通るようです。もういれちゃって、誰かが
ぎゃっと言ってから考えるでいいんじゃないでしょうか。
2.0でるころにはSnow Leopard サポートしなくても怒られない予感がするし

では trunk には入れることにしますか。¶

[田中 哲][たなか あきら][Tanaka Akira]

あいざわです

Lionでは問題なく通るようです。もういれちゃって、誰かが
ぎゃっと言ってから考えるでいいんじゃないでしょうか。
2.0でるころにはSnow Leopard サポートしなくても怒られない予感がするし

SnowLeopardでも問題なく動くことを確認しました。