From: "mame (Yusuke Endoh)" Date: 2013-02-18T21:06:55+09:00 Subject: [ruby-core:52439] [ruby-trunk - Bug #1388] cygwin-1.7, gcc4-4.3, and ruby-1.9. make btest #236 test_io.rb Segmentation fault Issue #1388 has been updated by mame (Yusuke Endoh). Description updated Target version changed from 2.0.0 to next minor ---------------------------------------- Bug #1388: cygwin-1.7, gcc4-4.3, and ruby-1.9. make btest #236 test_io.rb Segmentation fault https://bugs.ruby-lang.org/issues/1388#change-36500 Author: neomjp (neomjp neomjp) Status: Assigned Priority: Low Assignee: nobu (Nobuyoshi Nakada) Category: Target version: next minor ruby -v: ruby 1.9.2dev (2009-04-08 trunk 23198) [i386-cygwin] =begin Cygwin 1.7 is currently under beta testing. It is currently at cygwin-1.7.0-46. If nothing goes overly wrong, the official 1.7.1 is planned to be released in June. http://sourceware.org/ml/cygwin-announce/2009-04/msg00025.html Two issues blocking the release are: 1) Stabilization of gcc-4.3; It is currently at gcc4-4.3.2-2, and several to-do's remain. http://sourceware.org/ml/cygwin/2009-03/msg00378.html http://sourceware.org/ml/cygwin/2009-03/msg00422.html Hopefully it will get ready in gcc4-4.3.2-3. 2) Compilation of all packages using the stable gcc-4.3. This bug report is about making ruby-1.9 ready for these new cygwin-1.7 and gcc-4.3. These are some of the patches required to make ruby trunk get compiled. * eval_intern.h [CYGWIN]: Remove #ifdef __CYGWIN__ for _setjmp() and _longjmp(). Cygwin-1.7 has its own definition in /usr/include/machine/setjmp.h . This is the minimally required patch to make the compilation go through to the end. --- origsrc/ruby-1.9.2-r23198/eval_intern.h 2009-02-22 10:43:59.000000000 +0900 +++ src/ruby-1.9.2-r23198/eval_intern.h 2009-04-18 01:26:41.843750000 +0900 @@ -66,9 +66,6 @@ char *strrchr(const char *, const char); #define ruby_setjmp(env) RUBY_SETJMP(env) #define ruby_longjmp(env,val) RUBY_LONGJMP(env,val) -#ifdef __CYGWIN__ -int _setjmp(), _longjmp(); -#endif #include #include * ruby.c (push_include_cygwin): Use cygwin_conv_path instead of cygwin_conv_to_posix_path which is deprecated in cygwin-1.7. * ruby.c (ruby_init_loadpath_safe): Use cygwin_conv_path instead of cygwin_conv_to_posix_path which is deprecated in cygwin-1.7. --- origsrc/ruby-1.9.2-r23198/ruby.c 2009-03-17 10:29:17.000000000 +0900 +++ src/ruby-1.9.2-r23198/ruby.c 2009-04-18 01:26:41.859375000 +0900 @@ -257,7 +257,8 @@ push_include_cygwin(const char *path, VA p = strncpy(RSTRING_PTR(buf), p, len); } } - if (cygwin_conv_to_posix_path(p, rubylib) == 0) + if (cygwin_conv_path(CCP_WIN_W_TO_POSIX | CCP_RELATIVE, p, rubylib, 1) + == 0) p = rubylib; push_include(p, filter); if (!*s) break; @@ -366,8 +367,10 @@ ruby_init_loadpath_safe(int safe_level) #elif defined __CYGWIN__ { char rubylib[FILENAME_MAX]; - cygwin_conv_to_posix_path(libpath, rubylib); - strncpy(libpath, rubylib, sizeof(libpath)); + if (cygwin_conv_path(CCP_WIN_W_TO_POSIX | CCP_RELATIVE, + libpath, rubylib, 1) + == 0) + strncpy(libpath, rubylib, sizeof(libpath)); } #endif p = strrchr(libpath, '/'); * strftime.c [CYGWIN]: Cygwin defines _timezone, _daylight, *_tzname[2], and tzname with dllimport attribute. But defines daylight and timezone without dllimport attribute. --- origsrc/ruby-1.9.2-r23198/strftime.c 2009-03-17 10:29:17.000000000 +0 900 +++ src/ruby-1.9.2-r23198/strftime.c 2009-04-18 01:26:41.859375000 +0900 @@ -120,12 +120,16 @@ extern char *strchr(); #define range(low, item, hi) max(low, min(item, hi)) -#if defined __WIN32__ || defined _WIN32 +#if defined __CYGWIN__ || defined __WIN32__ || defined _WIN32 #define DLL_IMPORT __declspec(dllimport) #endif #ifndef DLL_IMPORT #define DLL_IMPORT #endif +#ifdef __CYGWIN__ +#define daylight _daylight +#define timezone _timezone +#endif #if !defined(OS2) && defined(HAVE_TZNAME) extern DLL_IMPORT char *tzname[2]; #ifdef HAVE_DAYLIGHT With the above three patches, ruby-1.9.2-r23198 can get compiled with only one warning: ** PTHREAD SUPPORT MODE WARNING: ** ** Ruby is compiled with --enable-pthread, but your Tcl/Tk library ** seems to be compiled without pthread support. Although you can ... This is expected because cygwin tcltk-20080420-1 is compiled without pthread support. But when I try to compile like CC=gcc-4 configure --program-suffix="-19" --disable-pthread make compilation fails. make: *** No rule to make target `thread_.h', needed by `miniprelude.o'. Stop. *** ERROR: make failed This is because THREAD_MODEL is empty in Makefile. Looking into configure.in, I can see that when if test "$rb_with_pthread" = "yes"; is false and case "$target_os" in when(cygwin*) then THREAD_MODEL gets undefined. (when(mingw*) is true, THREAD_MODEL=win32.) If I compile like CC=gcc-4 configure --program-suffix="-19" --disable-pthread make THREAD_MODEL=w32 the compilation goes through to the end, and thread-win32.c seems to be used instead of thread-pthread.c. But the same warning persists. ** PTHREAD SUPPORT MODE WARNING: ** ** Ruby is compiled with --enable-pthread, but your Tcl/Tk library ** seems to be compiled without pthread support. Although you can ... This is wrong because --disable-pthread is used. Looking into ext/tk/extconf.rb, I can see that this warning is emitted when # check pthread mode if (macro_defined?('HAVE_NATIVETHREAD', '#include "ruby.h"')) # ruby -> enable unless tcl_enable_thread # ruby -> enable && tcl -> disable But include/ruby/ruby.h has #define HAVE_NATIVETHREAD without any #ifdefs. So the pthread mode check in ext/tk/extconf.rb always evaluates to be true even when pthread support is disabled. This should be corrected. If these issues are corrected, then ruby-1.9 trunk can get compiled without warnings. When I tried make run or make runruby, it failed. * common.mk (TESTRUN_SCRIPT): Correct the path to test.rb --- origsrc/ruby-1.9.2-r23198/common.mk 2009-04-10 11:32:15.000000000 +0900 +++ src/ruby-1.9.2-r23198/common.mk 2009-04-18 04:35:13.968750000 +0900 @@ -117,7 +117,7 @@ TESTSDIR = $(srcdir)/test TESTWORKDIR = testwork -TESTRUN_SCRIPT = $(srcdir)/test.rb +TESTRUN_SCRIPT = $(srcdir)/sample/test.rb BOOTSTRAPRUBY = $(BASERUBY) With this patch, the results of make run or runruby are make run not ok/test: 900 failed 1 Fnot ok system 9 -- .../ruby-1.9.2-r23198/sample/test.rb:1948:in `
' make runruby end of test(test: 900) which is expected and good. miniruby.exe does not support euc-jp, shift_jis, windows-1251, cp932 in Encoding.name_list, so make run is expected to fail at that test. But the result of make btest is bad. #236 test_io.rb: at_exit { p :foo } megacontent = "abc" * 12345678 #File.open("megasrc", "w") {|f| f << megacontent } Thread.new { sleep rand*0.2; Process.kill(:INT, $$) } r1, w1 = IO.pipe r2, w2 = IO.pipe t1 = Thread.new { w1 << megacontent; w1.close } t2 = Thread.new { r2.read } IO.copy_stream(r1, w2) rescue nil r2.close; w2.close r1.close; w1.close #=> killed by SIGABRT (signal 6) | bootstraptest.tmp.rb:2: [BUG] Segmentation fault | ruby 1.9.2dev (2009-04-15 trunk 23198) [i386-cygwin] | | -- control frame ---------- | c:0004 p:---- s:0010 b:0010 l:000009 d:000009 CFUNC :p | c:0003 p:0011 s:0006 b:0006 l:000aec d:000005 BLOCK bootstraptest.tmp.rb:2 | c:0002 p:---- s:0004 b:0004 l:000003 d:000003 FINISH | c:0001 p:0000 s:0002 b:0002 l:000aec d:000aec TOP
:19 | --------------------------- | bootstraptest.tmp.rb:2:in `block in
' | bootstraptest.tmp.rb:2:in `p' | | [NOTE] | You may have encountered a bug in the Ruby interpreter or extension libraries. | Bug reports are welcome. | For details: http://www.ruby-lang.org/bugreport.html | FAIL 1/890 tests failed make: *** [btest] Error 1 make btest-ruby also emits several errors, but I will submit it as another issue because this report is already too long... =end -- http://bugs.ruby-lang.org/