Issue 6560 - fix malformed updates.json

Issue 6560 - fix malformed updates.json

[tlucas, 2018-04-06 09:08:42]

Patch Set 1:
Fix generation of malformed updates.json for developement builds for Adblock
Plus for Firefox.

[kzar, 2018-04-06 09:55:49]

I had not heard of the abpssembly repository at all, kind of curious how come
that exists?

https://codereview.adblockplus.org/29744579/diff/29744580/sitescripts/extensi...
File sitescripts/extensions/template/gecko.json (right):

https://codereview.adblockplus.org/29744579/diff/29744580/sitescripts/extensi...
sitescripts/extensions/template/gecko.json:14: }
If there are multiple extensions won't we be missing the commas between the
entries here?

[tlucas, 2018-04-06 10:08:32]

Patch Set 2
* Addressed Dave's comment

On 2018/04/06 09:55:49, kzar wrote:
> I had not heard of the abpssembly repository at all, kind of curious how come
> that exists?

`abpssembly` is a fork of sitescripts, meant to someday split up the builds and
the rest of sitescripts :)

https://codereview.adblockplus.org/29744579/diff/29744580/sitescripts/extensi...
File sitescripts/extensions/template/gecko.json (right):

https://codereview.adblockplus.org/29744579/diff/29744580/sitescripts/extensi...
sitescripts/extensions/template/gecko.json:14: }
On 2018/04/06 09:55:49, kzar wrote:
> If there are multiple extensions won't we be missing the commas between the
> entries here?

Good point, fixed it.

[kzar, 2018-04-06 10:14:05]

LGTM