{"action":"created","issue":{"url":"https://api.github.com/repos/NixOS/nixpkgs/issues/14417","repository_url":"https://api.github.com/repos/NixOS/nixpkgs","labels_url":"https://api.github.com/repos/NixOS/nixpkgs/issues/14417/labels{/name}","comments_url":"https://api.github.com/repos/NixOS/nixpkgs/issues/14417/comments","events_url":"https://api.github.com/repos/NixOS/nixpkgs/issues/14417/events","html_url":"https://github.com/NixOS/nixpkgs/pull/14417","id":145448534,"number":14417,"title":"[WIP] input methods: use GTK3_IM_MODULE_FILE","user":{"login":"ericsagnes","id":367880,"avatar_url":"https://avatars.githubusercontent.com/u/367880?v=3","gravatar_id":"","url":"https://api.github.com/users/ericsagnes","html_url":"https://github.com/ericsagnes","followers_url":"https://api.github.com/users/ericsagnes/followers","following_url":"https://api.github.com/users/ericsagnes/following{/other_user}","gists_url":"https://api.github.com/users/ericsagnes/gists{/gist_id}","starred_url":"https://api.github.com/users/ericsagnes/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ericsagnes/subscriptions","organizations_url":"https://api.github.com/users/ericsagnes/orgs","repos_url":"https://api.github.com/users/ericsagnes/repos","events_url":"https://api.github.com/users/ericsagnes/events{/privacy}","received_events_url":"https://api.github.com/users/ericsagnes/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/NixOS/nixpkgs/labels/2.status:%20work-in-progress","name":"2.status: work-in-progress","color":"e6e6e6"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":19,"created_at":"2016-04-03T03:27:34Z","updated_at":"2016-04-03T16:52:08Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/NixOS/nixpkgs/pulls/14417","html_url":"https://github.com/NixOS/nixpkgs/pull/14417","diff_url":"https://github.com/NixOS/nixpkgs/pull/14417.diff","patch_url":"https://github.com/NixOS/nixpkgs/pull/14417.patch"},"body":"**Work in progress, please do not merge yet.**\r\n\r\nThis is an implementation of #14019 fix.\r\n\r\n@ttuegel Could you review this PR code?\r\n\r\nA few personal concerns:\r\n\r\n- ~~Using a systemd unit to generate the cache files doesn't feel right, is there a better way to do that?~~ solved in the second commit\r\n- ~~Having an option (`i18n.inputMethod.package`) just to pass the input method package doesn't feel right.~~ solved by setting `internal = true`\r\n\r\nNote: \r\n- Testing this PR will trigger a quite long rebuild of the gtk+3 package.\r\n- I add commits to keep track of progress, all commits will be merged into one before merging."},"comment":{"url":"https://api.github.com/repos/NixOS/nixpkgs/issues/comments/205009061","html_url":"https://github.com/NixOS/nixpkgs/pull/14417#issuecomment-205009061","issue_url":"https://api.github.com/repos/NixOS/nixpkgs/issues/14417","id":205009061,"user":{"login":"abbradar","id":1174810,"avatar_url":"https://avatars.githubusercontent.com/u/1174810?v=3","gravatar_id":"","url":"https://api.github.com/users/abbradar","html_url":"https://github.com/abbradar","followers_url":"https://api.github.com/users/abbradar/followers","following_url":"https://api.github.com/users/abbradar/following{/other_user}","gists_url":"https://api.github.com/users/abbradar/gists{/gist_id}","starred_url":"https://api.github.com/users/abbradar/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/abbradar/subscriptions","organizations_url":"https://api.github.com/users/abbradar/orgs","repos_url":"https://api.github.com/users/abbradar/repos","events_url":"https://api.github.com/users/abbradar/events{/privacy}","received_events_url":"https://api.github.com/users/abbradar/received_events","type":"User","site_admin":false},"created_at":"2016-04-03T16:52:08Z","updated_at":"2016-04-03T16:52:08Z","body":"@ericbmerritt Thanks! Let's see what upstream thinks.\r\n\r\nFrom another look (brief, though) those variables can be actually useful to have GTK3_ variants:\r\n\r\n* GTK_IM_MODULE_FILE\r\n* GTK_THEME\r\n* GTK_DATA_PREFIX\r\n* GTK_PATH\r\n* GTK_IM_MODULE\r\n\r\nNotice that I haven't included `GTK_EXE_PREFIX` because it's visible in the patch that GTK uses it with `/gtk-3.0` postfix, so it should be fine even when co-used with GTK2. Maybe I'm missing something, though.\r\n\r\nOthers are not really important and/or mostly for debugging, I think.\r\n\r\nNotice that I don't have any knowledge on GTK topics and this is just me reading the patch and concluding what a variable does from its name!"}}