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@xxxxxxxxx> 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. _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review