Crash on start 1.8.9

Wuppertaler93 · July 20, 2016, 3:00am

Hello Community,

I want use Sponge on my server but is crash on starting the server

my installed mods:
-Evs-Ivs-1.8.9-4.1.0-universal
-MoreCommands-1.8.9-2.7
-Pixelmon-1.8.9-4.2.7-universal
-pixelmonfriends-1.8.9-1.3.1-universal
-SecurityCraft v1.8.1-1.8.8
-TrainerCommands-1.8.9-2.1.0-universal
-WonderTrade-1.8.9-4.3.1-universal
-worldedit-forge-mc1.8.9-6.1.1
-colorchat-1.8.8-1.0.2-universal
-k4lib-1.8.9-1.0.18-universal

my Forge version is: 11.15.1.1902
tested Sponge version: spongeforge-1.8.9-1890-4.2.0-BETA-1574

crashlog: http://pastebin.com/3XTeQKMb

hope you can help me

and sry for my very bad english

Wuppertaler93 · July 20, 2016, 3:00am

crash log
http://pastebin.com/n653eYxx

TrenTech · July 20, 2016, 4:09am

Missing Mods:
    k4lib : any

net.minecraftforge.fml.common.MissingModsException: Mod colorchat (ColorChat) requires [k4lib]

Seems pretty clear

http://mods.curse.com/mc-mods/minecraft/224740-k4lib

Wuppertaler93 · July 20, 2016, 10:06am

Without Sponge is it working and the k4lib is installed, i have removed the k4lib and colorchat and now i have not a crash log on crash i have fml-server-latest only

edit: why is my post flaged as spam? i want help and i can´t post 2 links in 1 message, now i can´t post 1 link -.-

Inscrutable · July 20, 2016, 10:36am

It’s an automated forum security feature to stop spambots from choking it. It’s a bit trigger happy at times, as new users have a limit on links they can post. I’ve unhidden the posts for now.

Wuppertaler93 · July 22, 2016, 1:28am

No one can help me?

Inscrutable · July 22, 2016, 8:40am

For your second error, try renaming the spongeforge.jar to 111spongeforge.jar. That will force it to load before other coremods, which appear to be messing with expected classes (ie. java.lang.ClassNotFoundException: net.minecraft.block.BlockRail).

Wuppertaler93 · July 23, 2016, 1:30am

i have edit it to 111sponge…
but it crash again

Zidane · July 23, 2016, 2:43am

@Wuppertaler93

[03:28:34] [main/TRACE] [FML/]: Registering transformer com.mrnobody.morecommands.asm.transform.TransformBlockRailBase

Sponge cannot find BlockRail, hence the crash and I’m willing to throw down some money that MoreCommands is classloading BlockRail too early (before Sponge can transform it with Mixin).

Remove it from your server and launch it, will probably load fine. @Aaron1011, investigate his codebase…he references a Reference class that has a few imports to Forge code.

gabizou · July 23, 2016, 2:49am

It’s actually a different issue:

