[Bug 209222] Review Request: prelude-lml - Prelude log analyzer

bugzilla at redhat.com bugzilla at redhat.com
Tue Jan 2 16:07:44 UTC 2007


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: prelude-lml - Prelude log analyzer


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=209222





------- Additional Comments From mtasaka at ioa.s.u-tokyo.ac.jp  2007-01-02 11:07 EST -------
(In reply to comment #7)
> >The prelude-lml.h header provide the necessary API for contributor to
> >write LML plugins. 
Ah.. for plugins... I see.

(In reply to comment #9)
> from upstream:
> 
> prelude-lmlS.c problem:
> >I'm not aware of any way to prevent removal of this file. I can also
> >tell that this file is not necessary for debugging (it's only used by
> >libtool as a way of setting up preopening).

It may be, however, there is a easy fix so fixing this
is preferable IMO.
-----------------------------------------------
%prep
%setup -q
sed -i.debug -e '/nlist/s|\$rm|: $rm|' ltmain.sh
-----------------------------------------------

By the way, for encoding issue, doesn't the following
make effect for you?
--------------------------------------------------
%build
%configure
make

for f in ChangeLog NEWS ; do
	iconv -f ISO-8859-15 -t UTF-8 $f > ${f}.tmp &&
		mv -f ${f}.tmp ${f} || rm -f ${f}.tmp
done
---------------------------------------------------
And.. does parallel make fail on this package?

Also, please fix the rest issues.

-- 
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.




More information about the Fedora-package-review mailing list