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

Dropped PHP 5.x and 7.0 support - minimum of PHP 7.1 requirement #785

Merged
merged 1 commit into from
May 23, 2017

Conversation

Majkl578
Copy link
Contributor

Supersedes and fixes #756.

.travis.yml Outdated
@@ -20,7 +19,8 @@ matrix:
env: PHPSTAN=1

before_script:
- composer --prefer-source install
- if [[ $TRAVIS_PHP_VERSION = hhvm ]]; then composer --ignore-platform-reqs --prefer-source install; fi
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

--ignore-platform-reqs is not acceptable here

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fine, I tried. 😊 Damn I hate HHVM for this versioning hell... When switched to php7 mode (hhvm --php -d hhvm.php7.all=true), it works, but it's messy...

@Majkl578 Majkl578 force-pushed the php7-bump branch 2 times, most recently from 3b1332c to a83d9bd Compare April 15, 2017 01:50
@Majkl578
Copy link
Contributor Author

Okay, HHVM will just not work in PHP 7 mode at all... It crashes Composer entirely in all versions (Symfony Finder specifically).
Suggestions?

@Majkl578
Copy link
Contributor Author

Majkl578 commented Apr 15, 2017

See facebook/hhvm#7198 (comment) and composer/composer#6230 - bug in HHVM itself.

@szymach
Copy link

szymach commented Apr 16, 2017

Hey, I am curious whether support for HHVM is actually worth the effort? Perhaps add it as an allowed failure until the Facebook team resolves the issue?

@Majkl578
Copy link
Contributor Author

Maybe, but it's not up to me to decide.
HHVM support was always pain in the ass, i.e. Nette Framework dropped it because it was too broken/incompatible with PHP.

@szymach
Copy link

szymach commented Apr 16, 2017

Well, according Saldaek in the issue you have provided, there's less people using HHVM and Composer, than PHP 5.3 (~0.4%). If I may be so bold, I would suggest sidelining (if not dropping altogether) support for it, since it only slows down development progress without being actually used by any substantial number of people.

I know, it is not your call, but maybe someone else in the Doctrine team will consider this.

@Ocramius
Copy link
Member

I think @szymach's observation makes it clear: supporting HHVM is not worth it, and won't be worth it any soon due to the amount of effort needed to do it, while a lot of its PHP5 compatibility issues remain unpatched (and aren't in the project's focus either, as it seems).

We can drop HHVM if we do an announcement, since we did indeed guarantee compatibility (see http://www.doctrine-project.org/2013/12/23/our-hhvm-roadmap.html )

@Majkl578
Copy link
Contributor Author

@Ocramius @lcobucci Before I rebase this, are we going to support 7.0 or 7.1?

@Ocramius
Copy link
Member

@Majkl578 7.1

@Majkl578 Majkl578 changed the title Drop PHP 5.x support Drop PHP 5.x & 7.0 upport May 23, 2017
@Majkl578
Copy link
Contributor Author

Roger, PR updated.

@Ocramius Ocramius changed the title Drop PHP 5.x & 7.0 upport Drop PHP 5.x & 7.0 support May 23, 2017
@Ocramius Ocramius self-assigned this May 23, 2017
@Ocramius Ocramius added this to the 2.8.0 milestone May 23, 2017
@Ocramius
Copy link
Member

👍

@Ocramius Ocramius merged commit 54d5eb6 into doctrine:master May 23, 2017
@Majkl578 Majkl578 deleted the php7-bump branch May 23, 2017 21:37
@Ocramius Ocramius changed the title Drop PHP 5.x & 7.0 support Dropped PHP 5.x and 7.0 support - minimum of PHP 7.1 requirement Jul 22, 2017
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.

3 participants