[ruby-dev:29449] Re: ext/dbm on ruby-1.8.5

なかだです。

At Sun, 3 Sep 2006 23:56:09 +0900,
Takahiro Kambe wrote in [ruby-dev:29445]:
> if /DBM_HDR/ =~ $CFLAGS and have_func(db_prefix("dbm_open"))
> 
> といった $CFLAGS の内容をチェックしていますが、$CFLAGS には DBM_HDRと
> いったパターンは含まれていないようです。これは、

db_checkの結果を使ったほうがいいような気がします。



Index: ext/dbm/extconf.rb
===================================================================
RCS file: /cvs/ruby/src/ruby/ext/dbm/extconf.rb,v
retrieving revision 1.16.2.1
diff -p -U 2 -r1.16.2.1 extconf.rb
--- ext/dbm/extconf.rb	25 May 2006 23:44:05 -0000	1.16.2.1
+++ ext/dbm/extconf.rb	3 Sep 2006 23:11:28 -0000
@@ -47,8 +47,8 @@ end
 
 if dblib
-  db_check(dblib)
+  dbm_hdr = db_check(dblib)
 else
-  for dblib in %w(db db2 db1 dbm gdbm gdbm_compat qdbm)
-    db_check(dblib) and break
+  dbm_hdr = %w(db db2 db1 dbm gdbm gdbm_compat qdbm).any? do |dblib|
+    db_check(dblib)
   end
 end
@@ -56,5 +56,5 @@ end
 have_header("cdefs.h") 
 have_header("sys/cdefs.h") 
-if /DBM_HDR/ =~ $CFLAGS and have_func(db_prefix("dbm_open"))
+if dbm_hdr and have_func(db_prefix("dbm_open"))
   have_func(db_prefix("dbm_clearerr")) unless $dbm_conf_have_gdbm
   create_makefile("dbm")


-- 
--- 僕の前にBugはない。
--- 僕の後ろにBugはできる。
    中田 伸悦