Hi guys and sorry for not getting back sooner... On 2013-05-19 01:29, Osamu Aoki wrote: > Hi, > > Now that freeze is over, let's think this change of "-l" option. > > On Mon, Dec 17, 2012 at 08:42:57AM +0100, Gunnar Hjalmarsson wrote: >> I decided to give the code reuse idea a shot, and the result is the >> attached patch. It's not much that can be done, really. >> >> Better than the first patch? I'm not sure. > > In my quick glange, it looks basically fine.
Please let us talk about the third variant I submitted: http://bugs.debian.org/cgi-bin/bugreport.cgi?msg=45;filename=im-config-installed-IMs-3.patch;att=1;bug=694941 It does the same thing, but the code is slightly neater. > But before applying this, let's discuss the use case of this patch. > > Is this output used by external script? > -> some gui tools which try to set im-config via > non interactive -n option. Yes, that's what the Language Support GUI in Ubuntu does. This is the function in the Language Support source that makes use of the -l option: def getAvailableInputMethods(self): inputMethods = subprocess.check_output(['im-config', '-l']).decode().split() return ['default'] + sorted(inputMethods) So the list presented to the user consists of: - default (i.e. auto mode) - xim (but it's labelled 'none' in the GUI) - the installed input method systems > Maybe we need to expose priority, install state of required package, ... Not sure what you mean by that. But I can say that I think the current -l option serves the needs of Ubuntu. -- Gunnar Hjalmarsson https://launchpad.net/~gunnarhj -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
