8000 Installing the ClassLoader component with Composer makes installing the ClassLoader component largely irrelevant · Issue #5949 · symfony/symfony-docs · GitHub
[go: up one dir, main page]
Skip to content

Installing the ClassLoader component with Composer makes installing the ClassLoader component largely irrelevant #5949

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.

Sign up for GitHub

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

Jump to bottom
Closed
hpierce1102 opened this issue Dec 4, 2015 · 1 comment
Closed

Installing the ClassLoader component with Composer makes installing the ClassLoader component largely irrelevant #5949

hpierce1102 opened this issue Dec 4, 2015 · 1 comment
Labels
hasPR A Pull Request has already been submitted for this issue.

Comments

@hpierce1102
Copy link

Installing the ClassLoader component by using Composer brings in Composer's own ClassLoader, which has significant overlap with Symfony's ClassLoader.

There's a note on The PSR-4 Class Loader page suggesting that the loader should only be used when Composer is not available - but it looks like this is relevant to the PSR-0 and ClassMap loaders too. I think it would be more clear if this note were moved to the introduction page.

It would also make things more clear if we tweaked the install instructions to promote $ git clone rather than using $ composer require symfony/class-loader. This would be more consistent with the documentation pages that follow that use require_once '/path/to/src/Symfony/Component/ClassLoader/ClassLoader.php'; instead of require_once vendor/autoload.php
@javiereguiluz javiereguiluz mentioned this issue Jan 18, 2017
Closed
@javiereguiluz
Copy link
Member

I agree ... but given that this component will be deprecated soon, in #7383 I propose to add just a note about this, instead of making all the changes proposed in this PR.

@javiereguiluz javiereguiluz added the hasPR A Pull Request has already been submitted for this issue. label Jan 18, 2017
xabbuh added a commit that referenced this issue Feb 3, 2017
@xabbuh
minor #7383 Added a note about not using the ClassLoader component (j…
d447deb 
…aviereguiluz)

This PR was squashed before being merged into the 2.7 branch (closes #7383).

Discussion
----------

Added a note about not using the ClassLoader component

This fixes #5949.

The component will be deprecated soon, so I guess it's OK to start warning our users.

Commits
-------

e96cbf8 Added a note about not using the ClassLoader component
@xabbuh xabbuh closed this as completed Feb 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
hasPR A Pull Request has already been submitted for this issue.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@javiereguiluz @xabbuh @hpierce1102
0