Bug 247943

Summary:	korundum doesn't compile with ruby 1.9.2-rc2
Product:	[Unmaintained] bindings	Reporter:	Stefano Crocco <stefano.crocco>
Component:	general	Assignee:	bindings-bugs-null
Status:	RESOLVED FIXED
Severity:	normal	CC:	scarpino
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	Gentoo Packages
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:	4.5.3
Sentry Crash Report:

Description Stefano Crocco 2010-08-15 12:06:57 UTC

Version:           unspecified (using KDE 4.5.0) 
OS:                Linux

Compiling korundum with ruby 1.9.2-rc2 fails in korundum/src/krubypluginfactory.cpp because of the use of the STR2CSTR macro which has been removed. The macro is used four times and only in the show_exception_message function.

To make korundum compile, I did the following:
* replaced the first and the last two calls (lines 56, 58 and 64) with StringValueCStr (which I saw used elsewhere in the sources)
* replaced the second call to STR2CSTR (line 57) with RSTRING_PTR (using StringValueCStr here didn't work because gcc complained about needing an lvalue

I don't know whether the changes I made will cause troubles when running korundum application, as I just finished installing it and haven't tried it as yet

Reproducible: Always

Comment 1 Andrea Scarpino 2011-05-25 11:19:51 UTC

I built korundum from git with ruby 1.9.2_p180

Comment 2 Stefano Crocco 2011-05-25 16:30:31 UTC

You're right. I have been compiling korundum with ruby 1.9.2 myself for a long time without problems. I had forgotten about this bug. It's been resolved (and has been so since version 4.5.3.