Unable to Patch C extension gems - What approach?

[Shawn Starr]

On Monday, February 13, 2012 11:25:42 PM Shawn Starr wrote:

<snip>

> I've found some example:
> 
> http://callsign.rubyforge.org/git?p=nokogiri.git;a=commitdiff_plain;h=a47cba
> d917e75e653cdb97a8d99f4bb5bbab4d50
> 
> +#include <ruby/encoding.h>
> +
> +#define ENCODED_STR_NEW2(str, encoding) \
> +  ({ \
> +    VALUE _string = rb_str_new2((const char *)str); \
> +    int _enc = rb_enc_find_index(encoding); \
> +    rb_enc_associate_index(_string, _enc); \
> +    _string; \
> +  })
> +
> 
> +#define ENCODED_STR_NEW(str, len, encoding) \
> +  ({ \
> +    VALUE _string = rb_str_new((const char *)str, (long)len); \
> +    int _enc = rb_enc_find_index(encoding); \
> +    rb_enc_associate_index(_string, _enc); \
> +    _string; \
> +  })
> +
> We just adapt accordingly, passes tests now without extra patch.. fixing
> rubygem-addressable also to not use any patch (it shouldn't need it now)
> 
> 

Can Vít or someone please review 
https://bugzilla.redhat.com/show_bug.cgi?id=588428? (rubygem-addressable) so I 
can push the latest package into fedora 17?

Thanks,
Shawn.

> _______________________________________________
> ruby-sig mailing list
> ruby-sig at lists.fedoraproject.org
> https://admin.fedoraproject.org/mailman/listinfo/ruby-sig