search

mirage / ocaml-uri

RFC3986 URI parsing library for OCaml
Other
98 stars 57 forks source link

Added an uri.mldylib in order to build uri.cmxs correctly. #29

Closed vbmithr closed 10 years ago

vbmithr commented 10 years ago

OASIS should do this.

avsm commented 10 years ago

Is there an OASIS bug on this?

vbmithr commented 10 years ago

On 29/11/2013 14:27, Anil Madhavapeddy wrote:

Is there an OASIS bug on this?

Yes. In the absence of foo.mldylib, foo.cmxs is produce from foo.cmx instead of being produced by foo.cmxa. It doesn't reuse the foo.mllib file. It is not necessary a bug in ocamlbuild (there is a rationale behind this I think), but in OASIS, when you specify foo as a library, foo.cmxs should be build from foo.cmxa (hence OASIS should generate the foo.mldylib file). Does this make sense ?

avsm commented 10 years ago

Yep, that makes sense -- I was wondering if you'd filed an OASIS bug on it or not though, as this should be fixed upstream as well (or else we'll keep forgetting to add this file and keep it in sync)

vbmithr commented 10 years ago

On 29/11/2013 15:00, Anil Madhavapeddy wrote:

Yep, that makes sense -- I was wondering if you'd filed an OASIS bug on it or not though, as this should be fixed upstream as well (or else we'll keep forgetting to add this file and keep it in sync)

There is something related here: https://forge.ocamlcore.org/tracker/?func=detail&atid=626&aid=1303&group_id=133

But this is not a completely accurate description of the issue. I'm going to check out OASIS bugs.