Closed dylanahsmith closed 3 years ago
cc @arthurnn in case you have context on why the previous attempt to use Swig 3 was reverted (https://github.com/arthurnn/memcached/commit/26531ee7dedb9a0a3090fe6c65546e79e53ef413) @casperisfine for review
The swig bindings were generated using a version of swig from 2012, so I regenerated them using the latest swig release. This should make it easier to contribute changes to the rlibmemcached.i file.
rlibmemcached.i
Also, I noticed that we were patching the swig output for JRuby in the C code, but JRuby doesn't support C extensions. I found out that Ruby 1.6 had experimental support for the MRI C API, which was disabled in 1.7 and then removed. So we shouldn't bother continuing to support it.
cc @arthurnn in case you have context on why the previous attempt to use Swig 3 was reverted (https://github.com/arthurnn/memcached/commit/26531ee7dedb9a0a3090fe6c65546e79e53ef413) @casperisfine for review
The swig bindings were generated using a version of swig from 2012, so I regenerated them using the latest swig release. This should make it easier to contribute changes to the
rlibmemcached.ifile.Also, I noticed that we were patching the swig output for JRuby in the C code, but JRuby doesn't support C extensions. I found out that Ruby 1.6 had experimental support for the MRI C API, which was disabled in 1.7 and then removed. So we shouldn't bother continuing to support it.