Caused by: java.lang.ClassFormatError: Duplicate field name&signature in class file net/minecraft/block/BlockRailBase
	at java.lang.ClassLoader.defineClass1(Native Method) ~[?:1.8.0_91]
	at java.lang.ClassLoader.defineClass(ClassLoader.java:763) ~[?:1.8.0_91]
	at java.security.SecureClassLoader.defineClass(SecureClassLoader.java:142) ~[?:1.8.0_91]
	at net.minecraft.launchwrapper.LaunchClassLoader.findClass(LaunchClassLoader.java:182) ~[launchwrapper-1.12.jar:?]
	at java.lang.ClassLoader.loadClass(ClassLoader.java:424) ~[?:1.8.0_91]
	at java.lang.ClassLoader.loadClass(ClassLoader.java:357) ~[?:1.8.0_91]
	at java.lang.ClassLoader.defineClass1(Native Method) ~[?:1.8.0_91]
	at java.lang.ClassLoader.defineClass(ClassLoader.java:763) ~[?:1.8.0_91]
	at java.security.SecureClassLoader.defineClass(SecureClassLoader.java:142) ~[?:1.8.0_91]
	at net.minecraft.launchwrapper.LaunchClassLoader.findClass(LaunchClassLoader.java:182) ~[launchwrapper-1.12.jar:?]
	at java.lang.ClassLoader.loadClass(ClassLoader.java:424) ~[?:1.8.0_91]
	at java.lang.ClassLoader.loadClass(ClassLoader.java:357) ~[?:1.8.0_91]
	at org.spongepowered.common.registry.provider.BlockPropertyIdProvider.<init>(BlockPropertyIdProvider.java:207) ~[BlockPropertyIdProvider.class:1.8.9-1890-4.2.0-BETA-1574]
	at org.spongepowered.common.registry.provider.BlockPropertyIdProvider$Holder.<clinit>(BlockPropertyIdProvider.java:303) ~[BlockPropertyIdProvider$Holder.class:1.8.9-1890-4.2.0-BETA-1574]
	at org.spongepowered.common.registry.provider.BlockPropertyIdProvider.getInstance(BlockPropertyIdProvider.java:49) ~[BlockPropertyIdProvider.class:1.8.9-1890-4.2.0-BETA-1574]
	at org.spongepowered.common.registry.provider.BlockPropertyIdProvider.getIdAndTryRegistration(BlockPropertyIdProvider.java:67) ~[BlockPropertyIdProvider.class:1.8.9-1890-4.2.0-BETA-1574]
	at org.spongepowered.common.registry.type.BlockTypeRegistryModule.registerBlockTrait(BlockTypeRegistryModule.java:128) ~[BlockTypeRegistryModule.class:1.8.9-1890-4.2.0-BETA-1574]
	at org.spongepowered.common.registry.type.BlockTypeRegistryModule.registerFromGameData(BlockTypeRegistryModule.java:116) ~[BlockTypeRegistryModule.class:1.8.9-1890-4.2.0-BETA-1574]
	at net.minecraft.block.Block.handler$onRegisterBlock$0(Block.java:131) ~[afh.class:?]
	at net.minecraft.block.Block.func_176215_a(Block.java:2426) ~[afh.class:?]
	at net.minecraft.block.Block.func_176219_a(Block.java:2430) ~[afh.class:?]
	at net.minecraft.block.Block.func_149671_p(Block.java:2177) ~[afh.class:?]
	at net.minecraft.init.Bootstrap.func_151354_b(Bootstrap.java:512) ~[kb.class:?]
	at net.minecraft.server.MinecraftServer.main(MinecraftServer.java:1407) ~[MinecraftServer.class:?]
	... 12 more

Zidane · July 23, 2016, 2:52am

@gabizou

github.com

