[Bug 346121] Review Request: malaga - Programming language for modelling of language-dependent grammatical information

[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: malaga - Programming language for modelling of language-dependent grammatical information


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





------- Additional Comments From vpivaini at cs.helsinki.fi  2007-10-28 12:57 EST -------
New spec: http://vpv.fedorapeople.org/packages/malaga.spec
New SRPM: http://vpv.fedorapeople.org/packages/malaga-7.11-0.4.fc7.src.rpm

(In reply to comment #5)
> - For now I can admit as it is, however please contact with upstream so
>   that binaries in malaga package actually uses libmalaga.

I'll do this. I might even try to make a patch myself, but the Fedora packaging
process will be a lot quicker if we won't stay waiting for it.

> Rest issues:
> For 7.11-0.3:
> 
> * readline dependency

CHANGES.txt mentions:
"The configure option "--with-readline" enables fancy command line editing,
provided that the GNU readline library is installed."
But also:
"When using "libmalaga", the readline library is no longer needed"

I'm not sure what to make of that, but I added the --with-readline option to
configure and also added BuildRequires readline-devel. I guess it won't hurt to
have the readline support in there.

> * Dependency for -devel package
>   - Well, after I reread your spec file, perhaps -devel
>     package does not depend on malaga package, only depends
>     on libmalaga package.
>     For -devel package,
> ------------------------------------------------------
> Requires:       %{name} = %{version}-%{release}
> ------------------------------------------------------
>     should be changed to
> ------------------------------------------------------
> Requires:       libmalaga = %{version}-%{release}
> ------------------------------------------------------

I tried this, but rpmlint said it was an error to not have Requires malaga in
-devel, so I left it as it was.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.