+
Skip to content

Conversation

FlipEverything
Copy link
Contributor

@FlipEverything FlipEverything commented Aug 2, 2018

Still work in progress.

Bugs that cause the build to fail:

  • PHP7.1 (mysql, pgsql, sqlite): Function mcrypt_encrypt() is deprecated
  • PHP7.1 (mysql, pgsql, sqlite): Failed asserting that two arrays are equal (DateTime values)
  • HHVM (sqlite): exception 'PDOException' with message 'safe_mode/open_basedir prohibits opening

Todo:

The mcrypt_encrypt function has been DEPRECATED as of PHP 7.1.0. We are encouraged to use openssl_encrypt instead.
@FlipEverything
Copy link
Contributor Author

Should we support PHP 5.4 (support officially ended in 2015) and PHP 5.5 (support officially ended in 2016)? More specifically I'm thinking about backwards compatibility "hacks" like this: 338497f I haven't seen anything like that in the current codebase.

Do the community have the need for official HHVM support? Just figured out that I don't have the time to maintain HHVM compatibility in top of the new PHP releases given the fact that PHP 7+ is very close to HHVM's performance and "HHVM will not aim to target PHP7". Currently the HHVM mysql+psql build is running just fine (and I don't have the time to figure out what's wrong with sqlite) so we can include it in the repo but I think that would be an (unnecessary) commitment.

I think we should add nightly to our build and allow it to fail. This will help us move to the next PHP versions in the future. With PHP5's deprecation later this year I think this is a very important thing to do.

What are your thoughts?
@vlucas @tuupola @nebulousGirl @willemwollebrants

@FlipEverything
Copy link
Contributor Author

ping

@vlucas
Copy link
Collaborator

vlucas commented Aug 10, 2018

I feel like if we change support for PHP versions, we might have to do a major release, or at least a clear minor version bump with notes on why. I would like to do another patch version release before we do that to get out all the fixes that are in master first.

@FlipEverything
Copy link
Contributor Author

I can agree with that.

  • We could support PHP<5.6 for a while as well.
  • What do you think about HHVM? Should we support it or should we open in the direction of PHP7.2+ only?

@vlucas
Copy link
Collaborator

vlucas commented Aug 22, 2018

HHVM has kind of fallen out of favor with PHP 7+ performance being on-par with it. I'd be okay with dropping it in the next major release if it is burdensome to maintain support for.

@vlucas
Copy link
Collaborator

vlucas commented Sep 14, 2018

This blog post just came out. Let's drop HHVM for sure in the next major version.
https://hhvm.com/blog/2018/09/12/end-of-php-support-future-of-hack.html

@FlipEverything
Copy link
Contributor Author

FlipEverything commented Nov 21, 2024

@vlucas Travis CI is no longer working, this PR is invalid in this form. It has to be updated after #302 is possibly merged.

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 this pull request may close these issues.

2 participants

点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载