After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 731457 - configure: connman backend unnecessarily requires 'connman.pc' that breaks multilib building
configure: connman backend unnecessarily requires 'connman.pc' that breaks mu...
Status: RESOLVED FIXED
Product: GUPnP
Classification: Other
Component: gupnp
0.20.x
Other Linux
: Normal minor
: ---
Assigned To: GUPnP Maintainers
GUPnP Maintainers
Depends on:
Blocks:
 
 
Reported: 2014-06-10 13:22 UTC by Michał Górny
Modified: 2014-06-29 11:36 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Do not check for connman.pc (843 bytes, patch)
2014-06-29 11:31 UTC, Jens Georg
committed Details | Review

Description Michał Górny 2014-06-10 13:22:34 UTC
configure.ac states:

  if test "x$with_context_manager" = "xconnman"; then
          PKG_CHECK_MODULES(CONNMAN, connman >= 0.80)
  fi

While at a first glance there's nothing immediately wrong with this, if you read connman.pc you'd notice that this file is meant to be used when building connman plugins. For this reason, it is installed in arch-specific pkg-config dir ($libdir/pkgconfig). Since connman can use plugins built for a single ABI only, on multilib systems connman.pc lands only in lib64 or the equivalent.

The gupnp connman code uses only connman's dbus interface, and therefore is not interested in plugin building details. Therefore multilib build of gupnp could use connman backend on every ABI. However, since connman.pc is installed for native ABI only, the configure check fails and prevents e.g. 32-bit gupnp from using connman.

Please consider removing the check or replacing it with something less fragile. I will also report a bug to connman upstream suggesting that the .pc file is misleading.
Comment 1 Michał Górny 2014-06-10 14:36:12 UTC
connman bugreport: https://01.org/jira/browse/CM-658
Comment 2 Jens Georg 2014-06-29 11:31:21 UTC
Created attachment 279524 [details] [review]
Do not check for connman.pc

Signed-off-by: Jens Georg <mail@jensge.org>
Comment 3 Jens Georg 2014-06-29 11:36:47 UTC
Attachment 279524 [details] pushed as e7a284d - Do not check for connman.pc