Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 398953 - x11-libs/libxcb-1.7 shouldn't RDEPEND on x11-proto/xcb-proto
Summary: x11-libs/libxcb-1.7 shouldn't RDEPEND on x11-proto/xcb-proto
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: Normal minor (vote)
Assignee: Gentoo X packagers
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-01-15 11:15 UTC by Maxim Kammerer
Modified: 2012-04-21 12:42 UTC (History)
1 user (show)

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 Maxim Kammerer 2012-01-15 11:15:42 UTC
libxcb RDEPENDs on xcb-proto - however, xcb-proto is apparently only used during libxcb's build for generating header files in /usr/include/xcb. xcb-proto RDEPENDs on python, which causes an effective python dependency for libxcb.

I suggest moving xcb-proto dependency to DEPEND in the ebuild.

If that is the wrong thing to do for some reason, I suggest the following approach:

1. Split xcb-proto's xcbgen package into IUSE=python.
2. If libxcb needs xcbgen during build (no idea whether it does), then RDEPEND on xcb-proto, and DEPEND on xcb-proto[python]
3. If libxcb doesn't need xcbgen during build, then no change is needed in libxcb's ebuild.
Comment 1 Chí-Thanh Christopher Nguyễn gentoo-dev 2012-01-17 10:41:32 UTC
The problem is probably similar to bug 342393. While libxcb does not strictly runtime-depend on xcb-proto, packages that build against libxcb might fail if xcb-proto is not installed.
Comment 2 Maxim Kammerer 2012-01-17 11:28:34 UTC
I have temporarily put xcb-proto in package.provided, unmerged it, and rebuilt the libxcb-dependant packages that I have:

[ebuild   R    ] x11-libs/libX11-1.4.4  USE="-doc -ipv6 -static-libs -test" 0 kB
[ebuild   R    ] x11-libs/xcb-util-0.3.8  USE="-doc -static-libs -test" 0 kB
[ebuild   R    ] x11-drivers/xf86-video-intel-2.17.0-r3  USE="dri sna" 0 kB
[ebuild   R    ] x11-apps/xdpyinfo-1.3.0  USE="xinerama -dga -dmx" 0 kB
[ebuild   R    ] x11-apps/xlsclients-1.1.2  0 kB
[ebuild   R    ] x11-apps/xwininfo-1.1.2  0 kB
[ebuild   R    ] x11-libs/startup-notification-0.12  USE="-static-libs" 0 kB

All merged fine. Maybe xcb-proto can be move to DEPEND after all.
Comment 3 Chí-Thanh Christopher Nguyễn gentoo-dev 2012-01-17 11:59:36 UTC
/usr/lib/pkgconfig/xcb-ewmh.pc:Requires: xcb xcb-proto
Not sure if any package in tree builds against that.
Comment 4 Maxim Kammerer 2012-01-17 12:49:40 UTC
(In reply to comment #3)
> /usr/lib/pkgconfig/xcb-ewmh.pc:Requires: xcb xcb-proto
> Not sure if any package in tree builds against that.

Doesn't seem so:

# find /usr/portage -name '*.ebuild' | xargs grep -l xcb-util-wm
/usr/portage/x11-libs/xcb-util/xcb-util-0.3.8.ebuild
/usr/portage/x11-libs/xcb-util-wm/xcb-util-wm-0.3.8.ebuild
Comment 5 Maxim Kammerer 2012-01-25 01:16:00 UTC
Hi, any news? Can xcb-proto be moved to DEPEND?
Comment 6 Matt Turner gentoo-dev 2012-01-25 04:52:24 UTC
(In reply to comment #5)
> Hi, any news? Can xcb-proto be moved to DEPEND?

No. Not really.

Portage needs to support a new kind of DEPEND for this case.
Comment 7 Chí-Thanh Christopher Nguyễn gentoo-dev 2012-01-26 00:38:30 UTC
After looking at the pkgconfig files, it appears mostly safe to drop the RDEPEND on xcb-proto.

The only package that needs to be adjusted appears to be xcb-util-wm, which should have the xcb-proto dependency anyway as xcb-ewmh.pc requires xcb-proto.
Comment 8 Diego Elio Pettenò (RETIRED) gentoo-dev 2012-02-08 22:58:55 UTC
Can I proceed with this as QA? I'm hitting this on one of my projects.
Comment 9 Matt Turner gentoo-dev 2012-02-08 23:47:27 UTC
(In reply to comment #8)
> Can I proceed with this as QA? I'm hitting this on one of my projects.

Yes, please.
Comment 10 Chí-Thanh Christopher Nguyễn gentoo-dev 2012-02-08 23:50:56 UTC
The dependency on xcb-proto was changed to build-time only.

Regarding the xcb-util-wm, the pkconfig entry seems bogus at first glance, but that is something upstream will have to address.
Comment 11 Chí-Thanh Christopher Nguyễn gentoo-dev 2012-04-21 12:42:58 UTC
This change broke cross compiling, bug 412909.