core sublibrary clashes with core alternate standard library

[roddyyaga]

Hey, I'm running into problems using this in a project that also depends on https://github.com/janestreet/core (inconsistent assumptions over interface for Core__Ref). Changing the riot files to reference riot.core doesn't fix this.

Could this be renamed to riot_core or something? That's how I fixed it locally. Possibly this is a bug in dune (I vaguely thought this was supposed to work), but even if it is I guess the fix for that will take a while, so it would be nice to fix it in riot.

[leostera]

Thanks for flagging this! 👍🏼 I'll look at how to hide those libs, this stuff should be contained in the Riot.Runtime module.

[leostera]

Incoming fix! 🚀 This should fix it: 329b093 – could you pin it locally and let me know if it solves your issue?

[leostera]

@roddyyaga did this fix it for you?

[FraMecca]

I get this with riot 0.0.8

File "bin/amq.ml", line 1:          
Error: The files /home/user/.opam/5.1.1/lib/async/async.cmi
       and bin/.main.eobjs/byte/dune__exe__Datatypes.cmi
       make inconsistent assumptions over interface Core
Had 1 error, waiting for filesystem changes...

[leostera]

As per @anmonteiro's suggestion we should try using (include_subdirs qualified) at the riot/runtime level to just ship a single library for the whole runtime.