Project

General

Profile

Actions

Bug #4937

closed

trunk@32244 MinGW build failure - thread_win32.c

Added by jonforums (Jon Forums) almost 13 years ago. Updated over 12 years ago.

Status:
Closed
Target version:
ruby -v:
ruby 1.9.3dev (2011-06-27 trunk 32242) [i386-mingw32]
Backport:
[ruby-core:37575]

Description

The signature of native_stop_timer_thread at

https://github.com/ruby/ruby/blob/trunk/thread_win32.c#L748

conflicts with it's usage by rb_thread_stop_timer_thread at

https://github.com/ruby/ruby/blob/trunk/thread.c#L2989

resulting in the following build error when using the RubyInstaller build recipes and gcc.exe (tdm-1) 4.5.2:

compiling ../../../../Users/Jon/Documents/RubyDev/ruby-git/thread.c
../../../../Users/Jon/Documents/RubyDev/ruby-git/thread.c: In function 'rb_thread_stop_timer_thread':
../../../../Users/Jon/Documents/RubyDev/ruby-git/thread.c:2989:5: error: too many arguments to function 'native_stop_timer_thread'
../../../../Users/Jon/Documents/RubyDev/ruby-git/thread_win32.c:748:1: note: declared here

Updated by luislavena (Luis Lavena) almost 13 years ago

  • Category set to build
  • Assignee set to ko1 (Koichi Sasada)

This was introduced in r32244 by Koichi Sasada. Assigning it to him.

Sasada-san, seems the signature change in thread_win32.c is missing, the following patch solves the compilation but I'm unsure the extend of the needed changes:

diff --git a/thread_win32.c b/thread_win32.c
index 5c20650..68fb346 100644
--- a/thread_win32.c
+++ b/thread_win32.c
@@ -745,7 +745,7 @@ rb_thread_create_timer_thread(void)
}

static int
-native_stop_timer_thread(void)
+native_stop_timer_thread(int close_anyway)
{
int stopped = --system_working <= 0;
if (stopped) {

Updated by nahi (Hiroshi Nakamura) almost 13 years ago

  • Target version set to 1.9.3

Updated by jonforums (Jon Forums) over 12 years ago

fixed by trunk@32268 with make test and make test-all TESTS='openssl fiddle psych' all passing

Updated by usa (Usaku NAKAMURA) over 12 years ago

  • Status changed from Open to Closed
Actions

Also available in: Atom PDF

Like0
Like0Like0Like0Like0