https://bugzilla.redhat.com/show_bug.cgi?id=1519834 --- Comment #6 from david08741@xxxxxxxxx --- Thanks for the review. I think I have addressed all the comments: I tried to take EPEL8 into account, however some dependencies are missing, such as netcdf. I will have a more detailed look once CentOS 8 is released, because I don't have any access to rhel 8 outside of copr. As long as I don't build for EPEL8, this shouldn't be an issue for the review for fedora? The man page was only a single page, describing the whole library, as such I disabled that for now, there is html documentation, which is probably easier to read and navigate. I will ask upstream whether we can add man pages for the executables. The updated URLs: Spec URL: https://copr-be.cloud.fedoraproject.org/results/davidsch/bout/fedora-rawhide-x86_64/01015792-bout++/bout++.spec SRPM URL: https://copr-be.cloud.fedoraproject.org/results/davidsch/bout/fedora-rawhide-x86_64/01015792-bout++/bout++-4.2.2-0.fc32.src.rpm -- 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
