Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 115858 - for x11-libs/qt-4.1: /usr/lib/qt4/*.pc -> ${ROOt}usr/$(get_libdit)/pkgconfig/
Summary: for x11-libs/qt-4.1: /usr/lib/qt4/*.pc -> ${ROOt}usr/$(get_libdit)/pkgconfig/
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Caleb Tennis (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-12-17 08:02 UTC by Carsten Lohrke (RETIRED)
Modified: 2007-04-11 22:43 UTC (History)
0 users

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Carsten Lohrke (RETIRED) gentoo-dev 2005-12-17 08:02:07 UTC
Apart from the placement of the pkgconfig files, the existing ebuilds aren't ${ROOT} enabled, yet.
Comment 1 Caleb Tennis (RETIRED) gentoo-dev 2005-12-20 05:53:13 UTC
I'll fix the pkgconfigs shortly.  What all needs to have a ROOT in front of it?  Just install directories?
Comment 2 Caleb Tennis (RETIRED) gentoo-dev 2005-12-20 06:05:22 UTC
I'm thinking that maybe it's smarter to move these files into /usr/$(get_libdir)/qt4/pkgconfig instead of /usr/lib/pkgconfig and then adding that path to the PKG_CONFIG_PATH environment variable when qt is installed.

Thoughts?
Comment 3 Carsten Lohrke (RETIRED) gentoo-dev 2005-12-20 06:41:56 UTC
(In reply to comment #1)
> What all needs to have a ROOT in front of it?
> Just install directories?

Basically yes. Portage utility functions/scripts/variables like dodir, insinto, $D etc. should handle it transparently, so only where / is directly referenced in ebuilds or build scripts have it hardcoded instead using $prefix, etc. it is needed. Usually I fix the ebuilds I come across and leave possible patches to those who use an alternative root path.

(In reply to comment #2)
> Thoughts?

As long as it gets found I don't care. Just to note that none of the applications I have installed doesn't use the standard pkgconfig directory.
Comment 4 Caleb Tennis (RETIRED) gentoo-dev 2005-12-20 06:47:43 UTC
I noticed that graphviz installs in /usr/lib/graphviz/pkgconfig, but maybe that's an ebuild mistake.

The only reason I'm a little afraid to use /usr/lib/pkgconfig is that the filenames aren't versioned (QtCore.pc, QtNetwork.pc...) so if/when Qt5 comes out what if the names conflict?

But, as well, if we use the environment variable approach, then it won't work for processes not started by the user (daemons and such).  I guess I'll just stick to /usr/lib/pkgconfig for now, and we can always fix it later if we need to.
Comment 5 Carsten Lohrke (RETIRED) gentoo-dev 2005-12-20 08:56:07 UTC
(In reply to comment #4)
> I noticed that graphviz installs in /usr/lib/graphviz/pkgconfig, but maybe
> that's an ebuild mistake.

Probably. I have graphviz-2.6 installed and the files are in /usr/lib/pkgconfig.

> The only reason I'm a little afraid to use /usr/lib/pkgconfig is that the
> filenames aren't versioned (QtCore.pc, QtNetwork.pc...) so if/when Qt5 comes
> out what if the names conflict?

I'd say it's up to Trolltech to provide foo-x.pc. That's how it's handled by other projects, too. Personally I don't understand why they didn't do so with qmake etc. either, instead leaving it to their userbase to make sure the correct binary is chosen.
Comment 6 Caleb Tennis (RETIRED) gentoo-dev 2006-01-04 06:03:34 UTC
pkgconfig files have been fixed (4.1.0-r1 anyway).  ROOT is still unfixed.
Comment 7 Caleb Tennis (RETIRED) gentoo-dev 2006-01-26 05:24:40 UTC
Carlo, if you want to tackle the $ROOT feel free.  I'm always afraid I'm going to mess something up.
Comment 8 Arvid Picciani 2006-04-03 23:51:50 UTC
should not be moved. i think linking is the better way, becouse /usr/qt/* is not that much lfs compliant.
maybe the version coliding could be reduced with subdirs like /usr/bin/qt4/
(like it is already for libs and includes)
Comment 9 Carsten Lohrke (RETIRED) gentoo-dev 2007-04-11 22:43:15 UTC
pkg config path is cared for and the proposed $ROOT change is invalid, based on a missunderstanding on my side, what folks intended to achieve.