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
I have a local change that adds a new property in the package.json that allows you to link against any of the otherlibs or anything that ocamlfind can find (as it runs ocamlfind query [name] under the hood).
I used to get sexplib to compile.
The problem is that the way I implemented it was by adding the cma file of that dependency in the command that generates the lib.cma, but it doesn't seem to be bundled properly. This means that when compile A which depends on sexplib, I get errors like Reference to undefined global 'Bigarray'. This might mean that the right way to implement this is by making a flat list of all of those dependencies and linking them at the very end when building the executable. This is really annoying as it totally breaks the encapsulation of the lib, and might have some partial-build times impact.
I'm just writing this issue here to keep track of it. Right now I'll just propagate that field up the chain so we can get Core to build.
I have a local change that adds a new property in the package.json that allows you to link against any of the
otherlibs
or anything that ocamlfind can find (as it runsocamlfind query [name]
under the hood).I used to get
sexplib
to compile.The problem is that the way I implemented it was by adding the cma file of that dependency in the command that generates the
lib.cma
, but it doesn't seem to be bundled properly. This means that when compile A which depends onsexplib
, I get errors likeReference to undefined global 'Bigarray'
. This might mean that the right way to implement this is by making a flat list of all of those dependencies and linking them at the very end when building the executable. This is really annoying as it totally breaks the encapsulation of the lib, and might have some partial-build times impact.I'm just writing this issue here to keep track of it. Right now I'll just propagate that field up the chain so we can get Core to build.
ps: this is what I'm calling otherlibs
The text was updated successfully, but these errors were encountered: