[Pkg-utopia-maintainers] Bug#499650: Bug#499650: Bug#499650: Bug#499650: dbus: wrong dependencies on libice6, libsm6 and libx11-6

Michael Biebl biebl at debian.org
Tue Sep 30 17:46:58 UTC 2008


Luca Capello wrote:
> Hi Michael!
> 
> On Tue, 30 Sep 2008 14:51:36 +0200, Michael Biebl wrote:
>> Michael Biebl wrote:
>>> The problem seem to be very simple:
>>> configure.in:915-935
>>>
>>> The AC_PATH_XTRA macro picks up additional X11 libs if installed and
>>> adds them to $X_EXTRA_LIBS.
>>>
>>> If you install in a clean chroot, $X_EXTRA_LIBS will be empty.
>> Minor correction:
>> The additional depedencies on -lSM and -lICE are added to $X_PRE_LIBS
> 
> Now that I think about it, I already experienced this weird problem [1].
> 
>> I guess we can either add build conflicts against all non-required X11
>> dev packages (I'm not sure if this is viable)
>> or patch out X_EXTRA_LIBS from configure.in.
> 
> I think the easiest solution would be to simply rebuild the package,
> which is also the less invasive one.

Well, that obviously doesn't fix the underlying problem and it can
happen again anytime in the future.
I asked on #debian-x for advice. Here is the relevant irc log:


[19:14] <mbiebl> Hi, I've got a problem regarding the usage of the
AC_PATH_XTRA macro in dbus
[19:14] <mbiebl> http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=499650
[19:14] <mbiebl> Depending on which X dev packages are installed, this
macro adds additional link dependencies
[19:16] <noshadow> mbiebl: AC_PATH_XTRA is very old and likes to include
just everything it can find. That needs either LIB-overriding or
build-conflicts...
[19:16] <mbiebl> noshadow: Is there a better, recommended way nowadays?
[19:16] <jcristau> use pkg-config
[19:17] <jcristau> and only check for the libs you're using
[19:17] <pusling> cmake has also such a feature to include everything
matching libX*
[19:17] <noshadow> mbiebl: I don't think anyone ever bothered.
Everything else has other drawbacks. do you really need XTRA?
[19:17] <noshadow> AC_PATH_X might already be easier...
[19:17] <jcristau> so PKG_CHECK_MODULES(X11, x11), and then use
X11_CFLAGS and X11_LIBS
[19:17] <mbiebl> noshadow: I don't know, why XTRA was used the way it is
now.
[19:18] <noshadow> but please do not add X11_CFLAGS into CFLAGS but to
CPPFLAGS when using  pkgtool
[19:18] <mbiebl> jcristau: I'd have to check with dbus upstream, if this
would be an acceptable solution
[19:18] <mbiebl> (as it would be xorg > 7.0 only I guess)
[19:18] <jcristau> yeah
[19:19] <jcristau> AC_PATH_XTRA seems wrong if all you use is -lX11, in
any case...
[19:19] <noshadow> mbiebl: all thos AC_PATH_X* stuff support the whole
Xt you most likely not need. so just a AC_CHECK_LIB for X11 might be
more usefulll
[19:20] <mbiebl> noshadow: My only guess here was, that this might be
required for other X11 implementations
[19:20] <mbiebl> But that's only wild guessing
[19:21] <jcristau> is there really any other x11 implementation worth
caring about today, though?
[19:21] <noshadow> mbiebl: I think if you only need X11, the only
advantage is that it does autodetection if you need -L/usr/X11R6/lib
[19:22] <mbiebl> jcristau: I don't care for anything else besides x.org,
others might disagree.
[19:22] <noshadow> oh, and /usr/X11R5 and /usr/X11R4
[19:22] <mbiebl> Especially as dbus is used on a lot of plattforms
[19:22] <mbiebl> But I guess it's best to bring this topic up on the
dbus m-l
[19:23] <noshadow> mbiebl: I'd suggest to just add options at which
non-standard path to look and check for libX11, then people can just
give the path when they have some very old implementation.

-- 
Why is it that all of the instruments seeking intelligent life in the
universe are pointed away from Earth?

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 260 bytes
Desc: OpenPGP digital signature
Url : http://lists.alioth.debian.org/pipermail/pkg-utopia-maintainers/attachments/20080930/81b10866/attachment-0001.pgp 


More information about the Pkg-utopia-maintainers mailing list