Skip to content

Disable UPX in en locale#2411

Open
andreybolonin wants to merge 4 commits into
php:mainfrom
andreybolonin:patch-3
Open

Disable UPX in en locale#2411
andreybolonin wants to merge 4 commits into
php:mainfrom
andreybolonin:patch-3

Conversation

@andreybolonin
Copy link
Copy Markdown

No description provided.

Signed-off-by: Andrey Bolonin <andreybolonin@users.noreply.github.com>
Comment thread docs/embed.md Outdated
Co-authored-by: Marc <m@pyc.ac>
Signed-off-by: Andrey Bolonin <andreybolonin@users.noreply.github.com>
Comment thread docs/embed.md
Signed-off-by: Marc <m@pyc.ac>
Comment thread docs/embed.md Outdated
Signed-off-by: Marc <m@pyc.ac>
Comment thread docs/embed.md
Comment on lines 149 to 150
On macOS, to reduce the size of the file before sending it, you can compress it.
We recommend `xz`.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is this actually still true?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

I am not author of this

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I know, but if we're touching the subject of compression, you say it's default on on macOS, the docs say it's not, and we're touching the docs, we should fix that part too :)

I'm asking because I don't have macOS to check it quickly.

@dunglas
Copy link
Copy Markdown
Member

dunglas commented May 13, 2026

Actually, I think we shouldn't enable UPX by default: dunglas/mercure#1182

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.

3 participants