Last Comment Bug 856475 - Warnings about harness-options.json missing from jetpack in an unmodified 1.14 download.
: Warnings about harness-options.json missing from jetpack in an unmodified 1.1...
Product: Graveyard
Classification: Graveyard
Component: Add-on Validation (show other bugs)
: unspecified
: All All
-- major
: 2013-05-30
Assigned To: Wraithan (Chris McDonald) [:wraithan]
Depends on:
  Show dependency treegraph
Reported: 2013-03-31 15:25 PDT by Wraithan (Chris McDonald) [:wraithan]
Modified: 2016-02-04 14:46 PST (History)
11 users (show)
See Also:
QA Whiteboard:
Iteration: ---
Points: ---

Addon that exhibits this behavior. (67.45 KB, application/x-xpinstall)
2013-03-31 16:13 PDT, Wes Kocher (:KWierso)
no flags Details

Description User image Wraithan (Chris McDonald) [:wraithan] 2013-03-31 15:25:54 PDT
Actual validator report:

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
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

Not sure if problem in validator or jetpack distribution. If it is validator I can fix it.
Comment 1 User image Wraithan (Chris McDonald) [:wraithan] 2013-03-31 15:43:22 PDT
`prefs.js` is covered by bug 826408, so it is mostly the `harness-options.json` that I am worried about.
Comment 2 User image Kris Maglione [:kmag] 2013-03-31 15:56:50 PDT
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.
Comment 3 User image Wes Kocher (:KWierso) 2013-03-31 16:13:39 PDT
Created attachment 731718 [details]
Addon that exhibits this behavior.
Comment 4 User image Wes Kocher (:KWierso) 2013-04-01 12:38:20 PDT
Alex, you made changes to somewhat recently. Any ideas here?
Comment 5 User image Alexandre Poirot [:ochameau] 2013-04-01 15:18:39 PDT
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 User image Alexandre Poirot [:ochameau] 2013-04-01 15:19:18 PDT
So I'm suggesting to get rid of this check:
Comment 7 User image Kris Maglione [:kmag] 2013-04-01 15:24:35 PDT
That's fine with me.
Comment 8 User image Wraithan (Chris McDonald) [:wraithan] 2013-04-01 15:55:04 PDT
Cool, consensus, I'll patch it and submit a pull request. Should go out this Thursday.
Comment 9 User image Fred Wenzel [:wenzel] 2013-05-23 09:40:30 PDT
Ping, hitting the same problem with my new add-on, and the error is a little confusing.

Note You need to log in before you can comment on or make changes to this bug.