Divine Journey 2 Addons

Divine Journey 2 Addons

345k Downloads

Some problems that is not in the DJ2 modpack

mczph opened this issue · 10 comments

commented

I'm trying to use DJ2Addon in my own modpack (I know this is not reconmmanded) and found a bug related to modular magic,it seem that it only support 1.4.0 version.
Here's the crash report when I tried to run with modular magic 1.6.0.
https://mclo.gs/J9E3wT9
If you don't want to fix this, it's ok, I can use an older version, thanks。

commented

That's odd. It should only be running that Mixin if that class is present, so there shouldn't be any reason for it to throw this error.
I'll look into it.

commented

@mczph Does this issue still occur in the new beta release? I made this mod to be usable in non-DJ2 packs, so I'd like to fix the problem if it's still happening.

commented

@mczph Does this issue still occur in the new beta release? I made this mod to be usable in non-DJ2 packs, so I'd like to fix the problem if it's still happening.

Will go for testing.

PS: After some time, I am now using mmce (it is a fork version of modular machinery, which has optimized and added a lot of content, and has incorporated modular magic related content in the latest version).

commented

@ByThePowerOfScience It seems that the new version works very well. The only problem is that it has a forced dependency on AgriCraft, which causes Industrial Foregoing to crash. I removed this entry in mixins.dj2addons.json and everything went fine. Here is the crash report:https://mclo.gs/9x3Dak5

commented

@ByThePowerOfScience It seems that the new version works very well. The only problem is that it has a forced dependency on AgriCraft, which causes Industrial Foregoing to crash. I removed this entry in mixins.dj2addons.json and everything went fine. Here is the crash report:https://mclo.gs/9x3Dak5

I'll take a look, thanks!

commented

@mczph I'm unable to reproduce the issue. Can you try again and make sure you're using the latest version of DJ2Addons and have removed any older versions from your mods directory?

commented

@ByThePowerOfScience Yes,I'm using the latest version.Maybe my description is not clear enough, the following is the smallest list of reproducible modlist.
image
crash report:https://mclo.gs/fVJBFgH
After enabing AgriCraft and its dependency,the game will run normally.
I think a check for whether the game has AgriCraft installed should be added to prevent this issue

commented

MixinBooter is currently not supported due to some compatibility issues on their part, which is subject to change. Try removing MixinBooter and testing with MixinBootstrap instead.

commented

@ByThePowerOfScience I have tested it and the same problem occurred.Can you please have a look on if there is a hard dependency on AgriCraft,thank you.

commented

@ByThePowerOfScience I have tested it and the same problem occurred.Can you please have a look on if there is a hard dependency on AgriCraft,thank you.

I have, and I've tested it several times in several different mod configurations. I'm just not able to encounter the error.

Closing this as CNR for now. If more people have this issue, I'll reopen it.