Closed
Bug 856475
Opened 11 years ago
Closed 11 years ago
Warnings about harness-options.json missing from jetpack in an unmodified 1.14 download.
Categories
(addons.mozilla.org Graveyard :: Add-on Validation, defect)
addons.mozilla.org Graveyard
Add-on Validation
Tracking
(Not tracked)
RESOLVED
FIXED
2013-05-30
People
(Reporter: wraithan, Assigned: wraithan)
References
()
Details
Attachments
(1 file)
67.45 KB,
application/x-xpinstall
|
Details |
Actual validator report: https://addons.mozilla.org/en-US/developers/addon/fightcode-utils/file/200874/validation Warnings: Missing Jetpack module Warning: A Jetpack module listed in harness-options.json could not be found in the add-on. Path: resources/fightcode-utils/main harness-options.json Jetpack module version mismatch Warning: A file in the Jetpack add-on does not match the SDK version specified in harness-options.json. Module: defaults/preferences/prefs.js Versions: 1.14/1.5-dev defaults/preferences/prefs.js Not sure if problem in validator or jetpack distribution. If it is validator I can fix it.
Assignee | ||
Comment 1•11 years ago
|
||
`prefs.js` is covered by bug 826408, so it is mostly the `harness-options.json` that I am worried about.
Comment 2•11 years ago
|
||
Can you upload the add-on that this is happening with? We're probably going to be getting a lot of submissions with Jetpack 1.14 in the coming days, so we need to address this as soon as possible.
Severity: normal → major
Assignee | ||
Updated•11 years ago
|
Summary: Warnings about 2 files missing from jetpack in an unmodified 1.14 download. → Warnings about harness-options.json missing from jetpack in an unmodified 1.14 download.
Alex, you made changes to manifest.py somewhat recently. Any ideas here?
Comment 5•11 years ago
|
||
Oh I haven't thought about the possible breakage of amo-validator when modifying the manifest... So what happens is that we changed a bit the manifest key. It isn't any longer the path in the xpi. For the addon attached in this bug, we have a main.js in /resources/kwierso-201/lib/main But here is how looks like the manifest: "manifest": { "kwierso-201/main": { "docsSHA256": null, "jsSHA256": "58771c4c07cbf2228c493da788d1f77d7724873d1e871f4b9b3fe68247bd9f36", "moduleName": "main", "packageName": "kwierso-201", "requirements": {}, "sectionName": "lib" } } The second important point is that we striped down the manifest. It now only contains addon modules (and no longer SDK modules). We still ship SDK modules in the xpi, but starting with FF21, modules shipped in Firefox will be used instead of xpi ones. That's to be compatible with FF20-. Otherwise, we will most likely stop shipping SDK modules in to SDK release, and also eventually simplify xpi layout to match source code layout. Having said that, I'm not sure it is that relevant to check for addon module checksum? It was more important for SDK modules, but not as important for addon ones.
Comment 6•11 years ago
|
||
So I'm suggesting to get rid of this check: https://github.com/mozilla/amo-validator/blob/master/validator/testcases/jetpack.py#L177-L188
Comment 7•11 years ago
|
||
That's fine with me.
Assignee | ||
Comment 8•11 years ago
|
||
Cool, consensus, I'll patch it and submit a pull request. Should go out this Thursday.
Comment 9•11 years ago
|
||
Ping, hitting the same problem with my new add-on, and the error is a little confusing.
Assignee | ||
Comment 10•11 years ago
|
||
https://github.com/mozilla/amo-validator/commit/8893e3817b7a9ab559fbd0a90ea0ab30f56c4b9b
Assignee: nobody → wraithan
Target Milestone: --- → 2013-05-30
Assignee | ||
Updated•11 years ago
|
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•8 years ago
|
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•