search

zeroc-ice / ice

All-in-one solution for creating networked applications with RPC, pub/sub, server deployment, and more.
https://zeroc.com
GNU General Public License v2.0
2.04k stars 592 forks source link

Review if we need to set dlopenflags to load IcePy #2414

Closed pepone closed 3 months ago

pepone commented 3 months ago

https://github.com/zeroc-ice/ice/blob/6d8603a75771bb35c03f331e3bc6b972589e931f/python/python/Ice/__init__.py#L24-L38

I believe this is no longer required because we moved SSL into the Ice core, and IcePy is linked with the Ice library. The previous issue was related to IcePy not being linked with IceSSL and performing dynamic_cast on types defined in IceSSL (IceSSL::EndpointInfo).

See also https://zeroc.atlassian.net/issues/ICE-3366

pepone commented 3 months ago

Removed this in #2413 and seems to work with all CI platforms.