Am Dienstag, 7. Dezember 2010, 04:43:10 schrieb Jeff Johnson:
> Talk to Per Oyvind if its the rpm-5_3 you want to change. rpm-5_4 is the
> same as HEAD, including your changes from this weekend.

Per, do I have your permission? Or, more appropriately, do you want to merge 
the corresponding changes from HEAD? Basically, it's about the following 
patch:

---%<---
RCS file: /v/rpm/cvs/rpm/configure.ac,v
retrieving revision 2.470
diff -p -u -w -r2.470 configure.ac
--- configure.ac        2 Nov 2010 15:57:10 -0000       2.470
+++ configure.ac        7 Dec 2010 14:52:36 -0000
@@ -1517,8 +1517,17 @@ RPM_CHECK_LIB(
     [Ruby], [ruby],
     [ruby], [ruby_init], [ruby.h],
     [no,external:none], [],
-    [ AC_DEFINE(WITH_RUBYEMBED, 1, [Define to 1 if you want embedded Ruby])
+    [
+        AC_DEFINE(WITH_RUBYEMBED, 1, [Define to 1 if you want embedded Ruby])
       WITH_RUBY_SUBDIR=ruby
+
+        WITH_RUBY_CPPFLAGS=["-I`ruby -rrbconfig -e \
+                'puts RbConfig::expand(RbConfig::CONFIG[\"rubyhdrdir\"])'` \
+            -I`ruby -rrbconfig -e \
+                 'puts 
RbConfig::expand(\"#{RbConfig::CONFIG[\"rubyhdrdir\"]}/$(arch)\")'`"
+        ]
+        WITH_RUBY_VENDORARCHDIR=[`ruby -rrbconfig -e \
+            'puts RbConfig::expand(RbConfig::CONFIG["vendorarchdir"])'`]
     ], [])
 AC_SUBST(WITH_RUBY_CPPFLAGS)
 AC_SUBST(WITH_RUBY_SUBDIR)

--->%---

> Embedded, not bindings, is the issue. There's no way to do 1.9.x
> embedding without a thread co-routine still afaik.

That's true, but  the problem lies mainly in querying the correct 
inlcude/libraries paths. I.e., on my system, simply including <ruby/ruby.h> 
doesn't work. One needs /usr/include/ruby/$ARCH as an additional per-
architecture include path. So it's neccesssary for both bindings and embedded.

Meh.

The flaw that is hacksolved by the coroutine solution doesn't become apparent 
until runtime AFAIK. And its IMHO currently the better alternative as it 
allows multiple interpreter instances and proper memory pooling, doesn't it?

                        Eric
______________________________________________________________________
RPM Package Manager                                    http://rpm5.org
Developer Communication List                        rpm-devel@rpm5.org

Reply via email to