Skip to content

[Intl] Remove resources data from classmap generation #54625

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

Merged
merged 1 commit into from
Apr 16, 2024

Conversation

shyim
Copy link
Contributor

@shyim shyim commented Apr 16, 2024

Q A
Branch? 5.4
Bug fix? yes
New feature? no
Deprecations? no
Issues Fix #...
License MIT

I was annoyed why my composer install takes 5 seconds to complete, so I debugged and the most time-consuming was optimized autoloader generation. Looking more deep into, I figured out that Composer looks into any of the "Data PHP" files of Intl. Removed it and it make it much faster for me. It has no negativ effect to Intl component removing them from classmap, as they don't contain something autoloadable.

composer dumpautoload -o on my project with 137 packages

Before: 4.662s
After: 1.833s

@fabpot
Copy link
Member

fabpot commented Apr 16, 2024

Thank you @shyim.

@fabpot fabpot merged commit 7a52d76 into symfony:5.4 Apr 16, 2024
10 of 12 checks passed
@shyim shyim deleted the fix-composer-autoload-speed branch April 17, 2024 07:06
@stof
Copy link
Member

stof commented Apr 17, 2024

Please do the same in the 7.1 branch for the Emoji component.

fabpot added a commit that referenced this pull request Apr 17, 2024
…shyim)

This PR was merged into the 7.1 branch.

Discussion
----------

[Emoji] Remove resources data from classmap generation

| Q             | A
| ------------- | ---
| Branch?       | 7.1
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Issues        |
| License       | MIT

Same as #54625 but for Emoji,

FYI: `@stof`

Commits
-------

fddce25 [Emoji] Remove resources data from classmap generation
This was referenced Apr 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants