554078 – dev-libs/libappindicator: disable introspection if use flag is disabled

Bug 554078 - dev-libs/libappindicator: disable introspection if use flag is disabled

Summary: dev-libs/libappindicator: disable introspection if use flag is disabled

Status:	RESOLVED FIXED

Alias:	None

Product:	Gentoo Linux
Classification:	Unclassified
Component:	[OLD] Library (show other bugs)
Hardware:	All Linux

Importance:	Normal normal (vote)
Assignee:	Ayatana Bugs

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:

Reported:	2015-07-06 13:40 UTC by Mario Kicherer
Modified:	2015-08-08 15:36 UTC (History)
CC List:	0 users

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

Attachments
Consider introspection use flag during configuration (patch,364 bytes, patch) 2015-07-06 13:41 UTC, Mario Kicherer	Details \| Diff
Corrected introspection flag handling and additional dependency (patch2,962 bytes, patch) 2015-07-09 12:40 UTC, Mario Kicherer	Details \| Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Mario Kicherer 2015-07-06 13:40:09 UTC

libappindicator has a "introspection" use flag but disabling it does not disable introspection during configuration and, hence, compilation fails if introspection is disabled system-wide.

Reproducible: Always

Comment 1 Mario Kicherer 2015-07-06 13:41:25 UTC

Created attachment 406246 [details, diff]
Consider introspection use flag during configuration

This patch should be applied to -r200 and -r300 ebuilds.

Comment 2 Mario Kicherer 2015-07-09 12:40:20 UTC

Created attachment 406400 [details, diff]
Corrected introspection flag handling and additional dependency

Better patch that includes a dependency towards gtk+[introspection?] and enables introspection only for the native ABI (thanks to mrueg for the hint). Again, applies to -r200 as well as -r300.

Comment 3 Mario Kicherer 2015-07-09 12:44:23 UTC

This patch should also fix bug #512976.

Comment 4 Mario Kicherer 2015-07-09 14:41:41 UTC

I created a pull request with a better patch: https://github.com/gentoo/gentoo-portage-rsync-mirror/pull/168

Comment 5 Michał Górny archtester

2015-08-08 15:36:27 UTC

  26 Jul 2015; Michał Górny <mgorny@gentoo.org>
  +files/libappindicator-12.10.0-conditional-py-bindings.patch,
  +libappindicator-12.10.0-r201.ebuild, +libappindicator-12.10.0-r301.ebuild,
  -files/libappindicator-12.10.0-disable-python.patch,
  libappindicator-12.10.0-r200.ebuild, libappindicator-12.10.0-r300.ebuild:
  Fix the build system in order to properly enable Python support. Fix
  introspection flag. https://github.com/gentoo/gentoo-portage-rsync-
  mirror/pull/168 by anyc.