Bug #4937

trunk@32244 MinGW build failure - thread_win32.c

Added by Jon Forums almost 3 years ago. Updated almost 3 years ago.

[ruby-core:37575]
Status:Closed
Priority:Normal
Assignee:Koichi Sasada
Category:build
Target version:1.9.3
ruby -v:ruby 1.9.3dev (2011-06-27 trunk 32242) [i386-mingw32] Backport:

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 'rbthreadstoptimerthread':
../../../../Users/Jon/Documents/RubyDev/ruby-git/thread.c:2989:5: error: too many arguments to function 'nativestoptimerthread'
../../../../Users/Jon/Documents/RubyDev/ruby-git/thread
win32.c:748:1: note: declared here

History

#1 Updated by Luis Lavena almost 3 years ago

  • Category set to build
  • Assignee set to 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/threadwin32.c b/threadwin32.c
index 5c20650..68fb346 100644
--- a/threadwin32.c
+++ b/thread
win32.c
@@ -745,7 +745,7 @@ rbthreadcreatetimerthread(void)
}

static int
-nativestoptimerthread(void)
+native
stoptimerthread(int closeanyway)
{
int stopped = --system
working <= 0;
if (stopped) {

#2 Updated by Hiroshi Nakamura almost 3 years ago

  • Target version set to 1.9.3

#3 Updated by Jon Forums almost 3 years ago

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

#4 Updated by Usaku NAKAMURA almost 3 years ago

  • Status changed from Open to Closed

Also available in: Atom PDF