@mattab opened this Issue on June 19th 2015 Owner

Between beta1 and beta2 the Piwik ZIP package grew 15Mb / 100% to 30Mb. See:

regressed size

let's investigate and make package smaller again, 30Mb is not reasonable.

@diosmosis commented on June 19th 2015 Member

The tests folder is 24MB, should it be in betas?

@mnapoli commented on June 19th 2015 Member

TCPDF is now installed with Composer (was previously included in the repo manually IIRC): https://github.com/piwik/piwik/blob/master/composer.json#L58

This dependency includes 27Mb of fonts. I guess that explains the difference. I don't know if we use them though, might require to carefully remove them. Also there are 2Mb of examples in there that we can get rid of.

@mattab commented on June 20th 2015 Owner

The tests folder is 24MB, should it be in betas?

all files in tests/* are deleted from the releases

This dependency includes 27Mb of fonts. I guess that explains the difference.

That explains it!! we will need to update the build script https://github.com/piwik/piwik-package/blob/master/scripts/build-package.sh to remove unused fonts cc @sgiehl in case you know which ones we need

Also created a follow up issue: automated test to detect when the Piwik files become too big #8158

@sgiehl commented on June 20th 2015 Member

See https://github.com/piwik/piwik/blob/master/core/ReportRenderer/Pdf.php#L93-L132
I guess all existing variants of the font we use here, could be removed aswell.

@mattab commented on June 22nd 2015 Owner

done in: https://github.com/piwik/piwik-package/issues/33#issuecomment-113996050

2.14.0-b6 is back to being 16M :+1:

refs #7908

@mattab commented on June 22nd 2015 Owner

it's still 3Mb bigger than 2.13.1 but after looking hard I can't find any more way to make package size smaller

This Issue was closed on June 22nd 2015
Powered by GitHub Issue Mirror