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

PHP version documentation outdated #582

Closed
moztech opened this issue Feb 3, 2020 · 2 comments · Fixed by #583
Closed

PHP version documentation outdated #582

moztech opened this issue Feb 3, 2020 · 2 comments · Fixed by #583
Labels

Comments

@moztech
Copy link

moztech commented Feb 3, 2020

https://getcomposer.org/doc/00-intro.md
The documentation states PHP version 5.3.2+ is required
Composer requires PHP 5.3.2+ to run

However the default installation is via composer and requires PHP 7.2+
https://github.com/composer/satis/blob/master/composer.json

"require": {
        "php": "^7.2",

Actual behaviour on a stock CentOS 7 box:

$ composer create-project composer/satis:dev-master
[InvalidArgumentException]
Could not find package composer/satis with stability stable in a version installable using your PHP version 5.4.16.

If the documentation could be updated to mirror the composer.json file it should prevent others following me into building a CentOS 7 server defaulting to PHP 5.4 which can't then install composer.
If the docs were within the project I'd happily have issued a PR but don't believe that to be the case.

@moztech moztech added the bug label Feb 3, 2020
@alcohol
Copy link
Member

alcohol commented Feb 3, 2020

Somewhat related to #403 - having the documentation in another repo right now does not make much sense.

@SvenRtbg
Copy link
Contributor

SvenRtbg commented Feb 3, 2020 via email

SvenRtbg added a commit to SvenRtbg/satis that referenced this issue Feb 3, 2020
Fixes composer#582.
Intentionally generic wording to prevent constant update sync issues.
alcohol pushed a commit that referenced this issue Feb 4, 2020
Fixes #582.
Intentionally generic wording to prevent constant update sync issues.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants