[ruby-dev:9921] Re: Forward: Error in NT makefile (PR#7)

小松です。

[ruby-dev:9918]が配送されなかったみたいなので再送します。

In the message of [ruby-dev:9900] Re: Forward: Error in NT makefile (PR#7)
    on 2000/5/29 22:49:43 Katsuyuki Komatsu <[e-mail]> wrote:
||そうじゃなくて失敗するときは 2 回 try するからなんですね.
||うーむ. 単に symbol のチェックなんだから関数のポインタだけの
||確認だけでいい気がしますね.
|
|ext/extmk.rb.ntがあったころに、[ruby-dev:7430]のように1回のみtry
|だったのを、ruby-gtkのために[ruby-dev:7452]で2回tryにしたらしい
|のですが、詳細を忘れてしまいました。

関数のポインタだけのチェックに変更してみましたが、
特に問題なさそうです。

# 互換性のために1回目のtryを残しておいたというのが真相らしい。

この枝、mingwの場合も通りますが問題ないかな？

それから、-Oxを-Ziとかに変更している場合のために、
lib/mkmf.rbで$(TARGET).ilkと*.pdbを消すようにしました。

Tue May 30 23:33:41 2000  Katsuyuki Komatsu  <[e-mail]>

	* ext/extmk.rb.in (have_library, have_func): remove unnecessary
	  try_link() call from the mswin32 platform branch.

	* lib/mkmf.rb (have_library, have_func): ditto.

	* lib/mkmf.rb (create_makefile): add $(TARGET).ilk and *.pdb
	  to cleanup files for mswin32.

diff -ru1p ruby-1.5.4.dist/ext/extmk.rb.in ruby-1.5.4/ext/extmk.rb.in
--- ruby-1.5.4.dist/ext/extmk.rb.in	Mon May 29 11:10:21 2000
+++ ruby-1.5.4/ext/extmk.rb.in	Tue May 30 23:33:41 2000
@@ -176,12 +176,4 @@ def have_library(lib, func="main")
 int main() { return 0; }
-int t() { #{func}(); return 0; }
-SRC
-      unless r
-        r = try_link(<<"SRC", libs)
-#include <windows.h>
-#include <winsock.h>
-int main() { return 0; }
 int t() { void ((*p)()); p = (void ((*)()))#{func}; return 0; }
 SRC
-      end
     else
@@ -228,12 +220,4 @@ def have_func(func)
 int main() { return 0; }
-int t() { #{func}(); return 0; }
-SRC
-    unless r
-      r = try_link(<<"SRC", libs)
-#include <windows.h>
-#include <winsock.h>
-int main() { return 0; }
 int t() { void ((*p)()); p = (void ((*)()))#{func}; return 0; }
 SRC
-    end
   else
diff -ru1p ruby-1.5.4.dist/lib/mkmf.rb ruby-1.5.4/lib/mkmf.rb
--- ruby-1.5.4.dist/lib/mkmf.rb	Mon May 29 11:10:22 2000
+++ ruby-1.5.4/lib/mkmf.rb	Tue May 30 23:33:41 2000
@@ -161,12 +161,4 @@ def have_library(lib, func="main")
 int main() { return 0; }
-int t() { #{func}(); return 0; }
-SRC
-      unless r
-        r = try_link(<<"SRC", libs)
-#include <windows.h>
-#include <winsock.h>
-int main() { return 0; }
 int t() { void ((*p)()); p = (void ((*)()))#{func}; return 0; }
 SRC
-      end
     else
@@ -223,12 +215,4 @@ def have_func(func)
 int main() { return 0; }
-int t() { #{func}(); return 0; }
-SRC
-    unless r
-      r = try_link(<<"SRC", libs)
-#include <windows.h>
-#include <winsock.h>
-int main() { return 0; }
 int t() { void ((*p)()); p = (void ((*)()))#{func}; return 0; }
 SRC
-    end
   else
@@ -407,3 +391,3 @@ all:		$(DLLIB)
 clean:;		@$(RM) *.#{$OBJEXT} *.so *.sl *.a $(DLLIB)
-		@$(RM) $(TARGET).lib $(TARGET).exp
+		@$(RM) $(TARGET).lib $(TARGET).exp $(TARGET).ilk *.pdb
 		@$(RM) Makefile extconf.h conftest.*

--
小松克行 / Katsuyuki Komatsu <[e-mail]>