Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 542376 - app-i18n/mozc-2.16.2037.102: Fix qt4 USE flag: USE="fcitx qt4 -ibus" forces ibus and ibus-qt dependencies
Summary: app-i18n/mozc-2.16.2037.102: Fix qt4 USE flag: USE="fcitx qt4 -ibus" forces i...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: CJK Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-03-06 20:10 UTC by hrnr
Modified: 2015-03-18 02:33 UTC (History)
0 users

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


Attachments
mozc-2.16.2037.102-qt4.patch (mozc-2.16.2037.102-qt4.patch,456 bytes, patch)
2015-03-06 20:10 UTC, hrnr
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description hrnr 2015-03-06 20:10:22 UTC
Created attachment 398258 [details, diff]
mozc-2.16.2037.102-qt4.patch

When I try to compile in USE="fcitx qt4 -ibus", also tries to compile "app-i18n/ibus" and "app-i18n/ibus-qt".
But it is not depend on the "app-i18n/ibus" and "app-i18n/ibus-qt".

Could please fix it?

Please find attached patch so that it will compile against USE="fcitx qt4 -ibus".


I'm sorry I reported https://bugs.gentoo.org/show_bug.cgi?id=542070 that overlooked this.

Thanks,
Comment 1 Andrew Savchenko gentoo-dev 2015-03-08 20:47:24 UTC
It's up to maintainer to decide, but frankly patch looks wrong to me: you have app-i18n/ibus-qt dependency twice in RDEPEND:

ibus? ( qt4? ( app-i18n/ibus-qt ) )
qt4? ( app-i18n/ibus-qt )

It should be only at one of this places. If qt4 can be compiled without ibus-qt, remove the second encounter.
Comment 2 hrnr 2015-03-09 02:56:16 UTC
(In reply to Andrew Savchenko from comment #1)
> It's up to maintainer to decide, but frankly patch looks wrong to me: you
> have app-i18n/ibus-qt dependency twice in RDEPEND:
> 
> ibus? ( qt4? ( app-i18n/ibus-qt ) )
> qt4? ( app-i18n/ibus-qt )
> 
> It should be only at one of this places. If qt4 can be compiled without
> ibus-qt, remove the second encounter.

My attached patch: https://bugs.gentoo.org/attachment.cgi?id=398258&action=diff

ibus? ( >=app-i18n/ibus-1.4.1  qt4? ( app-i18n/ibus-qt ) )
qt4? ( dev-qt/qtgui:4 app-i18n/zinnia )

app-i18n/ibus-qt dependency is only one in RDEPEND.
Comment 3 Andrew Savchenko gentoo-dev 2015-03-09 08:28:08 UTC
Oh, sorry, I overlooked "-" in front of a line.
Comment 4 Naohiro Aota gentoo-dev 2015-03-18 02:33:45 UTC
Thanks for the patch. Applied!

+  18 Mar 2015; Naohiro Aota <naota@gentoo.org> mozc-2.16.2037.102.ebuild:
+  Fix ibus-qt dependency suggested by hrnr. #542376