Skip to content

Disable UPX in en locale#2411

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

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

dunglas commented May 13, 2026

Copy link
Copy Markdown
Member

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

@dunglas

dunglas commented Jul 18, 2026

Copy link
Copy Markdown
Member

Superseded by #2429, which makes UPX packing opt-in via COMPRESS=1 (off by default). The current docs already describe this, so NO_COMPRESS is no longer needed. Thanks!

@dunglas dunglas closed this Jul 18, 2026
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