Bug 478669

Summary: port imsettings-xfce to new xfconf backend
Product: [Fedora] Fedora Reporter: Kevin Fenzi <kevin>
Component: imsettingsAssignee: Akira TAGOH <tagoh>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: low    
Version: rawhideCC: christoph.wickert, i18n-bugs, tagoh
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2009-02-26 04:13:04 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Kevin Fenzi 2009-01-03 06:27:21 UTC
The upcoming Xfce 4.6 version will be dropping the old mcs config setup in favor of a new xfconf framework. 

I am intending to import 4.6b2 into rawhide soon.
It would be great if you could port the imsettings-xfce plugin to the new settings model. 

Some useful links: 
http://svn.xfce.org/index.cgi/xfce/xfconf/trunk/
http://mocha.xfce.org/~kelnos/xfconf/docs-current/

Happy to try and get more info or help from upstream...

Comment 1 Akira TAGOH 2009-01-05 02:08:30 UTC
Thank you for letting me know, Kevin. I'll work on this!

Comment 2 Christoph Wickert 2009-01-14 00:48:12 UTC
Akira, please forgive me if this is a stupid question:
What is imsettings-xfce actually needed for? Doesn't imsettings store it's config and load it on startup? It is started automatically through the normal fdo autostart and can be configured through the the icon in the tray, so is this package really needed?

Comment 3 Akira TAGOH 2009-01-14 01:51:50 UTC
That relies on xsettings to deliver IM configuration on demand for gtk+ applications at least and also uses the configuration in gconf that gnome-settings-daemon stored. which does take care of it on GNOME. since xfce has own xsettings manager and it doesn't support the key for immodule, we needed add-on to do that.
If adding the key to xfconf upstream is reasonable, that would be good. but I guess they don't like gconf at all?  Though I can put the configuration for xfce anywhere if you like. then xfconf just delivers it through xsettings. there shouldn't be additional dependencies there.

Otherwise separate component like this would be good to manage/maintain it.

Comment 4 Kevin Fenzi 2009-02-12 20:03:54 UTC
Any news here?

I am going to remove imsettings-xfce from the comps-f11 group for now, so people can get groupinstalls. We can add it back in if it's done in time for f11?

Or should we be looking at an ibus solution here?

Comment 5 Akira TAGOH 2009-02-13 01:22:08 UTC
Yes, please. that sounds reasonable. I'm still trying to have a certain time to work on this, but I didn't and may need more time to accomplish this.

ibus is an alternative solution for SCIM, but not imsettings FWIW.

Comment 6 Akira TAGOH 2009-02-18 10:06:19 UTC
After looking at xfce4-settings source code, there seems no way to provide a xsettings key outside XFCE because xfconf doesn't link to xsettings directly as xfce-mcs-manager did. so I've submitted a patch to suport it in xfce4-settings.

http://bugzilla.xfce.org/show_bug.cgi?id=4961

FYI

Comment 7 Akira TAGOH 2009-02-19 07:31:52 UTC
Should I file a bug for backporting a patch for our package, or do you want to wait for new release?

Comment 8 Kevin Fenzi 2009-02-19 21:21:58 UTC
We can do either one. Thats a pretty minor patch, so if it would help you, I can apply it now. 

On the other hand, hopefully 4.6 final will be out soon, so we can pick it up then. 

Which would you prefer?

Comment 9 Akira TAGOH 2009-02-20 02:03:00 UTC
Sure. I'll be ready to make a release, including this fix and others soon. if 4.6 final will be out before a mass-rebuild, I can wait for that then. otherwise backporting a patch would be appreciated. though I could just push new one without it and would be buildable. but not working properly without it.

Comment 10 Kevin Fenzi 2009-02-20 22:02:34 UTC
ok. I have just built a version with the patch included. 
Let me know if you spot any problems with it or need additional work done on it.

Comment 11 Akira TAGOH 2009-02-25 02:08:29 UTC
Fixed in imsettings-0.106.0-1.fc11.

Comment 12 Kevin Fenzi 2009-02-25 02:21:39 UTC
Excellent. ;) 

Feel free to re-add to comps and close this when you are sure the new version is working ok.

Comment 13 Christoph Wickert 2009-02-25 02:27:22 UTC
In fact you not really removed it from comps but only from the xfce-desktop group, it's still in the imput-methods group.

Comment 14 Akira TAGOH 2009-02-25 02:34:07 UTC
Right because it only prevents xfce installation or upgrading.

Comment 15 Kevin Fenzi 2009-02-26 03:12:27 UTC
ok, so this is back in. Shall we close this bug now?
Or is there any further reason to leave it open?

Comment 16 Akira TAGOH 2009-02-26 04:13:04 UTC
No. I was about to close this. thanks for reminding me.