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

Remove archives support in reltool #6836

Merged
merged 1 commit into from
Feb 22, 2023

Conversation

josevalim
Copy link
Contributor

reltool would package applications in archives
by default but there is little to no gain in
doing so. Furthermore, this ability pushed
complexity into other parts of Erlang/OTP,
such as the code server, erl_prim_loader, and
the init module.

This release makes it so archives are no longer
supported but the underlying options still warn
to avoid breaking changes. No functionality should
be removed from the assembled released (except
that they will have 100% fewer archives).

reltool would package applications in archives
by default but there is little to no gain in
doing so. Furthermore, this ability pushed
complexity into other parts of Erlang/OTP,
such as the code server, erl_prim_loader, and
the init module.

This release makes it so archives are no longer
supported but the underlying options still warn
to avoid breaking changes. No functionality should
be removed from the assembled released (except
that they will have 100% fewer archives).
@github-actions
Copy link
Contributor

github-actions bot commented Feb 9, 2023

CT Test Results

    2 files    14 suites   8m 58s ⏱️
149 tests 146 ✔️ 3 💤 0
165 runs  162 ✔️ 3 💤 0

Results for commit 82ee790.

♻️ This comment has been updated with latest results.

To speed up review, make sure that you have read Contributing to Erlang/OTP and that all checks pass.

See the TESTING and DEVELOPMENT HowTo guides for details about how to run test locally.

Artifacts

// Erlang/OTP Github Action Bot

@josevalim josevalim mentioned this pull request Feb 9, 2023
@josevalim
Copy link
Contributor Author

This should enable safe merging of #6683, #6777, and #6693.

Perhaps the best route forward is to merge this PR for Erlang/OTP 26 and schedule the three PRs above for OTP 27.

@rickard-green rickard-green added the team:VM Assigned to OTP team VM label Feb 13, 2023
@jhogberg jhogberg added the testing currently being tested, tag is used by OTP internal CI label Feb 16, 2023
@jhogberg
Copy link
Contributor

jhogberg commented Feb 16, 2023

We had a technical board meeting today and decided to merge this for OTP 26 RC2. We'll also add release notes in OTP 26 to warn people that the default code path choice will be changed to strict in OTP 27.

@jhogberg jhogberg merged commit 3ee5114 into erlang:master Feb 22, 2023
@jhogberg
Copy link
Contributor

Merged, thanks for the PR!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team:VM Assigned to OTP team VM testing currently being tested, tag is used by OTP internal CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants