https://bugzilla.redhat.com/show_bug.cgi?id=1798796 --- Comment #2 from Jerry James <loganjerry@xxxxxxxxx> --- (In reply to dan.cermak from comment #1) > Thanks for submitting this, package is approved! Thank you for the review, Dan! > Stupid question though: why are ocaml-ppx-derivers-devel and > ocaml-result-devel required? They are not mentioned in the opam file > upstream. I think they are both missing dependencies of ocaml-migrate-parsetree-devel. I'm not sure of that, but if they are, I should file a bug on ocaml-migrate-parsetree to get them added. Evidence: %{_libdir}/ocaml/ocaml-migrate-parsetree/opam contains this: depends: [ "result" "ppx_derivers" "dune" {build & >= "1.9.0"} "ocaml" {>= "4.02.3"} ] And %{_libdir}/ocaml/ocaml-migrate-parsetree/migrate_parsetree_ast_io.mli contains "open Result". All I know for sure is that omitting those two BRs leads to a build failure, with a message indicating certain files from their -devel subpackages are missing. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component _______________________________________________ package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/package-review@xxxxxxxxxxxxxxxxxxxxxxx
