[ruby-list:32797] dir_config (mkmf.rb) のオプション指定の優先順位

たむらです。

mkmf.rb の dir_configですが、--with-<target>-{include,lib}を利用するときに
引数の数で挙動が変わってしまいます。

$ ruby -v
ruby 1.6.5 (2001-09-19) [i686-cygwin]

$ ruby -rmkmf -e 'p dir_config("mysql", "/usr/local")' \
 -- --with-mysql-dir=/bar
["/bar/include", "/bar/lib"]             # 1

$ ruby -rmkmf -e 'p dir_config("mysql", "/usr/local")' \
  -- --with-mysql-include=/bar/include
["/usr/local/include", "/usr/local/lib"] # 2

$ ruby -rmkmf -e 'p dir_config("mysql", "/usr/local/include","/usr/local/lib")' \
  -- --with-mysql-dir=/bar
["/bar/include", "/bar/lib"]             # 3

$ ruby -rmkmf -e 'p dir_config("mysql", "/usr/local/include","/usr/local/lib")' \
  -- --with-mysql-include=/bar/include
["/bar/include", "/usr/local/lib"]       # 4

(2) のパターンは、(4)が望ましいと思われます。

ところで、--with-<target> な引数ですが、指定のパターンとしては
 a) なし
 b) --with-<target>-dir のみ
 c) --with-<target>-include あるいは、--with-<target>-lib のみ
 d) --with-<target>-include と --with-<target>-lib 

というケースをサポートすれば良いと思います。
現在の mkmf.rb では、--with-<target>-dir はきちんと見てますが、
--with-<target>-{include,lib}の扱いがちょっと甘いみたいです。
# dir_config の中で、with_config(target + "-dir", default) が nilでないと
# with_config(target + "-include", default)などの評価をしない

--with-<target>-* の評価の順番を --with-<target>-{include,lib}優先に
しました。 --with-<target>-{include,lib}評価後に、--with-<target>-dir
の設定を反映させます。

後、dir_config が引数2つの時に同じものを補う処理を入れました。

--- /home/tamura/src/CYGWIN/ruby-1.6.5/lib/mkmf.rb      Fri Aug 17 14:04:33 2001
+++ /usr/local/lib/ruby/1.6/mkmf.rb     Wed Dec  5 15:15:19 2001
@@ -333,21 +333,27 @@

 def dir_config(target, idefault=nil, ldefault=nil)
   if idefault && ldefault == nil
-    default = idefault
+    if idefault || ldefault != nil
+      default = idefault || ldefault
+    else
+      default = '/usr/local'
+    end
     idefault = default + "/include"
     ldefault = default + "/lib"
   end

   dir = with_config(target + "-dir", default)

-  idir, ldir = if dir then [
-      dir + "/include",
-      dir + "/lib"
-    ] else [
-      with_config(target + "-include", idefault),
-      with_config(target + "-lib", ldefault)
-    ] end
-
+  idir = with_config(target + "-include")
+  ldir = with_config(target + "-lib")
+  if dir
+    idir = dir + "/include" unless idir
+    ldir = dir + "/lib" unless ldir
+  else
+    idir = idir ? idir : idefault
+    ldir = ldir ? ldir : ldefault
+  end
+
   if idir
     idircflag = "-I" + idir
     $CPPFLAGS += " " + idircflag unless $CPPFLAGS.split.include?(idircflag)

dir_config の初期ディレクトリ指定がない場合に、'/usr/local' を補うのは
マズイかな ?