[
https://issues.apache.org/jira/browse/CB-4495?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13802155#comment-13802155
]
ASF subversion and git services commented on CB-4495:
-----------------------------------------------------
Commit bb7bc33a8ab2a12bcd5540b4b6420b47a9ee76c9 in branch refs/heads/2.9.x from
[~kami]
[ https://git-wip-us.apache.org/repos/asf?p=cordova-android.git;h=bb7bc33 ]
[CB-4495] Modify start-emulator script to exit immediately on a fatal emulator
error.
(cherry picked from commit 121b74fa0c8b5c2aea25173d128b5745c3b85390)
> start-emulator script doesn't fail (exit) early on fatal emulator errors
> ------------------------------------------------------------------------
>
> Key: CB-4495
> URL: https://issues.apache.org/jira/browse/CB-4495
> Project: Apache Cordova
> Issue Type: Improvement
> Components: Android
> Affects Versions: 3.0.0
> Reporter: Tomaz Muraus
> Assignee: Andrew Grieve
> Priority: Minor
> Fix For: 3.1.0
>
> Attachments:
> 0001-Modify-start-emulator-script-to-exit-immediately-on-.patch
>
>
> Problem
> (https://github.com/apache/cordova-android/pull/77)
> Currently start-emulator script doesn't exist early on Linux if a fatal error
> happens.
> This means you need to wait 300 seconds (default timeout) before you receive
> first feedback from the cli tool that something is wrong. An even then (after
> a timeout), it's hard to debug the problem because adb stdout and stderr are
> simply redirected /dev/null. This makes it for a bad UI and a frustrated user.
> For example:
> ....
> Waiting for emulator.....
> If I run the emulator manually I see it produces a fatal error:
> emulator: ERROR: unknown skin name 'WVGA800'
> Proposed Solution
> This branch does the following:
> Modify the emulator start script to redirect log to a temporary file. This
> way it's easier to debug actual issues.
> Modify wait_for_emulator function to exit immediately and print an error if a
> fatal error is detected while starting the emulator
> Example output with my change:
> Saving emulator log to: /tmp/tmp.VTOblrPktV
> Emulator failed to start, fatal error detected
> Error: emulator: ERROR: unknown skin name 'WVGA800'
> Full log available at: /tmp/tmp.VTOblrPktV
> Exiting...
> Github pull request with a patch for the proposed solution is available at
> https://github.com/apache/cordova-android/pull/77. Once people are pleased
> with the patch, I'll attach final version to this ticket.
--
This message was sent by Atlassian JIRA
(v6.1#6144)
