Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 582638 - app-i18n/ibus doesn't call "dconf update" when installing
Summary: app-i18n/ibus doesn't call "dconf update" when installing
Status: CONFIRMED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: CJK Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-05-10 09:25 UTC by sh37092
Modified: 2017-08-02 20:59 UTC (History)
1 user (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 sh37092 2016-05-10 09:25:54 UTC
Hi. I found that I can't change any setting of ibus-pinyin. After some investigation, I found this issue relates to /etc/dconf/db/ibus. There are several lines in ebuild of ibus:

src_prepare() {
        ...

        # We run "dconf update" in pkg_postinst/postrm to avoid sandbox violations
        sed -e 's/dconf update/:/' \
                -i data/dconf/Makefile.{am,in} || die

        ...
}

I'm a Gentoo newbie, do we forget to call "dconf update"?
Comment 1 Akinori Hattori gentoo-dev 2017-08-01 14:28:38 UTC
https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=6ba4d6b62275049baa71d3fbc7ac408e8815fb4b

Thank you for reporting!
Comment 2 Mart Raudsepp gentoo-dev 2017-08-02 19:25:36 UTC
Absolutely nothing else in the tree does this. Are you sure this is correct..?
Comment 3 Pacho Ramos gentoo-dev 2017-08-02 19:44:00 UTC
Per:
https://codereview.appspot.com/51730044

We should take care of running it, also the sed line could be dropped

And I see Fedora and OpenSuSE packages running it too
Comment 4 Mart Raudsepp gentoo-dev 2017-08-02 20:59:51 UTC
yeah, if they fixed the package, no need to sed anymore. But I think it was already removed in the bump by hattya. The package itself mustn't do it, as we need it in postinst - per that link, it seems they've fixed it to skip it if DESTDIR is set, like other projects do (I looked at gdm earlier).

If we are failing to dconf update and it's needed, guess we'll need to track this on our own bug and get this fixed for all packages, similar to gconf schemas.
The current simple "dconf update" feels like something that might not honor prefix systems properly, but that's a detail we should figure out for gnome2-utils.eclass changes then.