8000 Update installation.rst by nicolas-zozol · Pull Request #550 · symfony-cmf/symfony-cmf-docs · GitHub
[go: up one dir, main page]

Skip to content
This repository was archived by the owner on Sep 16, 2021. It is now read-only.

Update installation.rst #550

Closed
wants to merge 1 commit into from
Closed

Update installation.rst #550

wants to merge 1 commit into from

Conversation

nicolas-zozol
Copy link

Once we have done sudo mv composer.phar /usr/local/bin/composer, we do not need anymore to type php composer.phar

Once we have done `sudo mv composer.phar /usr/local/bin/composer`, we do not need anymore to type `php composer.phar`
@dantleech
Copy link
Member

But then windows users cannot copy composer.phar to that directory.

The official Symfony documentation uses the php composer.phar method

http://symfony.com/doc/current/book/installation.html

Maybe we need to explain rather that if you install composer to your $PATH you no longer need to prefix "php"

@dbu
Copy link
Member
dbu commented Aug 21, 2014

or just remove the mv /usr/local/bin idea from the install
instructions to not confuse newbies.

experienced users know what composer is and can handle that.

@wouterj
Copy link
Member
wouterj commented Aug 23, 2014

Thank you for opening this PR! However, if we do it here, we should use composer everywhere in the complete Symfony documentation. Otherwise, people will get confused.

That's why I created an issue about this in the core docs: symfony/symfony-docs#4170

Let's see how that works out, thank you for starting this! (I like using composer instead of php composer.phar)

@wouterj
Copy link
Member
wouterj commented Oct 25, 2014

Fixed by #590

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants
0