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(a)bluewin.ch 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.