Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CRuby enc/*.c needs other way to detect #78

Merged
merged 1 commit into from
Dec 10, 2016

Conversation

nurse
Copy link
Contributor

@nurse nurse commented Dec 10, 2016

When CRuby compiles encoding modules (enc/*.[ch]), those modules are
built into each object files. It means they don't include ruby.h.
To switch them CRuby mode, they need to detect another flags.
In this case, they check RUBY_EXPORT and ONIG_ENC_REGISTER.

When CRuby compiles encoding modules (enc/*.[ch]), those modules are
built into each object files. It means they don't include ruby.h.
To switch them CRuby mode, they need to detect another flags.
In this case, they check RUBY_EXPORT and ONIG_ENC_REGISTER.
@coveralls
Copy link

coveralls commented Dec 10, 2016

Coverage Status

Coverage remained the same at 81.372% when pulling 6eff152 on nurse:for-ruby-2.4 into 94a7436 on k-takata:master.

hsbt pushed a commit to ruby/ruby that referenced this pull request Dec 10, 2016
* https://github.com/k-takata/Onigmo/blob/Onigmo-6.0.0/HISTORY
* fix for ruby 2.4: k-takata/Onigmo#78
* suppress warning: k-takata/Onigmo#79
* include/ruby/oniguruma.h: include onigmo.h.
* template/encdb.h.tmpl: ignore duplicated definition of EUC-CN in
  enc/euc_kr.c. It is defined in enc/gb2313.c with CRuby macro.

git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@57045 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
@k-takata k-takata merged commit c13c243 into k-takata:master Dec 10, 2016
@k-takata
Copy link
Owner

Thanks.

@nurse nurse deleted the for-ruby-2.4 branch December 11, 2016 06:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants