Closed hannesm closed 1 year ago
(alternatively, the lower bound for OCaml could be increased to 4.13 -- I think this is as well fine, since most users of omd won't use old OCaml compilers)
Hi! Sounds like your patch is a good idea: I didn't realise stdcompat was so brittle (tho I see I also filed an issue there awhile back.)
Thanks for the patch!
Thanks for your reply, @shonfeder. What is the plan forward in terms of OCaml version lower bound? Is just putting it on 4.13 fine with you? I opened #305 that requires OCaml 4.13 (with 5.0 out, I don't think maintaining old 4.x Stdlibs is worth it).
Dear Madam or Sir,
first of all thanks for maintaining this package. I've some trouble with the stdcompat dependency (which fails to install for me, reported https://github.com/thierry-martinez/stdcompat/issues/28).
So, I looked into OCaml 4.08 and omd without stdcompat, and the only two functions used from Stdcompat are List.find_map and String.for_all. Would you accept a patch (see below) that removes the stdcompat dependency (since omd requires OCaml 4.08).
WDYT?