+ extlib 1.7.8#17998
Conversation
…eviously pulled by extlib)
|
Commit: 06f28d1 A pull request by opam-seasoned @ygrek. 🌤️ opam-lint warnings 06f28d1
☀️ Installability check (+1)
|
…y uses Base64, previously pulled by extlib)
…viously pulled by extlib)
to clarify: extlib has implementation of Base64 module. There is also a separate opam package base64 (unrelated to extlib) which has much more elaborate implementation of base64. Due to all extlib modules being at top-level of the namespace - there was a name conflict which prevented using extlib and base64 opam packages in the same project. In this release extlib drops Base64 from the minimal build (but it is still available in the full build, opam package extlib-compat) to avoid the above problem, recommended solution is to switch to standalone base64 implementation. Short-term solution may be to use extlib-compat (but I stopped uploading it to opam for the lack of interest, let me know if it is needed). |
|
ah.. Looks like I made a bad judgement then. This is a rather large breakage for a point release, I'm rather surprised here. |
|
Mh, thinking about it a bit more, it makes sense if taken from the point of vue of extlib + extlib-compat I guess. I hadn't realized this was the goal of extlib-compat. Could you also send a PR for extlib-compat.1.7.8? that'd be wonderful. I'll handle the addition of |
|
extlib has inherent namespacing problem, and reducing number of modules (ideally leaving only Ext* ones) is one way to solve it eventually. Hence, I think it is best for ecosystem if dependent projects eventually migrated off these modules. I will upload extlib-compat. |
|
actually I just realized that probably easiest way to solve packaging/conflicts issues may be to provide one-module extlib-base64 extlib-utf8 etc libraries (best in the sense that namespacing problem remains, but extlib choices will not create new problems for existing code) |
Use base64 encoding from Cryptokit instead of extlib, because extlib 1.7.8 removed module Base64. See ocaml/opam-repository#17998
1.7.8 (2021-01-19)