Skip to content

Comments

Port to 1.17.#12

Open
John-Paul-R wants to merge 2 commits intosmonnnn:masterfrom
John-Paul-R:1.17
Open

Port to 1.17.#12
John-Paul-R wants to merge 2 commits intosmonnnn:masterfrom
John-Paul-R:1.17

Conversation

@John-Paul-R
Copy link

Update gradle things.
Update usages of PlayerListHeaderS2CPacket to conform to 1.17 class spec.

Update gradle things.
  - Had to set `duplicatesStrategy = 'INCLUDE'` to build on my machine.
  not 100% sure what this does, but seems to build correctly atm.
Also, update usages of `PlayerListHeaderS2CPacket` where required (things changed 1.16 -> 1.17)
@John-Paul-R
Copy link
Author

Check build.gradle's processResources. I made slight changes to fix a build error. I believe it runs correctly (mod runs on my machine when built this way), but just double-check it's doing what you intended.

@John-Paul-R John-Paul-R marked this pull request as ready for review July 5, 2021 00:13
@John-Paul-R
Copy link
Author

Note: these changes will be incompatible with 1.16, so if you want to keep support for 1.16 if/when you add new features, it might be worth merging these changes to new branch.

@John-Paul-R
Copy link
Author

Check build.gradle's processResources. I made slight changes to fix a build error. I believe it runs correctly (mod runs on my machine when built this way), but just double-check it's doing what you intended.

Upon doing further research, this seems to be a standard fix that many modders have made upon updating to 1.17. Should work as intended.

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.

1 participant