[Telepathy] [Bug 15199] Broken pkg-config file

[bugzilla-daemon at freedesktop.org]

http://bugs.freedesktop.org/show_bug.cgi?id=15199


Simon McVittie <simon.mcvittie at collabora.co.uk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED




--- Comment #1 from Simon McVittie <simon.mcvittie at collabora.co.uk>  2008-03-25 09:08:06 PST ---
You're requesting that we revert a change I was asked to make a while ago.
Here's an edited transcript of the IRC discussions in which I was persuaded to
start using Requires.private in the first place - "Mithrandir" here is Tollef
Fog Heen, the pkg-config maintainer, who confirms that as long as we depend on
pkg-config >= 0.21, the change I made is legitimate and indeed to be
recommended.

If there is a concrete bug caused by this change (e.g. a package that uses
telepathy-glib fails to build or run in a particular environment) then please
let us know. Otherwise, I'm inclined to mark this NOTABUG.

11:38 < sjoerd> see http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=456335 for 
some interesting discussion

11:23 < sjoerd> It's a nasty issue
11:23 < sjoerd> Up untill now i haven't seen good guidelines to properly
cleanup
 .pc files
11:24 < sjoerd> Their don't see any good rules to when something does and when
s
omething doesn't go itno Libs vs Libs.private
11:24 < Mithrandir> in general, dependencies should go to Libs.private.
11:24 < Mithrandir> or Requires.private.
11:25 < sjoerd> So what 's left for Requires: ?
11:25 < smcv> we have GLib things in our API
11:26 < smcv> so we need at least GLib and GObject in Requires
11:26 < Mithrandir> smcv: that's fine, since Requires still means cflags are
pas
sed along.
11:26 < Zdra> that makes me think that .pc could be updated wit tp-glib
11:26 < Mithrandir> sjoerd: non-C languages, at least.
11:26 < smcv> Mithrandir: but Requires.private wouldn't be sufficient, right?
11:28 < Mithrandir> smcv: sorry, I misspoke.  Requires.private still means -I
ar
e passed along.
11:28 < Mithrandir> (iirc, etc)
11:30 < smcv> Mithrandir: pkg-config doesn't mention Requires.private - is it
bl
eeding-edge?
11:30 < smcv> pkg-config(1) I mean
11:30 < Mithrandir> smcv: the docs are lagging a bit, and well, bleeding-edge;
i
t's a year old or so.
11:30 < Mithrandir> iirc
11:33 < smcv> Mithrandir: judging by fd.o #3097, Requires.private does not do
wh
at we want
11:34 < smcv> Mithrandir: code using telepathy-glib will not compile without
-I/
usr/include/glib-2.0, which Requires.private does not give us
11:38 < bigon> patch about Require.private
http://www.mail-archive.com/util-linu
x-ng at vger.kernel.org/msg00962.html
11:46 < smcv> ok, so it appears
http://bugs.debian.org/cgi-bin/bugreport.cgi?bug
=340904 gives us the semantics Mithrandir wants
11:46 < smcv> until they change it again :-/

12:32 < Mithrandir> smcv: 3097 is ages old, and the semantics have changed once 
after that, introducing -I even for items in Requires.private.
12:36 < smcv> Mithrandir: yeah, having read some mailing list threads, it seems 
we can just require pkg-config 0.21
12:37 < smcv> Mithrandir: so that's what I merged
12:38 < Mithrandir> smcv: if the semantics for some reason doesn't fit how you
u
se it, please prod me so I can either get pkg-config adjusted or show you how
to
 use it correctly.
12:39 < Mithrandir> since people who use pkg-config tend not to tell me what
the
y want changed that often..
12:39 < smcv> Mithrandir: I'm using Requires.private for dbus, dbus-glib, glib, 
gobject now
12:39 < Mithrandir> you might want Requires: pkg-config >= 0.21 as well, as you 
noted.
12:39 < smcv> Mithrandir: libtelepathy still has a Requires on telepathy-glib
be
cause various symbols that used to be part of libtelepathy have migrated (!)
12:39 < smcv> Mithrandir: and yes, I'm checking for 0.21 in the configure.ac


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.