[Bug 195683] Review Request: smarteiffel - The GNU Eiffel Compiler and Libraries

[bugzilla at redhat.com]

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 at math.uh.edu changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |tibbs at math.uh.edu




------- Additional Comments From tibbs at math.uh.edu  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.