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

_______________________________________________
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]