-
Notifications
You must be signed in to change notification settings - Fork 63
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
[1.12.2] Migrate to patch based system #231
Comments
When migrating I've found several codes that aren't or may not be good ( or safe ). These parts will be listed below:
|
I am not sure about number 2, but I agree with you on number 1. Is EULA checking removed in our current codebase? If so, can you readd it? |
Almost done! :) |
There was a merge to master two days ago. Please be sure to apply those changes as you see fit before your PR. Thank you so much for your work, and let me know if you need anything! |
Well it seems craftbukkit has already included some realtime fixes, for example in EntityItem.java. |
Seems that akarin's code base is pretty old now, some mixins are actually conflict with recent craftbukkit changes. lol |
All codes are done :) |
@josephworks I'm not so familiar with shell so I'm not so clear how to make the ./akarin work. It would be great that if you can help. |
@siyy123 Sorry for the late response. We can definitely look into getting an ./akarin script to work in the future. |
For better code organization and compatibility, migrating mixin to a patch based system should be a good try.
I'm already working on this and it seems to be easy for me.
Things to be merged
The text was updated successfully, but these errors were encountered: