-
Notifications
You must be signed in to change notification settings - Fork 2
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
Any possibility of this being resurrected? #1
Comments
Sure, I'll see what I can do. It's crazy to think that I wrote this almost two years ago! Send me pull requests as you come up with them. |
Excellent! The first thing I did when I got the code was run fmt & fix on it. This left one compiler error: I have not yet worked out the best way to resolve that, as far as I understand time.Duration should take an int64. Next up, I will add all the current blocks, then begin to add all the current packets in the protocol. |
You may have to add an explicit cast to time.Duration (I.e. change the
|
well... I'm not sure if I did this correctly but at least it compiles now. I'm not going to send you a pull request until I update a few of the protocol packets to current (so at least talks a little bit to the server). I am not sure which server you used when you first did this... but I've tried it as is with 1.1 and it didn't work. Cheers! |
I know this has been ignored for a couple of years and minecraft has moved on. However, this is something I think could be really interesting / useful.
I'd be happy to update the easy things, like the block id's & packet definitions for the communication protocol. But some of the other stuff is probably beyond my abilities right now, to be honest. So I hope that if I was able to do the repetitive stuff, you'd be able to update the rest rather quickly.
Cheers!
The text was updated successfully, but these errors were encountered: