Skip to content

version update to 1.12#7

Draft
Aaronstar-McClure wants to merge 17 commits intobspkrs:masterfrom
Aaronstar-McClure:1.12.2
Draft

version update to 1.12#7
Aaronstar-McClure wants to merge 17 commits intobspkrs:masterfrom
Aaronstar-McClure:1.12.2

Conversation

@Aaronstar-McClure
Copy link
Copy Markdown

@Aaronstar-McClure Aaronstar-McClure commented Mar 28, 2021

Trying to update to 1.12. Just making the pr for now, will try adding the changes I have so far when I get home.
Edit: forgot to mention, issue #6

updated gradle and scripts. tried updating old files but would not work. going to re add important lines to build.gradle later, but this is necessary to even get the workspace setup
@Aaronstar-McClure
Copy link
Copy Markdown
Author

updated gradle so i could initialize the dev environment

sorts the imports, removes unused ones, and potentially corrects broken ones where methods were moved
todo: fix config generation at the bottom, need to convert to modname:itemname.

verify: ambiguous corrected import; minecraftforge.fml.common.registry.GameData no longer exists. auto corrected to net.minecraftforge.registries.GameData this is an internal registry and should not be used. net.minecraftforge.fml.common.registry.GameRegistry is the closest to original version and may have necessary functions to correct.
verify correct change to compareTo
again, only thing left is the registry problem
todo: figure out biome checker and loot table system
@Aaronstar-McClure
Copy link
Copy Markdown
Author

well, that is the first pass of fixes. mostly easy, some a bit complicated.
todo: correct the many and varied instances of GameData.getItemRegistry().getNameForObject
this seems to rely on public item id's, which is no longer a thing...
todo: figure out the loot system as chestGenHooks is no longer a thing....

@Aaronstar-McClure
Copy link
Copy Markdown
Author

@bspkrs if you are interested in looking at this and giving me pointers that would defiantly help. this is my first real dive into modding, so i really would like to figure this out

@MichaelHillcox
Copy link
Copy Markdown

Looks good so far, I can do a proper review of it later on. I'll talk to Bspkrs sometime soon about getting this repo moved to the org.

@Aaronstar-McClure
Copy link
Copy Markdown
Author

Cool, currently I'm looking into if I can pull the core dep from github rather then the bspk.rs repo. That might make it a bit eaiser and have fewer steps in the future...

@Aaronstar-McClure
Copy link
Copy Markdown
Author

Aaronstar-McClure commented Mar 29, 2021

Here's something that might help
Do you know the groupName:moduleName for bspkrsCore? I can't seem to get Gradle to resolve the dependency from GitHub

@MichaelHillcox
Copy link
Copy Markdown

You can use cursemaven.com to pull it into the project :)

@Aaronstar-McClure
Copy link
Copy Markdown
Author

i cant seem to get the dependency to resolve... any thoughts?

turns out there was an extra step not in the instructions where i had to add a gradle plugin...
@Aaronstar-McClure
Copy link
Copy Markdown
Author

Aaronstar-McClure commented Mar 30, 2021

Turns out the maven link did not want to work, so I had to use a gradle plugin instead to grab from curse. It should work now...

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.

2 participants