You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Catalyst uses the .so file extension for dynamic libraries on macOS in a few places (see below). We should rename these libraries so that they use the standard .dylib extension.
A few examples where we use the .so file extension (this is not necessarily an exhaustive list):
@mlxd recommended this, since .dylib is the conventional file extension for dynamic libraries on macOS. I gather that on macOS there is a subtle difference between shared objects (which get the .so extension by convention) and dynamic libraries (which get the .dylib extension). If I understand correctly, we're creating dynamic libraries in our macOS builds, but giving some of them the .so file extension, so it's a slight misnomer. I think the argument for uniformity is a valid one, especially since there don't appear to be any issues with our macOS wheels shipped with .so files, so happy to discuss more.
Catalyst uses the
.so
file extension for dynamic libraries on macOS in a few places (see below). We should rename these libraries so that they use the standard.dylib
extension.A few examples where we use the
.so
file extension (this is not necessarily an exhaustive list):catalyst/setup.py
Lines 241 to 254 in 57bc011
catalyst/runtime/lib/registry/CMakeLists.txt
Line 43 in 57bc011
catalyst/frontend/catalyst/utils/CMakeLists.txt
Line 46 in 57bc011
Be sure to update the relevant Makefile commands and build scripts as well, for example:
catalyst/Makefile
Lines 215 to 216 in 57bc011
The text was updated successfully, but these errors were encountered: