[Bug 222347] Review Request: g-wrap - A tool for creating Scheme interfaces to C 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: g-wrap - A tool for creating Scheme interfaces to C libraries


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





------- Additional Comments From notting at redhat.com  2007-06-08 17:05 EST -------
(In reply to comment #3)
> rpmlint has some complaints:
> I admit I find it odd to see /usr/lib64/g-wrap/include/ffi.h:
>    E: g-wrap-devel only-non-binary-in-usr-lib
> Why not /usr/share/g-wrap, or with the rest of the installed headers in
> /usr/include/g-wrap?

It's wordsize-specific - having it in /usr/share or /usr/include would cause
multilib conflicts.

> Seems to me that the license is LGPL, not GPL.  The libffi license seems to be
> MIT but that's compatible and doesn't change the overall license of the package.

Oops, fixed in CVS.

> I note that there is a test suite, but some notes about it being disabled on
> x86_64.  I tried a quick "make check" on i386 rawhide and all the tests failed
> without attempting to actually test anything, so I must be missing something.

The test suite has never seemed to work. I should have probably put more effort
into figuring out why.

So...

> * no libtool .la files.

These are re-added in the current package; see bug 238263. gnucash uses a
version of libtool_ltdl as a module loader, and it does not work without either
a) .la files b) .so files in the main package. As moving the .so files breaks
multilib development, it was simplest to put the .la files back.


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