983406 - TEST-UNEXPECTED-FAIL: setup.js | error parsing http://mochi.test:8888/androidx86.json

Status: RESOLVED FIXED

(Testing :: General, defect)

Description

[Geoff Brown [:gbrown]]

Possible fallout from the recent .json/.ini changes.

https://tbpl.mozilla.org/php/getParsedLog.php?id=36093528&tree=Ash&full=1#error0

http://mozilla-releng-blobs.s3.amazonaws.com/blobs/ash/sha512/5c736bcc8884f0dbd2312d854905ffe409cee8a84b4f1c156be0636534a413a272f21ec7110be02fa9575720a5bd4d8ae0cc5daf866d9b683d936593f2b290ec

03-13 21:27:22.542 E/Profiler( 2047): BPUnw: [10 total] thread_register_for_profiling(me=0xb8518948, stacktop=0x971ffc6f)
03-13 21:27:22.542 W/TextureGenerator( 2047): Clearing GL error: 0x500
03-13 21:27:22.562 I/GeckoDump( 2047): TEST-UNEXPECTED-FAIL: setup.js | error parsing http://mochi.test:8888/androidx86.json (SyntaxError: JSON.parse: expected ',' or '}' after property value in object at line 50 column 3 of the JSON data)
03-13 21:27:22.562 E/GeckoConsole( 2047): [JavaScript Error: "SyntaxError: JSON.parse: expected ',' or '}' after property value in object at line 50 column 3 of the JSON data" {file: "http://mochi.test:8888/manifestLibrary.js" line: 46}]

Comment 1

[Geoff Brown [:gbrown]]

Attachment: add a missing comma to androidx86.json

Comment 2

[Joel Maher ( :jmaher ) (UTC -8)]

Comment on attachment 8390841 [details] [diff] [review]
add a missing comma to androidx86.json

Review of attachment 8390841 [details] [diff] [review]:
-----------------------------------------------------------------

apologies for breaking this, thanks for the patch!

Comment 3

[Geoff Brown [:gbrown]]

No worries -- the price of progress!

https://hg.mozilla.org/integration/mozilla-inbound/rev/047a0b9a8fb8

Comment 4

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/047a0b9a8fb8