Skip to content

Basic PHP7 compatibility#89

Open
moeffju wants to merge 1 commit intohabari:0.9.2from
moeffju:php-7
Open

Basic PHP7 compatibility#89
moeffju wants to merge 1 commit intohabari:0.9.2from
moeffju:php-7

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