search

saleyn / eixx

Erlang C++ Interface
Apache License 2.0
137 stars 26 forks source link

Why "Temporary disable support for a C++ erlang node" #41

Closed heri16 closed 3 years ago

heri16 commented 3 years ago

14 months ago, there was a commit message that says "Temporary disable support for a C++ erlang node."

https://github.com/saleyn/eixx/blob/2253d6417b386086b43291c55444c4e620177ee1/include/eixx/eixx.hpp#L62

Is this still disabled? What is the rationale or reasoning for this feature removal?

saleyn commented 3 years ago

With later versions of BOOST I was getting some segfaults, which I had no time to review and fix. You are welcome to uncomment and submit patches if you can make it work.

heri16 commented 3 years ago

Thanks for the update! I'll try to make it work.

Would the latest working version be Boost 1.59 or Boost 1.64?

https://github.com/saleyn/eixx/blob/c5b3d8f6580f054070b93147abde762577be3998/src/basic_otp_node_local.cpp#L22

saleyn commented 3 years ago

I don't quite recall. Haven't used that part of code for several years. Only have been using the marshaling part.

heri16 commented 3 years ago

Any production use of this library that you could share about? So our engineering manager can be see a good future for this open-source code, rather than using ei directly.

saleyn commented 3 years ago

The marshaling part of the library is being used is several production projects. The node connectivity part was used in production several years ago, until that system removed dependency on boost ASIO, at which point there was no solid need to maintain that compatibility in eixx. If needed to revive it, I can do so on consulting basis, or you can do it yourself, and send patches.