Ticket #7818 (reopened defect)

Opened 6 years ago

Last modified 6 years ago

Load the XIM GTK+ input module conditionally

Reported by: sayamindu Owned by: mstone
Priority: high Milestone: 9.1.0-cancelled
Component: distro Version: not specified
Keywords: localization, keyboard 8.2-759:+ cjbfor9.1.0 Cc: cscott, dsd
Action Needed: diagnose Verified: no
Deployments affected: Blocked By:
Blocking:

Description

The XIM module, while partially enabling us to fix the Amharic Compose issue (#7474), seems creates problems for layouts in locales which do not have a glibc locale defined (eg: Dari, etc). We need Compose support only for Amharic right now (and also probably Khmer - which has not been tested very extensively so far), and so I propose that we load the XIM module conditionally.

It is to be noted however, that the module which is loaded by default (gtk-im-context-simple) does have support for commonly used Compose sequences (which are used in the Western world).

Attachments

olpc-utils_load_xim_conditionally.patch (1.0 kB) - added by sayamindu 6 years ago.
Patch to implement the behaviour

Change History

Changed 6 years ago by sayamindu

Patch to implement the behaviour

  Changed 6 years ago by sayamindu

  • blocking 7269 added

(In #7269) The patch attached to #7818 fixes the Dari keyboard.

  Changed 6 years ago by sayamindu

  • cc cscott added

follow-up: ↓ 4   Changed 6 years ago by cscott

sayamindu appears to have committed this patch to git; olpc-utils just needs rebuilding.

in reply to: ↑ 3   Changed 6 years ago by sayamindu

  • next_action changed from review to package

Replying to cscott:

sayamindu appears to have committed this patch to git; olpc-utils just needs rebuilding.

That's correct. I had submitted the build task in Koji when it went down - I am waiting for Koji to come up.

  Changed 6 years ago by dsd

  • cc dsd added

This is now in joyride. Please confirm that it works and supply a change log and test case, then move this bug into 'approve for release' state

  Changed 6 years ago by sayamindu

  • next_action changed from package to approve for release

It works.

|Test Case| In a machine with an Amharic keyboard, install the build switch to the Amharic layout in sugar

Verify (in the mesh search bar) that the sequence u h produces ሁ

  Changed 6 years ago by dsd

Please also confirm that "olpc-utils-0.86-1.olpc3" is the package in question? Scott will probably ask for a changelog entry too.

  Changed 6 years ago by cscott

  • next_action changed from approve for release to test in release

olpc-utils-0.86-1.olpc3 will be in 758 (it was requested in 8142); please test.

  Changed 6 years ago by sayamindu

olpc-utils-0.86-1.olpc3 contains the fix.

ChangeLog: dlo#7818: Load the XIM GTK Input Module conditionally

  Changed 6 years ago by dsd

Sayamindu, can you test this in 759? Or instruct someone else how it can be tested? This can then be moved into 'finalize' state.

  Changed 6 years ago by sayamindu

  • next_action changed from test in release to finalize

Works in 759.

  Changed 6 years ago by dsd

  • keywords 8.2-759:+ added

  Changed 6 years ago by gregorio

  • status changed from new to closed
  • resolution set to fixed

  Changed 6 years ago by kimquirk

  • status changed from closed to reopened
  • resolution deleted
  • milestone changed from 8.2.0 (was Update.2) to 8.2.1

Reopening as I am not getting the character that Sayamindu got in build 759 in the lastest, 760. Is it a regression or am I not testing this properly?

I get the character in a box FE on top and 75 below it. See #8127... the test case is the same for both and I don't think the result is correct. Moving this to 8.2.1.

  Changed 6 years ago by gregorio

  • next_action changed from finalize to diagnose

  Changed 6 years ago by mchua

  • blocking 7269 removed

(In #7269) Kim says: This has been closed. We need Dari speaking people to fully test all aspects of keyboard, but this specific bug is now closed. Removing blocker so that we can close it.

  Changed 6 years ago by mstone-xmlrpc

  • keywords cjbfor9.1.0 added
  • milestone changed from 8.2.1 to 9.1.0

Pushing out to 9.1.0, per edmcnierney's request.

Note: See TracTickets for help on using tickets.