Skip to content

Conversation

@GrahamCampbell
Copy link
Contributor

No description provided.

###############################################################################
# ICU
# We need to compile ICU from source because Amazon Linux 2 only has ICU 50.2
# which is cripplingly old and we want to support newer Unicode standards
Copy link
Member

Choose a reason for hiding this comment

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

do we know which minimum version we want or precisely which features (which will give us the version)?

I'm asking for #122

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The very latest version has the very latest unicode standard. We're not really looking for new "features", more so the implementation of the characters from the latest standard.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

https://cldr.unicode.org/downloads/cldr-47 is the standard implemented by 77.1.

@mnapoli
Copy link
Member

mnapoli commented Aug 15, 2025

@GrahamCampbell have you checked if that increases the layer size?

We already have intl.so at ~600KB, but no icu lib in /opt/libs. I'm afraid with this change we'll now have new lib that might be huge?

@GrahamCampbell
Copy link
Contributor Author

Let me find out. Stay tuned...

@GrahamCampbell
Copy link
Contributor Author

Oh daym. The image is double the size... That also means the PHP 8.5 is not good. Maybe we want to hold off on PHP 8.5 until AL 2023?

@mnapoli
Copy link
Member

mnapoli commented Aug 15, 2025

Yes that sounds reasonable then 👍

I will cleanup up #122 and update it with the code changes I have locally so that we can merge that into the v3 branch soon, and then work from v3.

@GrahamCampbell GrahamCampbell deleted the newer-icu branch August 15, 2025 14:17
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.

2 participants