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

Basic PHP7 compatibility #89

Open
wants to merge 1 commit into
base: 0.9.2
Choose a base branch
from
Open

Basic PHP7 compatibility #89

wants to merge 1 commit into from

Conversation

moeffju
Copy link
Member

@moeffju moeffju commented Dec 12, 2017

All plugins etc. need to be converted to use HabariError instead of Error.

@moeffju
Copy link
Member Author

moeffju commented Dec 12, 2017

I remember we looked at namespaces some time, but I can’t quite make heads or tails of the submodule structure.

@miklb
Copy link
Member

miklb commented Dec 12, 2017

I think this was done in the master branch, which is quite a divergence from 0.9.x

@moeffju
Copy link
Member Author

moeffju commented Dec 21, 2017

That makes sense, but I guess we should maybe push out a release then? :)

@miklb
Copy link
Member

miklb commented Dec 21, 2017

I've actually been thinking about that, and I believe my comment was misplaced. This PR does make sense, and a release would also be appropriate.

Apologies for the confusion.

@iromeister
Copy link

Is there any hope for a PHP7 compatible release? No activity for 2 years now...

@Konzertheld
Copy link
Member

@iromeister No, unfortunately there are no releases planned as there has been no development activity for quite a while. You can try this branch or even experiment with the master branch. The master branch is very experimental and differs widely from 0.9.2 but it is PHP 7 compatible.

If you try this branch successfully, let me know, I will then test it myself and could merge it afterwards.

Please make sure to do backups before testing either option.

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

Successfully merging this pull request may close these issues.

4 participants