https://bugzilla.redhat.com/show_bug.cgi?id=1519834



--- Comment #6 from [email protected] ---
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 -- [email protected]
To unsubscribe send an email to [email protected]
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/[email protected]

Reply via email to