MrNobody98/morecommands/blob/master/src/main/java/com/mrnobody/morecommands/asm/ASMNames.java#L94


	/**
	 * An enum representing the name of a method
	 * 
	 * @author MrNobody98
	 */
	public static enum Method {
		Object_init(Type.Object, "<init>", "<init>", "V"),
		ItemStack_onItemUse(Type.ItemStack, "onItemUse", "func_179546_a", Type.EnumActionResult.getDesc(), Type.EntityPlayer.getDesc(), Type.World.getDesc(), Type.BlockPos.getDesc(), Type.EnumHand.getDesc(), Type.EnumFacing.getDesc(), "FFF"),
		ItemStack_getItem(Type.ItemStack, "getItem", "func_77973_b", Type.Item.getDesc()),
		Item_onItemUse(Type.Item, "onItemUse", "func_180614_a", Type.EnumActionResult.getDesc(), Type.EntityPlayer.getDesc(), Type.World.getDesc(), Type.BlockPos.getDesc(), Type.EnumHand.getDesc(), Type.EnumFacing.getDesc(), "FFF"),
		ItemStackChangeSizeEvent_init(Type.ItemStackChangeSizeEvent, "<init>", "<init>", "V", Type.EntityPlayer.getDesc(), Type.ItemStack.getDesc(), "II"),
		EventHandler_post(Type.EventHandler, "post", "post", "Z", Type.Event.getDesc()),
		ForgeHooks_onPlaceItemIntoWorld(Type.ForgeHooks, "onPlaceItemIntoWorld", "onPlaceItemIntoWorld", Type.EnumActionResult.getDesc(), Type.ItemStack.getDesc(), Type.EntityPlayer.getDesc(), Type.World.getDesc(), Type.BlockPos.getDesc(), Type.EnumFacing.getDesc(), "FFF", Type.EnumHand.getDesc()),
		BlockRailBase_getRailMaxSpeed(Type.BlockRailBase, "getRailMaxSpeed", "getRailMaxSpeed", "F", Type.World.getDesc(), Type.EntityMinecart.getDesc(), Type.BlockPos.getDesc()),
		ItemStack_damageItem(Type.ItemStack, "damageItem", "func_77972_a", "V", "I", Type.EntityLivingBase.getDesc()),
		ItemStack_setCount(Type.ItemStack, "setCount", "func_190920_e", "V", "I"), //TODO: Change in other versions (and remove superfluous obfs)
		DamageItemEvent_init(Type.DamageItemEvent, "<init>", "<init>", "V", Type.EntityLivingBase.getDesc(), "I", Type.ItemStack.getDesc()),
		Style_init(Type.Style, "<init>", "<init>", "V")
		;
		
		private final String deobfName, obfName, descriptor;

github.com

MrNobody98/morecommands/blob/master/src/main/java/com/mrnobody/morecommands/asm/transform/TransformBlockRailBase.java

package com.mrnobody.morecommands.asm.transform;

import java.util.Set;

import org.objectweb.asm.MethodVisitor;
import org.objectweb.asm.Opcodes;

import com.google.common.collect.ImmutableSet;
import com.mrnobody.morecommands.asm.ASMNames;
import com.mrnobody.morecommands.asm.WriterTransformer;

/**
 * Transforms {@link net.minecraft.block.BlockRailBase} so that its 
 * {@link net.minecraft.block.BlockRailBase#getRailMaxSpeed(net.minecraft.world.World, net.minecraft.entity.item.EntityMinecart, int, int, int)}
 * method does not return a constant value but fetches it from a field named "maxRailSpeed" which is private and gets
 * generated with this transformer too. Additionally generates a public setter for this field.
 * 
 * @author MrNobody98
 */
public class TransformBlockRailBase extends WriterTransformer {

This file has been truncated. show original

gabizou · July 23, 2016, 5:41pm

Oh, I’m willing to bet that the transformer is trying to re-add the field and method because of the transformer chain. Wanna suggest or PR some mixins?

Aaron1011 · July 24, 2016, 12:02am

While excluding the block rail base transformer with an @Resource annotation does appear to fix the issue, I’m still trying to work out exactly what’s causing this - in particular, why it’s only affecting this one transformer.

gabizou · July 24, 2016, 8:05pm

@Aaron1011 because the transformer is not expecting to modify the class twice.

Aaron1011 · July 24, 2016, 9:08pm

None of the transformers are expecting to modify a class twice. However, the transformed bytecode from Mixin’s initial first pass is only used internally (by Mixin). I’m still trying to figure out what’s causing the issue for that transformer.

gabizou · July 24, 2016, 9:10pm

I’m almost willing to submit a PR to convert some of the transformers to use mixins.

Aaron1011 · July 24, 2016, 9:14pm

I just realized what the issue is - it’s quite simple.

The base class WriterTransformer re-uses a ClassWriter between calls to transform. The error makes perfect sense - cv.visitField and cv.visitMethod are being called twice.

Aaron1011 · July 24, 2016, 9:39pm

@Wuppertaler93: I’ve opened a PR which resolves this issue.

Wuppertaler93 · July 24, 2016, 11:10pm

ok thank you, I must wait for a fix now right?