[Bug 753577] Review Request: gmsh - finite element grid generator
bugzilla at redhat.com
bugzilla at redhat.com
Mon Nov 14 09:04:11 UTC 2011
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=753577
--- Comment #2 from Alexey Vasyukov <vasyukov at gmail.com> 2011-11-14 04:04:10 EST ---
Dan, many thanks for comments.
> - system copy of the ANN library (from the ann package)
> must be used instead of the copy in contrib/ANN
Ok. I'll work on it.
> - sources contains mpeg encoder which is a patented thing
Ok. Got the issue. I'll just remove mpeg - it does not affect core
functionality.
> - contrib/tetgen contains a non-commercial only clause
Metis has also similar non-commercial only clause. I disabled them both in
CMake configure. Is it not enough? Should I remove them completely from
sources?
> - the contrib dir contains stuff that could/should exists as separate packages
Ok. I'll re-check it. Could you advice which packages require attention?
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
More information about the package-review
mailing list