Skip to content
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

[2.7+] Installation hint for required php-extensions phar #5919

Closed
Brutus5000 opened this issue Nov 28, 2015 · 1 comment
Closed

[2.7+] Installation hint for required php-extensions phar #5919

Brutus5000 opened this issue Nov 28, 2015 · 1 comment
Labels
actionable Clear and specific issues ready for anyone to take them. hasPR A Pull Request has already been submitted for this issue. Installer

Comments

@Brutus5000
Copy link

Hi guys,

I have a suggestion for the installation doc:
It would be great if you could add the hint that the phar-extension for php is required to be active to run "symfony new my_new_project".

Otherwise you get no response from calling symfony in the shell. I only found out because i used the suggested workaround using composer. Composer gives you a hint.

I don't know whether phar is active in php on default. On my synology nas it's definitely not active on default.

@xabbuh xabbuh added Installer actionable Clear and specific issues ready for anyone to take them. labels Nov 30, 2015
@xabbuh
Copy link
Member

xabbuh commented Nov 30, 2015

Imo such a change makes sense. We can simply add a note directive to relevant sections of the documentation:

.. note::

    Please make sure that the `phar extension`_ is installed and enabled.
    Otherwise, the installer will simply return without doing anything nor
    giving you any output.

And add the bottom of the changed files add this then:

.. _`phar extension`: http://php.net/manual/en/intro.phar.php

Thanks for reporting this @Brutus5000.

@wouterj wouterj added the hasPR A Pull Request has already been submitted for this issue. label Dec 5, 2015
wouterj added a commit that referenced this issue Dec 17, 2015
This PR was squashed before being merged into the 2.3 branch (closes #5963).

Discussion
----------

Add note about 'phar extension' dependency

| Q             | A
| ------------- | ---
| Doc fix?      | yes
| New docs?     | no
| Applies to    | all
| Fixed tickets | #5919

Commits
-------

696c154 Add note about 'phar extension' dependency
@wouterj wouterj closed this as completed Jan 7, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
actionable Clear and specific issues ready for anyone to take them. hasPR A Pull Request has already been submitted for this issue. Installer
Projects
None yet
Development

No branches or pull requests

3 participants