Skip to content
This repository has been archived by the owner on Feb 9, 2021. It is now read-only.

Workaround for pthreads Collectable update + merge master #3802

Closed
wants to merge 25 commits into from

Conversation

PEMapModder
Copy link
Collaborator

I'm not sure if this is done correctly. I hadn't tested it with the knowledge that AsyncTask is being scheduled on the server, but at least the server starts up correctly with PHP 7.0.1 + pthreads 3.1.5

shoghicp and others added 22 commits August 23, 2015 00:25
All plugins will need to bump the API if they want to use this.
NOTE THAT THIS IS NOT THE FINAL API 2.0.0 AND THAT THERE WILL BE MORE CHANGES.
To start updating, you might also want to read https://secure.php.net/manual/en/migration70.php and specifically https://secure.php.net/manual/en/migration70.incompatible.php

To compile PHP7 with some of the required dependencies, use https://gist.github.com/shoghicp/166ab26ce5cc7a390f45
ONLY LINUX IS TESTED, DO NOT ASK FOR OTHER PLATFORMS!

----- THIS VERSION IS NOT SUPPORTED -----

This version WILL crash randomly in unexpected places due to PHP7, pthreads, PocketMine or cosmic rays.

Handle with care, and store under direct sunlight for the best performance.
@onebone
Copy link
Contributor

onebone commented Dec 25, 2015

👍

@PEMapModder PEMapModder changed the title Workaround for pthreads Collectable update Workaround for pthreads Collectable update + merge master Dec 25, 2015
@PEMapModder
Copy link
Collaborator Author

Also merged PocketMine:master into this PR.

I would appreciate if anyone would help test this branch with a plugin that uses AsyncTask, to confirm that it works.

@PEMapModder
Copy link
Collaborator Author

http://pastebin.com/gDHWj0Gb
This happened when I am testing with this branch. Not sure if it is caused by the changes in this PR yet.
Edit: confirmed that the same configuration and maps cannot be reproduced on the master branch.

@PEMapModder PEMapModder closed this Feb 6, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

10 participants