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 ------- Additional Comments From gemi@xxxxxxxxxx 2006-06-24 06:46 EST ------- (In reply to comment #5) > 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. I will use the current download url: http://gforge.inria.fr/frs/download.php/586/SmartEiffel-2-2.tar.bz2 > 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. I don't think it makes much sense to change the settings in /etc/serc. > 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.) yes. > 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. I always thought that /usr/libexec is the standard directory for executables that are called as subprocesses and not directly by the user. I really don't understand the warning, /usr/libexec seems pretty standard to me... > 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.) Hmm, if I chmod 0644 these files, I must also remove the shebang. Maybe I should move the contrib directory to /usr/share/SmartEiffel. There are also a few files to patch, paths etc... -- 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