Closed Bug 1111352 Opened 9 years ago Closed 9 years ago

Incomplete keyboard layout can silently break keyboard app build

Categories

(Firefox OS Graveyard :: Gaia::Keyboard, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: gerard-majax, Assigned: rickychien)

References

Details

Attachments

(4 files)

Reproduced after building and updating my Nexus S, which is running current master.

On boot, logcat shows:
> 12-14 17:23:01.660    77    77 E GeckoConsole: Content JS ERROR: InputAppList: No input apps installed? 
> 12-14 17:23:01.660    77    77 E GeckoConsole:     at InputAppList.prototype._setInputApps (app://system.gaiamobile.org/shared/js/input_mgmt/input_app_list.js:273:4)
Maybe a build issue: comparing the list of applications in my profile with the latest update package I produced and this one, I see the keyboard application is missing.
Attached file Profile built
Attached file profile webapps.json
Please note we have a keyboard app defined in webapps.json, but there is no folder in the profile
Attached file Keyboard build error
Any idea why this build error did not resulted in a general build failure ?

|./build.sh gaia| succedded with this in the middle :(
Flags: needinfo?(timdream)
Flags: needinfo?(ricky060709)
The error mentions 'te' keyboard, introduced in bug 1046900.
Depends on: 1046900
Changing the title to properly describe the real issue. This is not the first time that it happens, can we take mitigations measures ?

Like, adding test case to make sure we do build all keyboard properly, etc. ?
Component: Gaia::System::Input Mgmt → Gaia::Keyboard
Summary: Keyboard not showing after update → Incomplete keyboard layout can silently break keyboard app build
Our Gb tests currently build with default config, you must be using some other build config.

That said, I agree with you we should create keyboard build script test for these configs.
Flags: needinfo?(timdream)
Yes, I have this in my .userconfig:
> export GAIA_KEYBOARD_LAYOUTS="$((find gaia/apps/keyboard/js/layouts/*.js | sed -e 's|gaia/apps/keyboard/js/layouts/||g' -e 's/\.js$//g') | tr -s '\r\n' ',' | sed -e 's/,$//g')"
Just in case it happens to anyone else, I had to remove objdir-gecko/, make a clean in gaia, and then rebuild a FOTA package to sideload. This made the keyboard application back. Then I had to go to Settings and reconfigure keyboards to get it back working.
Agree with Tim, adding keyboard integration test in build/test/integration for ensuring keyboard-related parameters (ex: GAIA_KEYBOARD_LAYOUTS) to work properly.
Flags: needinfo?(ricky060709)
The other issue is that when the app build step failed, the Gaia build will succeed with the package missing. That's a silent error we can't afford.

Ricky, could you setup the keyboard app build test based on script in comment 8, and also fix the issue above? Thanks. We probably need it before bug 1029951 can land.
Blocks: 1029951
Flags: needinfo?(ricky060709)
Sure! Maybe I'm not really understand how to verify keyboard-layout. I'll needinfo keyboard partners if I have further question.
Assignee: nobody → ricky060709
Status: NEW → ASSIGNED
Flags: needinfo?(ricky060709)
Update: Talked to Ricky offline. We will land a basic test asserting the default keyboard config here, and I will follow up with a test supporting build w/ all layouts in bug 1112460 (since a GAIA_KEYBOARD_LAYOUTS=* shortcut will be landed there). Ricky will clone another bug for fixing comment 11 (ensure the build failed when app build step fails).
As mentioned on comment 11, Bug 1112490 has been filed to address the silent error. And I'm going to land a test for default keyboard as soon as possible.
Attachment #8537738 - Flags: review?(gduan)
Comment on attachment 8537738 [details] [review]
Gaia PR for keyboard test

Please move it to apps/keyboard/test/build/integration and set r to me again, thanks!
Attachment #8537738 - Flags: review?(gduan)
Comment on attachment 8537738 [details] [review]
Gaia PR for keyboard test

PR updated! Please check it again thanks!
Attachment #8537738 - Flags: review?(gduan)
Comment on attachment 8537738 [details] [review]
Gaia PR for keyboard test

LGTM, r=gduan
Attachment #8537738 - Flags: review?(gduan) → review+
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: