Decompile errors

[fomenta]

Hi,

 

I believe I followed all the steps correctly, but I am getting decompile patching errors.

 

What I did:

[*]Downloaded MCP

[*]Updated MCP (./mcp62/updatemcp.sh)

[*]Copied vanilla minecraft.jar into ./mcp62/jars/

[*]Added up-to-date modloader and forge-client (recommended) into minecraft.jar

[*](Did not delete META-INF)

[*]Extracted the forge sources into the mcp folder -- so now I have a folder ./mcp62/forge

[*]Verified that ./mcp62/forge/conf was copied to ./mcp62/conf

[*]Run ./mcp62/forge/install.sh

 

Everything looks fine until it encounters an error (while patching it seems):

 

 

$ ./install.sh 
=================================== Minecraft Forge Setup Start =================================
== MCP 6.2 (data: 6.2, client: 1.2.5, server: 1.2.5) ==
WARNING:
The cleanup script will delete all folders created by MCP, including the
src folder which may contain changes you made to the code, along with any
saved worlds from the client or server.
If you really want to clean up, enter "Yes" Yes
!! Updates available. Please run updatemcp to get them. !!
> Cleaning temp
> Cleaning src
> Cleaning bin
> Cleaning reobf
> Cleaning jars
> Cleaning logs
== MCP 6.2 (data: 6.2, client: 1.2.5, server: 1.2.5) ==
# found wine, jad, ff, jad patches, ff patches, osx patches, srgs, name csvs, doc csvs, param csvs, astyle, astyle config
!! Updates available. Please run updatemcp to get them. !!
> Creating Retroguard config files
!! Modified jar detected. Unpredictable results !!
== Decompiling client using fernflower ==
> Creating SRGs
> Applying Retroguard
> Applying MCInjector
> Unpacking jar
> Copying classes
> Decompiling
> Copying sources
> Applying fernflower fixes
> Applying patches
'patch -p1 -u -i ../../temp/temp.patch -d src/minecraft' failed : 1

== ERRORS FOUND ==

1 out of 1 hunk FAILED -- saving rejects to file net/minecraft/client/Minecraft.java.rej
1 out of 1 hunk FAILED -- saving rejects to file net/minecraft/src/RenderGlobal.java.rej
==================

> Cleaning comments
- Done in 241.95 seconds
== Reformating client ==
> Cleaning sources
> Replacing OpenGL constants
> Reformating sources
- Done in 14.52 seconds
!! renaming disabled !!
!! Missing server jar file. Aborting !!
!! recompile disabled !!
Applying Forge ModLoader patches
== MCP 6.2 (data: 6.2, client: 1.2.5, server: 1.2.5) ==
== Updating client ==
> Adding javadoc
> Renaming sources
- Done in 3.68 seconds
Applying forge patches
patching file net/minecraft/src/SoundManager.java.patch
patching file minecraft/net/minecraft/src/SoundManager.java
Hunk #1 succeeded at 4 with fuzz 1 (offset 2 lines).
Hunk #2 FAILED at 39.
Hunk #3 succeeded at 119 with fuzz 2 (offset 62 lines).
Hunk #4 FAILED at 78.
Hunk #5 FAILED at 166.
Hunk #6 FAILED at 219.
Hunk #7 FAILED at 244.
Hunk #8 FAILED at 279.
Hunk #9 succeeded at 325 with fuzz 2 (offset 27 lines).
6 out of 9 hunks FAILED -- saving rejects to file minecraft/net/minecraft/src/SoundManager.java.rej
patching file net/minecraft/src/EntityMooshroom.java.patch
patching file minecraft/net/minecraft/src/EntityMooshroom.java
Hunk #1 FAILED at 1.
Hunk #2 FAILED at 30.
Hunk #3 succeeded at 68 with fuzz 2 (offset 3 lines).
2 out of 3 hunks FAILED -- saving rejects to file minecraft/net/minecraft/src/EntityMooshroom.java.rej
patching file net/minecraft/src/ItemBow.java.patch
patching file minecraft/net/minecraft/src/ItemBow.java
Hunk #1 succeeded at 3 with fuzz 2 (offset 2 lines).
Hunk #2 FAILED at 16.
Hunk #3 FAILED at 105.
2 out of 3 hunks FAILED -- saving rejects to file minecraft/net/minecraft/src/ItemBow.java.rej
patching file net/minecraft/src/WorldGenDungeons.java.patch
patching file minecraft/net/minecraft/src/WorldGenDungeons.java
Hunk #1 succeeded at 3 with fuzz 1 (offset 1 line).
Hunk #2 FAILED at 119.
1 out of 2 hunks FAILED -- saving rejects to file minecraft/net/minecraft/src/WorldGenDungeons.java.rej
patching file net/minecraft/src/BlockPressurePlate.java.patch
patching file minecraft/net/minecraft/src/BlockPressurePlate.java
Reversed (or previously applied) patch detected!  Assume -R? [n] 

 

 

This is followed by 20 or so more of these messages.

 

This is the tail part of the log, after all the patch errors:

 

 

== MCP 6.2 (data: 6.2, client: 1.2.5, server: 1.2.5) ==
== Recompiling client ==
> Cleaning bin
> Recompiling
'"javac" -Xlint:-options -deprecation -g -source 1.6 -target 1.6 -classpath "lib:lib/*:jars/bin/minec...' failed : 1

== ERRORS FOUND ==

EntitySheep.java:287: isShearable(net.minecraft.src.ItemStack,net.minecraft.src.World,int,int,int) is already defined in net.minecraft.src.EntitySheep
    public boolean isShearable(ItemStack item, World world, int X, int Y, int Z)
                  ^

EntitySheep.java:293: onSheared(net.minecraft.src.ItemStack,net.minecraft.src.World,int,int,int,int) is already defined in net.minecraft.src.EntitySheep
    public ArrayList<ItemStack> onSheared(ItemStack item, World world, int X, int Y, int Z, int fortune)
                                ^

Entity.java:242: getEntityData() is already defined in net.minecraft.src.Entity
    public NBTTagCompound getEntityData()
                          ^

Entity.java:252: shouldRiderSit() is already defined in net.minecraft.src.Entity
    public boolean shouldRiderSit()
                  ^

RenderEngine.java:19: forge.ForgeHooksClient is already defined in a single-type import
import net.minecraft.src.forge.ForgeHooksClient;
^

ItemRenderer.java:7: forge.ForgeHooksClient is already defined in a single-type import
import net.minecraft.src.forge.ForgeHooksClient;
^

ItemRenderer.java:8: forge.IItemRenderer is already defined in a single-type import
import net.minecraft.src.forge.IItemRenderer;
^

ItemRenderer.java:9: forge.MinecraftForgeClient is already defined in a single-type import
import net.minecraft.src.forge.MinecraftForgeClient;
^

Chunk.java:12: forge.ForgeHooks is already defined in a single-type import
import net.minecraft.src.forge.ForgeHooks;
^

Explosion.java:10: forge.ISpecialResistance is already defined in a single-type import
import net.minecraft.src.forge.ISpecialResistance;
^

World.java:4469: addTileEntity(net.minecraft.src.TileEntity) is already defined in net.minecraft.src.World
    public void addTileEntity(TileEntity entity)
                ^

World.java:4488: isBlockSolidOnSide(int,int,int,int) is already defined in net.minecraft.src.World
    public boolean isBlockSolidOnSide(int X, int Y, int Z, int side)
                  ^

Block.java:1251: getLightValue(net.minecraft.src.IBlockAccess,int,int,int) is already defined in net.minecraft.src.Block
    public int getLightValue(IBlockAccess world, int x, int y, int z)
              ^

Block.java:1265: isLadder(net.minecraft.src.World,int,int,int) is already defined in net.minecraft.src.Block
    public boolean isLadder(World world, int x, int y, int z)
                  ^

Block.java:1281: isBlockNormalCube(net.minecraft.src.World,int,int,int) is already defined in net.minecraft.src.Block
    public boolean isBlockNormalCube(World world, int x, int y, int z)
                  ^

Block.java:1296: isBlockSolidOnSide(net.minecraft.src.World,int,int,int,int) is already defined in net.minecraft.src.Block
    public boolean isBlockSolidOnSide(World world, int x, int y, int z, int side)
                  ^

Block.java:1325: isBlockReplaceable(net.minecraft.src.World,int,int,int) is already defined in net.minecraft.src.Block
    public boolean isBlockReplaceable(World world, int x, int y, int z)
                  ^

Block.java:1340: isBlockBurning(net.minecraft.src.World,int,int,int) is already defined in net.minecraft.src.Block
    public boolean isBlockBurning(World world, int x, int y, int z)
                  ^

Block.java:1357: isAirBlock(net.minecraft.src.World,int,int,int) is already defined in net.minecraft.src.Block
    public boolean isAirBlock(World world, int x, int y, int z)
                  ^

Block.java:1368: getHardness(int) is already defined in net.minecraft.src.Block
    public float getHardness(int meta)
                ^

Block.java:1383: blockStrength(net.minecraft.src.World,net.minecraft.src.EntityPlayer,int,int,int) is already defined in net.minecraft.src.Block
    public float blockStrength(World world, EntityPlayer player, int x, int y, int z)
                ^

Block.java:1395: blockStrength(net.minecraft.src.EntityPlayer,int) is already defined in net.minecraft.src.Block
    public float blockStrength(EntityPlayer player, int meta)
                ^

Block.java:1407: canHarvestBlock(net.minecraft.src.EntityPlayer,int) is already defined in net.minecraft.src.Block
    public boolean canHarvestBlock(EntityPlayer player, int meta)
                  ^

Block.java:1430: removeBlockByPlayer(net.minecraft.src.World,net.minecraft.src.EntityPlayer,int,int,int) is already defined in net.minecraft.src.Block
    public boolean removeBlockByPlayer(World world, EntityPlayer player, int x, int y, int z)
                  ^

Block.java:1442: addCreativeItems(java.util.ArrayList) is already defined in net.minecraft.src.Block
    public void addCreativeItems(ArrayList itemList)
                ^

Block.java:1446: blockFireSpreadSpeed is already defined in net.minecraft.src.Block
    protected static int blockFireSpreadSpeed[] = new int[256];
                        ^

Block.java:1447: blockFlammability is already defined in net.minecraft.src.Block
    protected static int blockFlammability[] = new int[256];
                        ^

Block.java:1461: getFlammability(net.minecraft.src.IBlockAccess,int,int,int,int,int) is already defined in net.minecraft.src.Block
    public int getFlammability(IBlockAccess world, int x, int y, int z, int metadata, int face)
              ^

Block.java:1478: isFlammable(net.minecraft.src.IBlockAccess,int,int,int,int,int) is already defined in net.minecraft.src.Block
    public boolean isFlammable(IBlockAccess world, int x, int y, int z, int metadata, int face)
                  ^

Block.java:1495: getFireSpreadSpeed(net.minecraft.src.World,int,int,int,int,int) is already defined in net.minecraft.src.Block
    public int getFireSpreadSpeed(World world, int x, int y, int z, int metadata, int face)
              ^

Block.java:1513: isFireSource(net.minecraft.src.World,int,int,int,int,int) is already defined in net.minecraft.src.Block
    public boolean isFireSource(World world, int x, int y, int z, int metadata, int face)
                  ^

Block.java:1532: setBurnProperties(int,int,int) is already defined in net.minecraft.src.Block
    public static void setBurnProperties(int id, int encouragement, int flammability)
                      ^

Block.java:1548: hasTileEntity(int) is already defined in net.minecraft.src.Block
    public boolean hasTileEntity(int metadata)
                  ^

Block.java:1561: getTileEntity(int) is already defined in net.minecraft.src.Block
    public TileEntity getTileEntity(int metadata)
                      ^

Block.java:1579: quantityDropped(int,int,java.util.Random) is already defined in net.minecraft.src.Block
    public int quantityDropped(int meta, int fortune, Random random)
              ^

Block.java:1595: getBlockDropped(net.minecraft.src.World,int,int,int,int,int) is already defined in net.minecraft.src.Block
    public ArrayList<ItemStack> getBlockDropped(World world, int x, int y, int z, int metadata, int fortune)
                                ^

Block.java:1622: canSilkHarvest(net.minecraft.src.World,net.minecraft.src.EntityPlayer,int,int,int,int) is already defined in net.minecraft.src.Block
    public boolean canSilkHarvest(World world, EntityPlayer player, int x, int y, int z, int metadata)
                  ^

Block.java:1642: canCreatureSpawn(net.minecraft.src.EnumCreatureType,net.minecraft.src.World,int,int,int) is already defined in net.minecraft.src.Block
    public boolean canCreatureSpawn(EnumCreatureType type, World world, int x, int y, int z)
                  ^

Block.java:1682: isBed(net.minecraft.src.World,int,int,int,net.minecraft.src.EntityLiving) is already defined in net.minecraft.src.Block
    public boolean isBed(World world, int x, int y, int z, EntityLiving player)
                  ^

Block.java:1698: getBedSpawnPosition(net.minecraft.src.World,int,int,int,net.minecraft.src.EntityPlayer) is already defined in net.minecraft.src.Block
    public ChunkCoordinates getBedSpawnPosition(World world, int x, int y, int z, EntityPlayer player)
                            ^

Block.java:1713: setBedOccupied(net.minecraft.src.World,int,int,int,net.minecraft.src.EntityPlayer,boolean) is already defined in net.minecraft.src.Block
    public void setBedOccupied(World world, int x, int y, int z, EntityPlayer player, boolean occupied)
                ^

Block.java:1728: getBedDirection(net.minecraft.src.IBlockAccess,int,int,int) is already defined in net.minecraft.src.Block
    public int getBedDirection(IBlockAccess world, int x, int y, int z)
              ^

Block.java:1742: isBedFoot(net.minecraft.src.IBlockAccess,int,int,int) is already defined in net.minecraft.src.Block
    public boolean isBedFoot(IBlockAccess world, int x, int y, int z)
                  ^

Chunk.java:186: Chunk(net.minecraft.src.World,byte[],byte[],int,int) is already defined in net.minecraft.src.Chunk
    public Chunk(World world, byte[] ids, byte[] metadata, int chunkX, int chunkZ)
          ^

Chunk.java:1578: cleanChunkBlockTileEntity(int,int,int) is already defined in net.minecraft.src.Chunk
    public void cleanChunkBlockTileEntity(int x, int y, int z)
                ^

NetworkManager.java:15: forge.ForgeHooks is already defined in a single-type import
import net.minecraft.src.forge.ForgeHooks;
^

TileEntity.java:234: canUpdate() is already defined in net.minecraft.src.TileEntity
    public boolean canUpdate()
                  ^

TileEntity.java:248: onDataPacket(net.minecraft.src.NetworkManager,net.minecraft.src.Packet132TileEntityData) is already defined in net.minecraft.src.TileEntity
    public void onDataPacket(NetworkManager net, Packet132TileEntityData pkt){}
                ^

BlockLeaves.java:331: isShearable(net.minecraft.src.ItemStack,net.minecraft.src.World,int,int,int) is already defined in net.minecraft.src.BlockLeaves
    public boolean isShearable(ItemStack item, World world, int x, int y, int z)
                  ^

BlockLeaves.java:337: onSheared(net.minecraft.src.ItemStack,net.minecraft.src.World,int,int,int,int) is already defined in net.minecraft.src.BlockLeaves
    public ArrayList<ItemStack> onSheared(ItemStack item, World world, int x, int y, int z, int fortune)
                                ^

BlockFire.java:464: canBlockCatchFire(net.minecraft.src.IBlockAccess,int,int,int,int) is already defined in net.minecraft.src.BlockFire
    public boolean canBlockCatchFire(IBlockAccess world, int x, int y, int z, int face)
                  ^

BlockFire.java:485: getChanceToEncourageFire(net.minecraft.src.World,int,int,int,int,int) is already defined in net.minecraft.src.BlockFire
    public int getChanceToEncourageFire(World world, int x, int y, int z, int oldChance, int face)
              ^

TileEntityBrewingStand.java:338: getStartInventorySide(int) is already defined in net.minecraft.src.TileEntityBrewingStand
public int getStartInventorySide(int side)
          ^

TileEntityBrewingStand.java:344: getSizeInventorySide(int) is already defined in net.minecraft.src.TileEntityBrewingStand
public int getSizeInventorySide(int side)
          ^

