1438415 - Updater size of Nightly windows build is large

Status: RESOLVED FIXED

(Release Engineering :: Release Requests, defect)

Description

[Alice0775 White]

Steps to reproduce:
1. Launch Nightly
2. Help > About Nightly


Actual Results:
35MB - 45MB/each update

Expected Results:
10MB/each update or less

Comment 1

[Mark]

For example, extracting from my updates.xml file, the most recent 10 Nightly updates had these sizes for the Win64 platform and the en-US language:

   Build-ID     Partial   Complete  ->  Partial  Complete
                Size (B)  Size (B)       Size      Size
--------------  --------  --------      -------  --------
20180210220139  40281928  45353635  ->  38.4 MB  43.3 MB
20180211100408  32112120  45390035  ->  30.6 MB  43.3 MB
20180211220056  40326256  45397963  ->  38.5 MB  43.3 MB
20180212100154  28660952  45387051  ->  27.3 MB  43.3 MB
20180212220112  40299558  45333147  ->  38.4 MB  43.2 MB
20180213100127  40453504  45525487  ->  38.6 MB  43.4 MB
20180213220104  32222984  45504379  ->  30.7 MB  43.4 MB
20180214111549  40508416  45580399  ->  38.6 MB  43.5 MB
20180214220040  40543442  45577031  ->  38.7 MB  43.5 MB
20180214224814    n/a     45518175  ->    n/a    43.4 MB

Before a week ago the partials were usually in the 6 MB to 9 MB range, not in the 38 MB area.

Comment 2

[Mark]

Attachment: updates.xml from which my table was extracted

Comment 3

[WaltS48 [:walts48]]

Also getting large updates on Linux.

Today's was 41.7 MB. The full installer is 63.7 MB

Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Firefox/60.0 ID:20180215103933

Comment 4

[Chris AtLee [:catlee]]

I've noticed this as well.

Simon, would you mind taking a look?

Comment 5

[Simon Fraser [:sfraser] ⌚️GMT]

Mea culpa, I'd missed an implied return value in a shell script, and hadn't made it explicit when adding some commands after the one that generates the return value.

Comment 6

[Simon Fraser [:sfraser] ⌚️GMT]

Attachment: Further data about nightly partial sizes

Comment 7

[Simon Fraser [:sfraser] ⌚️GMT]

Attachment: Bug 1438415 Re-order mbsdiff hook to allow exit values

Review commit: https://reviewboard.mozilla.org/r/220750/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/220750/

Comment 8

[Chris AtLee [:catlee]]

Comment on attachment 8951441 [details]
Bug 1438415 Re-order mbsdiff hook to allow exit values

https://reviewboard.mozilla.org/r/220750/#review226668

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/412e0f6a151c
Re-order mbsdiff hook to allow exit values r=catlee

Comment 10

[Narcis Beleuzu [:NarcisB]]

https://hg.mozilla.org/mozilla-central/rev/412e0f6a151c