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

Nukkit support #13

Open
MrPowerGamerBR opened this issue Feb 5, 2017 · 11 comments
Open

Nukkit support #13

MrPowerGamerBR opened this issue Feb 5, 2017 · 11 comments

Comments

@MrPowerGamerBR
Copy link

@MrPowerGamerBR MrPowerGamerBR commented Feb 5, 2017

(I already added Nukkit support to my TaskChain fork https://github.com/PocketDreams/TaskChain however I didn't create a pull request because I don't know if you are accepting pull requests for other server softwares...)

(And my TaskChain fork only works with my Nukkit fork until Nukkit/Nukkit#1459 is pulled to upstream, so...)

@willies952002
Copy link

@willies952002 willies952002 commented Feb 6, 2017

Personally, I wouldn't see why a PR for adding support for another platform would be an issue :D

@aikar
Copy link
Owner

@aikar aikar commented Feb 6, 2017

If you can do the following, I can pull it if you PR:

  • Fix the whitespace in root pom.xml
  • Remove eclipse project stuff
  • Rename the Bukkit inner class to Nukkit
  • Provide Nukkit in a public repository that the nukkit pom file includes like Bukkit does so I can compile w/o having Nukkit on my system.
@MrPowerGamerBR
Copy link
Author

@MrPowerGamerBR MrPowerGamerBR commented Feb 6, 2017

@aikar yeah, my fork was like "well, it works, so..." (and I tried to clean up the Eclipse mess... but it didn't work)

The problem is that the current code won't work on the current version of Nukkit, Nukkit's tasks doesn't include the plugin instance on the task so I think some parts of the code won't work (right?), so I will probably do a clean pull request when my pull request @ Nukkit is pulled. (and yes, I tested my modifications on Nukkit... it works fairly well (I already use TaskChain on my PC server), waaay better than creating a bunch of schedulers)

@MrPowerGamerBR
Copy link
Author

@MrPowerGamerBR MrPowerGamerBR commented Feb 25, 2017

The PR was pulled to Nukkit, so I will probably create a PR to add Nukkit to TaskChain soon.

The problem is the Maven repo, there isn't an official Maven repo for Nukkit, so that will be a bit difficult.

@aikar
Copy link
Owner

@aikar aikar commented Feb 25, 2017

How do people currently build plugins then? do you really make them compile locally just to get artifacts?

@MrPowerGamerBR
Copy link
Author

@MrPowerGamerBR MrPowerGamerBR commented Feb 25, 2017

@aikar by "you" you mean "they", right? I'm not a Nukkit developer.

But yeah, most of them installs the Nukkit artifact in the local repo or use unofficial maven repository. (Mengcraft's Maven repo)

@aikar
Copy link
Owner

@aikar aikar commented Feb 25, 2017

long as it compiles then unofficial should be fine? if thats what the community is use to using.

@aikar
Copy link
Owner

@aikar aikar commented Aug 16, 2017

@MrPowerGamerBR @imjack

You both now have Nukkit versions. I've added Mengcraft to my Nexus Repo as a proxy, so if one of you can PR it (without the mengcraft reference, since it will be in my repo cache instead), we can officially get this support added.

@MrPowerGamerBR
Copy link
Author

@MrPowerGamerBR MrPowerGamerBR commented Aug 16, 2017

@aikar I stopped using Nukkit since February because it is on "life support" (it is only receiving protocol updates sometimes) so I migrated to other platforms, sorry 😢

@MrPowerGamerBR
Copy link
Author

@MrPowerGamerBR MrPowerGamerBR commented Aug 16, 2017

@aikar not really "active" when you compare PocketMine-MP (which Nukkit was based of)

Nukkit only receives some protocol fixes (when a new MCPE version is released) and some very critical fixes, but everyone of the main Nukkit team is inactive.

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

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.