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: smarteiffel - The GNU Eiffel Compiler and Libraries https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=195683 tibbs@xxxxxxxxxxx changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |tibbs@xxxxxxxxxxx ------- Additional Comments From tibbs@xxxxxxxxxxx 2006-06-23 22:32 EST ------- I agree about the debug package; there's no point in it. I can't seem to reach upstream to download the source tarball at the moment. Builds fine in mock on x86_64, development; no parallel build, but perhaps that can't be helped as there's not really a makefile. The whole install process is pretty bizarre. (The tarball even includes a windows exe. Weird.) rpmlint spews a pile of warnings, but I'll grep out all of the "devel-file-in-non-devel-package" warnings as this is a compiler. That leaves: W: smarteiffel conffile-without-noreplace-flag /etc/serc Not sure what's up here. Is it expected that the user will need to edit this file? If upgrading risks overwriting expected local configuration then noreplace is indeed the proper thing to do, but I suppose this may be an occasion where it's not warranted. E: smarteiffel zero-length /usr/share/SmartEiffel/short/tex3/hook832 Generally zero-length files shouldn't be packaged, but I think in this case the file needs to be there. (If I understand correctly, it's a formatting hook, where some text could be inserted at various points in some generated output.) W: smarteiffel non-standard-dir-in-usr libexec Seems acceptable to me, or at least in line with other accepted uses of libexec, although I wonder how this would work with multilib. W: smarteiffel doc-file-dependency /usr/share/doc/smarteiffel-2.2/contrib/htmldoc/htmlshort /usr/bin/env (and several others) I don't think these should be executable. (Wow, a program written in Ruby which handles running the SmartEiffel compiler under gdb.) -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug, or are watching the QA contact. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review