cordova-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF subversion and git services (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (CB-4495) start-emulator script doesn't fail (exit) early on fatal emulator errors
Date Thu, 15 Aug 2013 15:49:48 GMT

    [ https://issues.apache.org/jira/browse/CB-4495?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13741092#comment-13741092
] 

ASF subversion and git services commented on CB-4495:
-----------------------------------------------------

Commit 121b74fa0c8b5c2aea25173d128b5745c3b85390 in branch refs/heads/master from [~kami]
[ https://git-wip-us.apache.org/repos/asf?p=cordova-android.git;h=121b74f ]

[CB-4495] Modify start-emulator script to exit immediately on a fatal emulator error.

                
> 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
>         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 is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Mime
View raw message