ItemShears.java:8: forge.IShearable is already defined in a single-type import
import net.minecraft.src.forge.IShearable;
^

Enchantment.java:205: canEnchantItem(net.minecraft.src.ItemStack) is already defined in net.minecraft.src.Enchantment
    public boolean canEnchantItem(ItemStack item)
                  ^

PlayerController.java:5: forge.ForgeHooks is already defined in a single-type import
import net.minecraft.src.forge.ForgeHooks;
^

EntityRenderer.java:9: forge.ForgeHooksClient is already defined in a single-type import
import net.minecraft.src.forge.ForgeHooksClient;
^

SoundManager.java:8: forge.ForgeHooksClient is already defined in a single-type import
import net.minecraft.src.forge.ForgeHooksClient;
^

SoundManager.java:9: forge.ModCompatibilityClient is already defined in a single-type import
import net.minecraft.src.forge.ModCompatibilityClient;
^

EntityMinecart.java:1421: dropCartAsItem() is already defined in net.minecraft.src.EntityMinecart
    public void dropCartAsItem()
                ^

EntityMinecart.java:1435: getItemsDropped() is already defined in net.minecraft.src.EntityMinecart
    public List<ItemStack> getItemsDropped()
                          ^

EntityMinecart.java:1459: getCartItem() is already defined in net.minecraft.src.EntityMinecart
    public ItemStack getCartItem()
                    ^

EntityMinecart.java:1468: isPoweredCart() is already defined in net.minecraft.src.EntityMinecart
    public boolean isPoweredCart()
                  ^

EntityMinecart.java:1479: isStorageCart() is already defined in net.minecraft.src.EntityMinecart
    public boolean isStorageCart()
                  ^

EntityMinecart.java:1488: canBeRidden() is already defined in net.minecraft.src.EntityMinecart
    public boolean canBeRidden()
                  ^

EntityMinecart.java:1502: canUseRail() is already defined in net.minecraft.src.EntityMinecart
    public boolean canUseRail()
                  ^

EntityMinecart.java:1512: setCanUseRail(boolean) is already defined in net.minecraft.src.EntityMinecart
    public void setCanUseRail(boolean use)
                ^

EntityMinecart.java:1521: shouldDoRailFunctions() is already defined in net.minecraft.src.EntityMinecart
    public boolean shouldDoRailFunctions()
                  ^

EntityMinecart.java:1530: getMinecartType() is already defined in net.minecraft.src.EntityMinecart
    public int getMinecartType()
              ^

EntityMinecart.java:1540: getCollisionHandler() is already defined in net.minecraft.src.EntityMinecart
    public static IMinecartCollisionHandler getCollisionHandler()
                                            ^

EntityMinecart.java:1550: setCollisionHandler(forge.IMinecartCollisionHandler) is already defined in net.minecraft.src.EntityMinecart
    public static void setCollisionHandler(IMinecartCollisionHandler handler)
                      ^

EntityMinecart.java:1559: getDrag() is already defined in net.minecraft.src.EntityMinecart
    protected double getDrag()
                    ^

EntityMinecart.java:1568: applyDragAndPushForces() is already defined in net.minecraft.src.EntityMinecart
    protected void applyDragAndPushForces()
                  ^

EntityMinecart.java:1599: updatePushForces() is already defined in net.minecraft.src.EntityMinecart
    protected void updatePushForces()
                  ^

EntityMinecart.java:1625: moveMinecartOnRail(int,int,int) is already defined in net.minecraft.src.EntityMinecart
    protected void moveMinecartOnRail(int i, int j, int k)
                  ^

EntityMinecart.java:1653: moveMinecartOffRail(int,int,int) is already defined in net.minecraft.src.EntityMinecart
    protected void moveMinecartOffRail(int i, int j, int k)
                  ^

EntityMinecart.java:1693: updateFuel() is already defined in net.minecraft.src.EntityMinecart
    protected void updateFuel()
                  ^

EntityMinecart.java:1704: adjustSlopeVelocities(int) is already defined in net.minecraft.src.EntityMinecart
    protected void adjustSlopeVelocities(int metadata)
                  ^

