Bug 247943 - korundum doesn't compile with ruby 1.9.2-rc2
Summary: korundum doesn't compile with ruby 1.9.2-rc2
Status: RESOLVED FIXED
Alias: None
Product: bindings
Classification: Developer tools
Component: general (show other bugs)
Version: unspecified
Platform: Gentoo Packages Linux
: NOR normal
Target Milestone: ---
Assignee: kde-bindings
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-08-15 12:06 UTC by Stefano Crocco
Modified: 2011-05-25 16:31 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In: 4.5.3


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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.