maideniles / MaidensMarvelousMaterials1.15.2

Other
0 stars 1 forks source link

1.16 Version does not generate a config file #2

Open PfluorescentZebra opened 2 years ago

PfluorescentZebra commented 2 years ago

I'm afraid the 1.16 version, maidensmaterials-1.0.2.jar, is not generating a config file. Using Forge 1.16.5-36.2.0. Other mods are generating config files correctly, such as JEI and JourneyMap so I do not believe it is an environment issue. Have been able to reproduce on two windows computers in multiple game instances.

SteveBeeblebrox commented 2 years ago

Hi, thanks for the bug report. Are you able to provide a copy of the game log?

PfluorescentZebra commented 2 years ago

Sure, I suspect what you want is this line: "[17:05:14] [main/WARN]: Reference map 'maideniles.refmap.json' for maidensmaterials.mixins.json could not be read. If this is a development environment you can ignore this message latest.log "

SteveBeeblebrox commented 2 years ago

That seems like an issue with mixins, but afaik this version of the mod doesn't use any.

It looks like a couple of the print statements aren't being hit. I think the issue is that the mod is trying to make the files before forge has even figured out where the config folder should be. We're working on a fix.

PfluorescentZebra commented 2 years ago

I noticed also in the log some json errors, and assumed they were a result of whatever is causing this. There are some items that are not craftable (palm planks, for instance). Will test once you have a fix to be sure that it is the same error and not something else, but thought I should let you know as well. Ran across it last night when I was in an oasis with no other tree type in sight.

SteveBeeblebrox commented 2 years ago

Thanks for pointing out the JSON issues. The ones I saw in the log are fine and it looks like you just don't have an optional dependency installed, but the planks is a problem. They shouldn't be related to the config though.

As far as this issue goes, I was apparently debugging the wrong version. In 1.16 everything looks right. I built this system from scratch and everything seems in place. Are you able to send a screenshot of the .minecraft/config folder?

PfluorescentZebra commented 2 years ago

Sure; I didn't see any dependencies listed on the dependencies tab; I assume you mean Patchouli? I didn't bother with it since it affects only the book and shouldn't be affecting the config generation. The pack I was building when I discovered the issue does have Patchouli. Even adding Patchouli into this tester I'm still not getting a config to generate. Which forge version are you testing on? I can try that too.

Here's the screenshot of the config, and the latest log with Patchouli. Didn't see any JSON errors at a glance, at least.
tempsnip latest.log

SteveBeeblebrox commented 2 years ago

Could you run a search on the mc folder for maidensmaterials.kvp? It might just be generating in the wrong spot.

PfluorescentZebra commented 2 years ago

It is in the defaultconfig folder... am I an idiot?

SteveBeeblebrox commented 2 years ago

That's the config file. We've had a couple people miss it before since it isn't something standard like json/toml/cfg. No worries.