[Bug 201178] Review Request: pinball - Emilia Pinball game

[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: pinball - Emilia Pinball game


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





------- Additional Comments From j.w.r.degoede@xxxxxx  2006-08-10 05:40 EST -------
(In reply to comment #8)
> Bad
> Missing rm -rf %{buildroot} at the start of %install
> 

Good catch! New version with this fixed (only specfile changed):
Spec URL: http://people.atrpms.net/~hdegoede/pinball.spec
SRPM URL: http://people.atrpms.net/~hdegoede/pinball-0.3.1-5.src.rpm


(In reply to comment #9)
> Can you also check that if you use the fedora libtool if the .la files are
> still needed?
Yes they are still needed.


(In reply to comment #10)
> It may be worth actually patching the app to use the fedora libtool and seems
> pretty trivial to do...
> 

It already is using the Fedora/system libtool, quoting from %setup:
"rm -fr libltdl", so there is no way its using its own version :)


> in LoaderModule.cpp, Line 86 change
> 
> lt_dlopen() to lt_dlopenext() and remove the extension on the passed in filename
> 
> There may be a couple more of those, so you'll need to grep the source.

That is not using the Fedora libtool, that is patching arounds libltdl's
behaviour / default use to open the .la files. All kde apps with plugings (as in
Fedora) do this, and I see no reason why pinball shouldn't/couldn't there is no
reason to deviate from upstream here / use ltdl in a non standard way here.

.la files are evil / must be removed for normal libraries which are intended to
link against, because when an application gets build using libtool and those .la
files are there libtool will prefer the .la files above the .so files and will
use the often broken dependencies in those .la files instead of the soname deps
in the .so files (which get tracked by rpm). This is known as libtool dependency
hell, but this only happens with libraries which are intented to link against by
other apps and which thus are usually installed directly into %_libdir and not
into a subdir of it as the *plugins* are. So there is no reason to start
patching and deviating from upstream here, the .la files are harmless because
they are for plugins and beside harmless also needed as this is the default mode
of operation of ltdl. If you don't like this file a bug against ltdl and the
zillion of kde packages doing the same.

(In reply to comment #11)
> Created an attachment (id=133896)
 --> (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=133896&action=view) [edit]
> Patch to allow ltdl to load from an so instead of an la file
> 
> Hey Hans, Here's a patch to load the modules from the .so file if the .la file
> is not present.  This patch would be a good starting point with upstream but
> may need a little work (I'm not sure whether the fallback lt_dlopen() is
> necessary and my C++ dates from before the STL so you can tell how rusty I am.)
>  I'll submit my spec file as well so you can diff for the changes.

Thanks,  but no thanks see above for the reasoning.


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