v.1.0.0 opam CI test failure about camomile

[toots]

Fixed in our latest source code, I've pushed a PR against the opam-repository PR..

[kit-ty-kate]

I'm not sure how jbuilder is supposed to work here but my guess is that the problem comes from here: https://github.com/yoriyuki/Camomile/blob/master/Camomile/jbuild#L42

As far as I understand, this creates a empty library right ? Well at least ocamlobjinfo doesn't see anything there:

$  ocamlobjinfo camomile_yuge.cma
File camomile_yuge.cma
Force custom: no
Extra C object files:
Extra C options:
Extra dynamically-loaded libraries:

[kit-ty-kate]

Oh no, that something else, sorry.
liquidsoap is looking for camomileLibraryDyn.cmi which isn't built

Btw there are no other problems:

• gasoline doesn't build for some reason (idk but it doesn't seem related to Camomile)
• patoline doesn't find its archive
• the Alpine Linux failure is a know issue and totally unrelated.

You can see all the logs by clicking to [logs] on the CI's page

[yoriyuki]

Thank you for investigating,

I don't know how configure.ml works, but camomileLibraryDyn.cmi is missing because we changed the module structure. We can revert the change but it is supposed to be an improvement. Maybe changing liquidsoup is easier? I think detecting a library by looking at a particular file is not terribly a good way to do so.

[rgrinberg]

Maybe changing liquidsoup is easier? I think detecting a library by looking at a particular file is not terribly a good way to do so.

Yes, this is definitely not the kind of backwards compatibility we strive to maintain. We should look into updating liquidsoap if it's relying on something hacky.

cc @toots

[toots]

I'll look at it.

[yoriyuki]

There are still build failures, but by failure of building camomile.0.8.7. Failure is seemingly random. I do not know opam's CI infrastructure, so I don't know why.

[yoriyuki]

Successfully merged to opam ocaml/opam-repository@820cd19

Thank you, everyone!