https://bugzilla.redhat.com/show_bug.cgi?id=1531955
--- Comment #6 from Alec Leamas <[email protected]> --- Here comes the full list of issues I see after making a full review: - seqan2 obsoletes seqan without providing seqan. - seqan2-devel does not depend on seqan2 = %{version}-%{release}. - A binary zip file is present in the examples package. - Several bundled files/directories in util/py_lib/seqan/dox/tpl/lib and include/seqan/stream. Please either remove these in %prep and/or bundle properly using Provides: bundled(...) - The License field comment does not match the actual file licenses in the sources. My proposal: Remove in %prep, add a text document describing the licensing situation. - ps2pswLinks.gawk and plot.awk in /usr/bin are executable but have no shebang. - There are tests available. Why are these not run? - The pkg-config file is not shipped with the -devel package for no obvious reason. - Likewise for seqan-config.cmake. Thoughts? -- 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]
