Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP
Browse files

Properly include st.h

  • Loading branch information...
commit 4f41aea8d523301a1d20b05f7a21b25f5bb13ea9 1 parent 27616ac
@vmg authored
Showing with 1 addition and 1 deletion.
  1. +1 −1  ext/redcarpet/redcarpet.h
View
2  ext/redcarpet/redcarpet.h
@@ -3,7 +3,7 @@
#define RSTRING_NOT_MODIFIED
#include "ruby.h"
-#include "st.h"
+#include "ruby/st.h"
#include <stdio.h>
#ifdef HAVE_RUBY_ENCODING_H

3 comments on commit 4f41aea

@benbc

Hello. This change breaks the gem for me (Ruby 1.8 on Ubuntu 12.04). My st.h is at /usr/lib/ruby/1.8/x86_64-linux/st.h rather than .../ruby/st.h.

I see other gems which deal with this problem like this:

extconf.rb:

have_header("ruby/st.h") || have_header("st.h")

then in the header:

#if HAVE_RUBY_ST_H
#include "ruby/st.h"
#endif
#if HAVE_ST_H
#include "st.h"
#endif

Do you think that would work for you? I don't know anything about native gem extensions, so I'm not really sure of the implications.

-Ben

@NullVoxPopuli

when is the gem going to be updated with this fix?

@bootstraponline

+1 for fixing the 12.04 issue

Please sign in to comment.
Something went wrong with that request. Please try again.