Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 548368 - x11-themes/qtcurve - add multilib support
Summary: x11-themes/qtcurve - add multilib support
Status: RESOLVED DUPLICATE of bug 542062
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Unspecified (show other bugs)
Hardware: AMD64 Linux
: Normal normal (vote)
Assignee: Lars Wendler (Polynomial-C) (RETIRED)
URL:
Whiteboard:
Keywords: PATCH
Depends on:
Blocks:
 
Reported: 2015-05-02 01:12 UTC by Kai Krakow
Modified: 2015-05-03 16:26 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
patch for ebuild file (1.8.18-multilib-build.patch,2.80 KB, patch)
2015-05-02 01:12 UTC, Kai Krakow
Details | Diff
multlib-enhanced ebuild (qtcurve-1.8.18.ebuild,2.99 KB, text/plain)
2015-05-02 01:14 UTC, Kai Krakow
Details
multlib-enhanced ebuild (qtcurve-1.8.18.ebuild,2.78 KB, text/plain)
2015-05-02 02:24 UTC, Kai Krakow
Details
patch for ebuild file (1.8.18-multilib-build.patch,2.44 KB, patch)
2015-05-02 02:25 UTC, Kai Krakow
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Kai Krakow 2015-05-02 01:12:34 UTC
Created attachment 402410 [details, diff]
patch for ebuild file

32-bit binary-only cannot apply desktop styles if qtcurve is in use on 64-bit systems. The problem is that qtcurve does not build itself with multilib.

I've provided a patch (and ebuild file) which enables this. It was tested successfully on a KDE4 desktop with applications like Skype. The QtCurve styles are now applied to 32-bit only applications.
Comment 1 Kai Krakow 2015-05-02 01:14:33 UTC
Created attachment 402412 [details]
multlib-enhanced ebuild

Added full ebuild
Comment 2 Kai Krakow 2015-05-02 02:24:25 UTC
Created attachment 402414 [details]
multlib-enhanced ebuild

Re-up: fixed one files going to the wrong installation path
Comment 3 Kai Krakow 2015-05-02 02:25:10 UTC
Created attachment 402416 [details, diff]
patch for ebuild file

Re-up: fixed one files going to the wrong installation path
Comment 4 Michael Palimaka (kensington) gentoo-dev 2015-05-02 17:14:27 UTC
Not marking as dup just yet so we don't forget about the patch.
Comment 5 Ben de Groot (RETIRED) gentoo-dev 2015-05-03 03:39:25 UTC
Please rewrite the patch to make qt4 optional (it's now being hardcoded in the cmake arguments at the end), and attach it to bug 542062.

*** This bug has been marked as a duplicate of bug 542062 ***
Comment 6 Kai Krakow 2015-05-03 16:26:44 UTC
I think, qt4 actually is optional. I'm just needed to pass in search paths which won't be used by cmake if QT4_ENABLE is false. It looks like qtcurve is internally very broken to support multilib systems, that's why I had to pass these arguments, otherwise it jumps to lib64 in the 32-bit case. May also be a shortcoming of the eclasses or supplied cmake macros itself. I'm not sure. In the end, that's just a hack.

I'll make it optional nevertheless but I cannot test the ebuild without qt4 and with qt5 then. That's up to others because I do use (and need) qt4 and don't use qt5 yet. Not sure if the cmake configure phase goes havoc in that case.