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

Dave Love <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]



--- Comment #19 from Dave Love <[email protected]> ---
I'll comment as I'd previously packaged openfoam and started on making it
comply with Fedora-isms.

The approach of putting it in its own tree (under %_libdir) is appropriate, and
it isn't a special case in that respect, but I can't see where it actually goes
now -- %prefix doesn't seem to be defined.  I would expect the source under
/usr/src, but I don't know if there's policy on that.

However, the packaging isn't at all right yet.  I haven't checked in detail,
but I noticed: Fedora doesn't allow conditionals for other distributions (which
I think is unfortunate), you need serial and openblas and mpich packages
(unless mpich won't work for some reason), I don't understand the bit about
CGAL missing in f32.


-- 
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