Remove the dependency on ounit2 and the relevant lit code. It seems
that ounit2 is not used at all and all OCaml binding tests pass without
it installed.
Thanks for Josh Berdine for noticing.
Differential D119884
[llvm] [bindings/OCaml] Remove unused dep on ounit2 mgorny on Feb 15 2022, 12:25 PM. Authored by
Details Remove the dependency on ounit2 and the relevant lit code. It seems Thanks for Josh Berdine for noticing.
Diff Detail
Event TimelineComment Actions Looks good, thanks! Note that it was @arbipher who first noticed that ounit is no longer used. Comment Actions It looks good to me. My 5 cent (for ocaml) is to keep it and it would be easier to really add some useful ounit tests, however, this PR also has shown what needs to do to revert it back. Comment Actions Ok, thanks for the correction and sorry for the mistake. Yeah, we can easily revert this if we ever introduce ounit-based tests. Comment Actions Do you think this should be backported to 14.x? On one hand, it isn't exactly urgent, but it kinda sucks to require this unused dep for another release. |