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

Add news post about Erlang/OTP 28, release candidate 1 #164

Merged
merged 2 commits into from
Feb 12, 2025

Conversation

bjorng
Copy link
Contributor

@bjorng bjorng commented Feb 10, 2025

No description provided.

Copy link

netlify bot commented Feb 10, 2025

Deploy Preview for erlang-org ready!

Name Link
🔨 Latest commit a07ade9
🔍 Latest deploy log https://app.netlify.com/sites/erlang-org/deploys/67ac8580a9849e000864e485
😎 Deploy Preview https://deploy-preview-164--erlang-org.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Contributor

@garazdawi garazdawi left a comment

Choose a reason for hiding this comment

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

lgtm. Here is the generated new item if you want to read that: https://deploy-preview-164--erlang-org.netlify.app/news/174

_news/174.md Outdated Show resolved Hide resolved
_news/174.md Outdated Show resolved Hide resolved
_news/174.md Outdated
[`sets`](https://www.erlang.org/doc/apps/stdlib/sets.html)
will now be represented as maps.

- Module [`re`](https://www.erlang.org/doc/apps/stdlib/re.html) has
Copy link

Choose a reason for hiding this comment

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

Should this link (and others) point to the new documentation for rc1?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, ideally, but I am not sure where this documentation will be. Maybe it's easier to adjust the links when the release has been built? Any thoughts, @proxyles or @garazdawi?

Copy link
Contributor

Choose a reason for hiding this comment

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

As the rc1 docs will have the version number of the app in them, it is easier to just update when the docs are published.

@kikofernandez
Copy link
Contributor

We should let our users know that, in the Github Release page, we have included a source Software Bill of Materials (source SBOM) and would like feedback on it. But this is not included in Erlang/OTP per se, just in the Release page and generated by a Github Action

@kikofernandez
Copy link
Contributor

@bjorng It is tricky to add this as a highlight in the OTP ticket, because the source SBOM is not in the Erlang/OTP repo, but an addition outside the OTP repo.

@bjorng
Copy link
Contributor Author

bjorng commented Feb 11, 2025

@kikofernandez I've pushed a fixup commit with a new paragraph about SBOMs. I included a link to an article I found that explains the concept of SBOMs.

@bjorng
Copy link
Contributor Author

bjorng commented Feb 11, 2025

@kikofernandez The link I found is on Gitlab and contains some irrelevant material (how Gitlab helps making SBOMs). Feel free to suggest another link.

@bjorng
Copy link
Contributor Author

bjorng commented Feb 11, 2025

@garazdawi There is one broken link:

Getting links from: http://localhost:4000/patches/OTP-27.2
[2025-02-11 07:35:37] ERROR `/patches/kernel_app.md' not found.
├─BROKEN─ http://localhost:4000/patches/kernel_app.md#os_cmd_shell (HTTP_404)
Finished! 712 links found. 711 excluded. 1 broken.

This doesn't seem to be caused by my PR, so it seems to be broken already in master.

_news/174.md Outdated Show resolved Hide resolved
- Some BIFs with side-effects (such as `binary_to_atom/1`) are
optimized in `try` ... `catch` in the same way as guard BIFs in
order to gain performance.

Copy link

Choose a reason for hiding this comment

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

Suggested addition:

  • Improved performance of the compiler's alias analysis pass to roughly halve the time required for the analysis. The improved performance allows improvements in the analysis to handle chains of extracts where it previously it only handled the innermost embed+extract pair and conservatively classified longer chains as producing an aliased value. (#8695)

Copy link
Contributor Author

@bjorng bjorng Feb 11, 2025

Choose a reason for hiding this comment

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

I added an edited version of your suggestion, which is lighter on technical details and also pointing what the benefit is for the user:

  • The compiler’s alias analysis pass is now both faster and less conservative, allowing optimizations of records and binary construction to be applied in more cases.

I've created a reminder for myself to add a release note for RC2 about compiler optimizations.

Copy link

Choose a reason for hiding this comment

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

I like what you did with my blurb!

@proxyles proxyles merged commit 9c34581 into erlang:master Feb 12, 2025
5 checks passed
@bjorng bjorng deleted the bjorn/otp28-rc1 branch February 12, 2025 13:56
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.

8 participants