Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 379447 - Various issues with dev-libs/libdbusmenu
Summary: Various issues with dev-libs/libdbusmenu
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] KDE (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Dennis Schridde
URL:
Whiteboard:
Keywords:
: 376571 (view as bug list)
Depends on:
Blocks:
 
Reported: 2011-08-16 17:32 UTC by Nirbheek Chauhan (RETIRED)
Modified: 2011-11-01 22:22 UTC (History)
2 users (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 Nirbheek Chauhan (RETIRED) gentoo-dev 2011-08-16 17:32:14 UTC
There are a number of issues with libdbusmenu. I'm filing one bug instead of separate bugs for each issue because each is relatively minor on its own.

(a) dev-libs/json-glib[introspection=] should be dev-libs/json-glib[introspection?], i.e. shouldn't force json-glib to be built without introspection with USE=-introspection.

(b) The use-flag description for introspection should be:

    <flag name="introspection">Use <pkg>dev-libs/gobject-introspection</pkg>
      for introspection</flag>

note the <pkg> tag.

(c) Please use EAPI 4 and REQUIRED_USE="vala? ( introspection )" instead of that silly eerror and die in pkg_setup(). When you move to EAPI 4, you can also drop src_install completely.

(d) You probably want to use dev-libs/glib:2[introspection?] in RDEPEND since the glib typelibs are needed at runtime by the libdbusmenu typelibs. That dep is currently just pulling in gobject-introspection, but that will change in the future.

(e) Please get yourself added as the maintainer in metadata.xml :)
Comment 1 Nirbheek Chauhan (RETIRED) gentoo-dev 2011-08-16 17:51:15 UTC
Another problem: the vala dep is most likely wrong. Unless it *only* compiles with vala:0, it should use the latest vala, i.e.:

vala? ( dev-lang/vala:0.12 )

and with:

[...]
        $(use_enable test tests) \
        $(use_enable vala) \
        VAPIGEN=$(type -p vapigen-0.12)

That will override the value of VAPIGEN from AC_PATH_PROG(VAPIGEN, vapigen) in configure.

The fact that all upstreams use an unversioned vapigen path is an unfortunate side-effect of the fact that all binary distros ship only one vala slot, and keep the binaries unversioned. See bug 338067 for the whole story.
Comment 2 Nirbheek Chauhan (RETIRED) gentoo-dev 2011-08-19 19:17:04 UTC
(In reply to comment #0)
> (d) You probably want to use dev-libs/glib:2[introspection?] in RDEPEND since
> the glib typelibs are needed at runtime by the libdbusmenu typelibs. That dep
> is currently just pulling in gobject-introspection, but that will change in the
> future.

This is no longer valid. I've masked the use-flag since it was pulling in gobject-introspection on too many systems.
Comment 3 Andreas K. Hüttel archtester gentoo-dev 2011-10-30 15:40:35 UTC
*** Bug 376571 has been marked as a duplicate of this bug. ***
Comment 4 Andreas K. Hüttel archtester gentoo-dev 2011-11-01 22:22:56 UTC
All done in 0.5.1