Forum: Ruby-Gnome 2 Hey all, here's some changes for compatibility with latest Ruby 1.9.1 preview 2

Announcement (2017-05-07): www.ruby-forum.com is now read-only since I unfortunately do not have the time to support and maintain the forum any more. Please see rubyonrails.org/community and ruby-lang.org/en/community for other Rails- und Ruby-related community platforms.
A9b059eb8692215d448423606757ab09?d=identicon&s=25 Joao Pedrosa (jpedrosa)
on 2008-12-03 08:45
(Received via mailing list)
Attachment: patch_ruby_gnome2_for_ruby_1.9.diff (730 Bytes)
Hi,

Please review these changes and apply them as you see fit.

I am not entirely sure about the correctness of this patch so
review it, please. :-)

I could not come up with something better due to my limited
C knowledge.

Thanks!

Cheers,
Joao

Index: glib/src/rbgobj_type.c
===================================================================
--- glib/src/rbgobj_type.c  (revisão 3447)
+++ glib/src/rbgobj_type.c  (cópia de trabalho)
@@ -28,6 +28,7 @@
 static ID id_superclass;
 static VALUE gtype_to_cinfo;
 static VALUE klass_to_cinfo;
+static VALUE rb_thread_critical;

 static GHashTable* dynamic_gtype_list;
 typedef struct {
Index: glib/src/rbgcompat.h
===================================================================
--- glib/src/rbgcompat.h  (revisão 3447)
+++ glib/src/rbgcompat.h  (cópia de trabalho)
@@ -25,6 +25,8 @@

 extern void rbgobj_fund_define_fundamental(RGFundamental *fundamental);

+#define rb_progname (rb_argv0)
+
 G_END_DECLS

 #endif
Ee6ffca720cc428d70247dcd7377dd48?d=identicon&s=25 Kouhei Sutou (Guest)
on 2008-12-09 13:56
(Received via mailing list)
Hi,

In <ca2428780812022345s6821db29i764f1b45adf2c1@mail.gmail.com>
  "[ruby-gnome2-devel-en] Hey all, here's some changes for compatibility
with latest Ruby 1.9.1 preview 2" on Wed, 3 Dec 2008 04:45:09 -0300,
  "Joao Pedrosa" <joaopedrosa@gmail.com> wrote:

>  static GHashTable* dynamic_gtype_list;
I don't know that we can just ignore rb_thread_critical. I'm
asking it in ruby-dev.

>  G_END_DECLS
>
>  #endif

We should use rb_argv0 directory instead of defining
rb_progname.
I'll do it (or apply a patch from someone ;-) later.


Thanks,
--
kou
This topic is locked and can not be replied to.