EntityMinecart.java:1737: getMaxSpeedRail() is already defined in net.minecraft.src.EntityMinecart
    public float getMaxSpeedRail()
                ^

EntityMinecart.java:1742: setMaxSpeedRail(float) is already defined in net.minecraft.src.EntityMinecart
    public void setMaxSpeedRail(float value)
                ^

EntityMinecart.java:1747: getMaxSpeedGround() is already defined in net.minecraft.src.EntityMinecart
    public float getMaxSpeedGround()
                ^

EntityMinecart.java:1752: setMaxSpeedGround(float) is already defined in net.minecraft.src.EntityMinecart
    public void setMaxSpeedGround(float value)
                ^

EntityMinecart.java:1757: getMaxSpeedAirLateral() is already defined in net.minecraft.src.EntityMinecart
    public float getMaxSpeedAirLateral()
                ^

EntityMinecart.java:1762: setMaxSpeedAirLateral(float) is already defined in net.minecraft.src.EntityMinecart
    public void setMaxSpeedAirLateral(float value)
                ^

EntityMinecart.java:1767: getMaxSpeedAirVertical() is already defined in net.minecraft.src.EntityMinecart
    public float getMaxSpeedAirVertical()
                ^

EntityMinecart.java:1772: setMaxSpeedAirVertical(float) is already defined in net.minecraft.src.EntityMinecart
    public void setMaxSpeedAirVertical(float value)
                ^

EntityMinecart.java:1777: getDragAir() is already defined in net.minecraft.src.EntityMinecart
    public double getDragAir()
                  ^

EntityMinecart.java:1782: setDragAir(double) is already defined in net.minecraft.src.EntityMinecart
    public void setDragAir(double value)
                ^

NetworkManager.java:454: getNetHandler() is already defined in net.minecraft.src.NetworkManager
    public NetHandler getNetHandler()
                      ^

ItemShears.java:112: useItemOnEntity(net.minecraft.src.ItemStack,net.minecraft.src.EntityLiving) is already defined in net.minecraft.src.ItemShears
    public void useItemOnEntity(ItemStack itemstack, EntityLiving entity)
                ^

ItemShears.java:138: onBlockStartBreak(net.minecraft.src.ItemStack,int,int,int,net.minecraft.src.EntityPlayer) is already defined in net.minecraft.src.ItemShears
    public boolean onBlockStartBreak(ItemStack itemstack, int x, int y, int z, EntityPlayer player)
                  ^

WorldRenderer.java:8: forge.ForgeHooksClient is already defined in a single-type import
import net.minecraft.src.forge.ForgeHooksClient;
^

EffectRenderer.java:23: effectList is already defined in net.minecraft.src.EffectRenderer
    private Hashtable<String, ArrayList<EntityFX>> effectList = new Hashtable<String, ArrayList<EntityFX>>();
                                                  ^

RenderItem.java:8: forge.ForgeHooksClient is already defined in a single-type import
import net.minecraft.src.forge.ForgeHooksClient;
^

RenderItem.java:9: forge.IItemRenderer is already defined in a single-type import
import net.minecraft.src.forge.IItemRenderer;
^

RenderItem.java:10: forge.MinecraftForgeClient is already defined in a single-type import
import net.minecraft.src.forge.MinecraftForgeClient;
^

SoundManager.java:330: getSoundSystem() is already defined in net.minecraft.src.SoundManager
    public static SoundSystem getSoundSystem()
                              ^

SoundManager.java:335: getSoundsPool() is already defined in net.minecraft.src.SoundManager
    public SoundPool getSoundsPool()
                    ^

SoundManager.java:340: getStreamingPool() is already defined in net.minecraft.src.SoundManager
    public SoundPool getStreamingPool()
                    ^

100 errors
==================

Client recompile failed, correct source then rerun updatemd5
!! Can not find server sources, try decompiling !!
=================================== Minecraft Forge Setup Finished =================================

 

 

 

Am I doing something wrong?

 

Thanks in advance

[LexManos]

Do not put Forge into your jar, Only modloader. Nothing else.

[fomenta]

Thank you Lex, that was the problem.  Sorry for the oversight and thanks for the super-fast response!