[ruby-dev:19501] Re: cpp in mkmf (Re: Problems building mysql ruby 2.4.4 on 1.8)

なかだです。

At Wed, 5 Feb 2003 13:05:24 +0900,
WATANABE Hirofumi wrote:
> > [ruby-talk:63715]の件ですが、CPPではなくてcpp_commandを使わなく
> > てはならないというのもありますが、
> 
> そうか。1.6と共通のextconf.rbを書くのが難しくなってしまった
> わけですね。やはりCPPとかCFLAGSをなくしたのはまずかったかな。
> 互換性のためにも定義したほうがよさそうですね。

%sの数とか位置があってるかどうかよく分かりませんが。

> > Borlandのcpp32は出力ファイル
> > 名が限定されているため、outfile引数は実際には意味がありません。
(snip)
> だから限定ではないと思います。bcc32/Makefile.subでの
> CPPOUTFILEの定義が間違っている感じではありますが。

stdoutに送れない、の間違いでした。

> > 出力ファイルも固定にしてしまったほうがいいような気がします。
> 
> に関しては賛成です。

限定されてないんなら、別に固定にすることもないかも?


Index: configure.in
===================================================================
RCS file: /cvs/ruby/src/ruby/configure.in,v
retrieving revision 1.166
diff -u -2 -p -r1.166 configure.in
--- configure.in	20 Jan 2003 12:51:50 -0000	1.166
+++ configure.in	5 Feb 2003 04:29:57 -0000
@@ -32,4 +32,5 @@ elif test -n "$rb_cv_cppoutfile"; then
   CPPOUTFILE="$rb_cv_cppoutfile"
 fi
+CPPOUTFILE="`echo \"$CPPOUTFILE\" | sed 's/conftest\.i/%s/'`"
 AC_SUBST(CPPOUTFILE)])
 
Index: bcc32/Makefile.sub
===================================================================
RCS file: /cvs/ruby/src/ruby/bcc32/Makefile.sub,v
retrieving revision 1.23
diff -u -2 -p -r1.23 Makefile.sub
--- bcc32/Makefile.sub	25 Jan 2003 18:59:34 -0000	1.23
+++ bcc32/Makefile.sub	5 Feb 2003 04:33:21 -0000
@@ -368,5 +368,5 @@ s,@DLDLIBS@,$(DLDLIBS),;t t
 s,@ENABLE_SHARED@,yes,;t t
 s,@OUTFLAG@,-o,;t t
-s,@CPPOUTFILE@,,;t t
+s,@CPPOUTFILE@,-o%s,;t t
 s,@LIBPATHFLAG@, -L%s,;t t
 s,@LIBARG@,%s.lib,;t t
Index: win32/Makefile.sub
===================================================================
RCS file: /cvs/ruby/src/ruby/win32/Makefile.sub,v
retrieving revision 1.43
diff -u -2 -p -r1.43 Makefile.sub
--- win32/Makefile.sub	25 Jan 2003 18:59:34 -0000	1.43
+++ win32/Makefile.sub	5 Feb 2003 04:34:34 -0000
@@ -363,5 +363,5 @@ s,@LIBRUBYARG@,$$(LIBRUBYARG_SHARED),;t 
 s,@ENABLE_SHARED@,yes,;t t
 s,@OUTFLAG@,-Fe,;t t
-s,@CPPOUTFILE@,-P,;t t
+s,@CPPOUTFILE@,-E > %s,;t t
 s,@LIBPATHFLAG@, -libpath:%s,;t t
 s,@LIBARG@,%s.lib,;t t
Index: lib/mkmf.rb
===================================================================
RCS file: /cvs/ruby/src/ruby/lib/mkmf.rb,v
retrieving revision 1.125
diff -u -2 -p -r1.125 mkmf.rb
--- lib/mkmf.rb	21 Jan 2003 19:33:19 -0000	1.125
+++ lib/mkmf.rb	5 Feb 2003 05:21:29 -0000
@@ -88,4 +88,5 @@ CPPOUTFILE = CONFIG['CPPOUTFILE']
 
 CONFTEST_C = "conftest.c"
+CONFTEST_I = "conftest.i"
 
 $INSTALLFILES ||= nil
@@ -222,5 +223,5 @@ end
 def cpp_command(outfile, opt="")
   "$(CPP) #$INCFLAGS -I#{$hdrdir} " \
-  "#$CPPFLAGS #$CFLAGS #{outfile} #{opt} #{CONFTEST_C}"
+  "#$CPPFLAGS #$CFLAGS #{CPPOUTFILE % outfile} #{opt} #{CONFTEST_C}"
 end
 
@@ -246,5 +247,5 @@ end
 
 def try_cpp(src, opt="")
-  try_do(src, cpp_command(CPPOUTFILE, opt))
+  try_do(src, cpp_command(CONFTEST_I, opt))
 ensure
   rm_f "conftest*"
@@ -278,22 +279,13 @@ def egrep_cpp(pat, src, opt="")
   src += "\n" unless /\n\z/ =~ src
   create_tmpsrc(src)
-  xpopen(cpp_command('', opt)) do |f|
-    if Regexp === pat
-      puts("    ruby -ne 'print if #{pat.inspect}'")
-      f.grep(pat) {|l|
-	puts "#{f.lineno}: #{l}"
-	return true
-      }
-      false
-    else
-      puts("    egrep '#{pat}'")
-      begin
-	stdin = $stdin.dup
-	$stdin.reopen(f)
-	system("egrep", pat)
-      ensure
-	$stdin.reopen(stdin)
-      end
-    end
+  return false unless xsystem(cpp_command(CONFTEST_I, opt))
+  pat = Regexp.new(pat) unless Regexp === pat
+  open(CONFTEST_I) do |f|
+    puts("ruby -ne 'print if #{pat.inspect}' #{CONFTEST_I}")
+    f.grep(pat) {|l|
+      puts "#{f.lineno}: #{l}"
+      return true
+    }
+    false
   end
 ensure
@@ -878,2 +870,21 @@ distclean:	clean
 realclean:	distclean
 "
+
+# for backward compatibility
+CFLAGS = $CFLAGS
+CPPFLAGS = $CPPFLAGS
+LINK = Config::expand(TRY_LINK.dup,
+		      'hdrdir' => $hdrdir,
+		      'src' => CONFTEST_C,
+		      'INCFLAGS' => $INCFLAGS,
+		      'CPPFLAGS' => "%s",
+		      'CFLAGS' => "#{CFLAGS} %s",
+		      'LDFLAGS' => "#$LDFLAGS %s",
+		      'LIBPATH' => "",
+		      'LOCAL_LIBS' => "%s",
+		      'LIBS' => "#$LIBRUBYARG_STATIC %s #$LIBS")
+CC = Config::expand(COMPILE_C.sub(/\$</, CONFTEST_C),
+		    'CPPFLAGS' => "#{CPPFLAGS} %s -I#{$hdrdir}",
+		    'CFLAGS' => "#{CFLAGS} %s %s")
+CPP = Config::expand("$(CPP) #$INCFLAGS -I#{$hdrdir} " \
+		     "%s %s #{CPPOUTFILE % CONFTEST_I} %s #{CONFTEST_C}")


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