Skip to content
This repository has been archived by the owner on Apr 7, 2018. It is now read-only.

Do not include Erlang library sources in the release #38

Merged
merged 1 commit into from Jul 9, 2014
Merged

Do not include Erlang library sources in the release #38

merged 1 commit into from Jul 9, 2014

Conversation

alco
Copy link
Contributor

@alco alco commented Jul 9, 2014

This cuts down 6 MB on the uncompressed size of the release.

This cuts down 6 MB on the uncompressed size of the release.
@alco
Copy link
Contributor Author

alco commented Jul 9, 2014

The release will still have include/ directories with .hrl files. I don't think those are needed for the release to work, but haven't found an option in relx to prune them.

bitwalker added a commit that referenced this pull request Jul 9, 2014
Do not include Erlang library sources in the release
@bitwalker bitwalker merged commit e24aec3 into bitwalker:master Jul 9, 2014
@bitwalker
Copy link
Owner

Good find! Thanks!

@tsloughter
Copy link

This really saves 6MB? I thought not including erts would save 6MB... but not dropping the source.

Also, right, there is no way to not include hrl files. It was discussed here erlware/relx#99

@alco
Copy link
Contributor Author

alco commented Jul 16, 2014

@tsloughter I checked again. Created a new Mix project and built release twice, with and without the sources:

$ du -hs rel_with_src
 32M    rel_with_src

$ du -hs rel
 26M    rel

@alco alco deleted the nosrc branch July 16, 2014 20:48
@tsloughter
Copy link

Oh wow, never mind then :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants