Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: ncarg - A Fortran and C based software package for scientific visualization https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=178263 ------- Additional Comments From ed@xxxxxxx 2006-01-29 16:12 EST ------- Hi Orion, I don't have an FC5 system running at the moment to test whether the ncarg binaries segfault or not. So I'll have to skip that part of the review for now. In any case, here are the parts that I could do: good: + source matches upstream + specfile is legible + license is ok and correctly included + builds in mock FC5 i386 + code not content -- although it apparently contains some basic map, font (?), etc. data for creating the graphics + dir ownership and permissions looks OK + no shared libs and no *.la files needswork: - please add an "unset NCARG" at the end of %build - please consider using Chris Chabot's suggestion in comment #4 - rpmlint emits a few warnings/errors: OK to ignore: E: ncarg script-without-shellbang /etc/profile.d/ncarg.csh E: ncarg script-without-shellbang /etc/profile.d/ncarg.sh W: ncarg-devel no-documentation Please put all the tutorial bits (/usr/lib/ncarg/tutorial/) in the -devel package or perhaps in a -doc package: W: ncarg devel-file-in-non-devel-package /usr/lib/ncarg/tutorial/c_colcon.c Probably, this should go in the devel package since I don't think it gets used via any run-time (eg. plug-in) methods. Or does it? W: ncarg devel-file-in-non-devel-package /usr/lib/ncarg/graphcaps/aed.a - please put all the /usr/lib/lib*.a files in /usr/lib/ncarg/ so that they don't pollute the /usr/lib/ namespace with generic names - please have the devel package require the main package: "Requires: %{name} = %{version}-%{release}" -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is. -- fedora-extras-list mailing list fedora-extras-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/fedora-extras-list
