Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prevent release from generating ez archives #5073

Merged
merged 1 commit into from
May 31, 2024

Conversation

nickva
Copy link
Contributor

@nickva nickva commented May 31, 2024

Erlang 26 removed the ability to create ez archive files in releases. We had to remove the excl_archive_filters option from our reltool.config file [1], as otherwise we couldn't make releases on 26 (make release barfed). However, that meant on 24 and 25 our releases would be generated with beam files packed in lots of silly .ez archives, which is not great.

Sadly, since 26, refuses to silently ignore excl_archive_filters we have to use a script file and only apply the config on <26 versions.

[1] a4bbb87

@nickva
Copy link
Contributor Author

nickva commented May 31, 2024

To test, make two copies of the src directory, one with asdf shell erlang 26 the other with 25. ./configure && make && make release in both and check that re/couchdb/lib/* doesn't have .ez archives.

Instead of

% ls lib
asn1-5.0.18.2/                                  hyper-1.0/
asn1-5.0.18.2.ez                                hyper-1.0.ez
b64url-3.3.2.post1-440-ga326cdc53/              ibrowse-4.4.2/
b64url-3.3.2.post1-440-ga326cdc53.ez            ibrowse-4.4.2.ez

It should look like

 % ls rel/couchdb/lib
asn1-5.0.18.2/                               hyper-1.0/
b64url-3.3.2.post1-440-ga326cdc53/           ibrowse-4.4.2/
chttpd-3.3.2.post1-440-ga326cdc53/           inets-7.5.3.4/

@nickva nickva requested a review from jaydoane May 31, 2024 05:06
@nickva nickva force-pushed the fix-releases-no-more-ez-archive branch from 43292ac to 2311015 Compare May 31, 2024 05:10
Copy link
Contributor

@jaydoane jaydoane left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested on both OTP 25 and 26, and confirmed that applications in rel/couchdb/lib/ did not contain .ez archives.

rel/reltool.config.script Outdated Show resolved Hide resolved
Erlang 26 removed the ability to create ez archive files in releases. We had to
remove the `excl_archive_filters` option from our reltool.config file [1], as
otherwise we couldn't make releases on 26 (`make release` barfed). However,
that meant on 24 and 25 our releases are generated with beam files packed
in lots of silly .ez archives, which is not great.

Sadly, since 26, refuses to silently ignore `excl_archive_filters` option, we
have to use a script file and only apply the config on <26 versions.

[1] a4bbb87
@nickva nickva force-pushed the fix-releases-no-more-ez-archive branch from 2311015 to 4f87898 Compare May 31, 2024 06:08
@nickva nickva merged commit 9070a64 into main May 31, 2024
23 checks passed
@nickva nickva deleted the fix-releases-no-more-ez-archive branch May 31, 2024 06:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants