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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]