Regarding possible upgrade issues from R3->R4 with ports

[Doridian]

The changed class files from R3->R4 were
Spoiler

net/minecraft/server/BlockFurnace.class
net/minecraft/server/BlockPumpkin.class
net/minecraft/server/BlockTNT.class
net/minecraft/server/Chunk.class
net/minecraft/server/Entity.class
net/minecraft/server/EntityArrow.class
net/minecraft/server/EntityBoat.class
net/minecraft/server/EntityEgg.class
net/minecraft/server/EntityEnderCrystal.class
net/minecraft/server/EntityEnderDragon.class
net/minecraft/server/EntityEnderPearl.class
net/minecraft/server/EntityEnderman.class
net/minecraft/server/EntityFireball.class
net/minecraft/server/EntityFishingHook.class
net/minecraft/server/EntityHuman.class
net/minecraft/server/EntityLiving.class
net/minecraft/server/EntityMinecart.class
net/minecraft/server/EntityPainting.class
net/minecraft/server/EntityPotion.class
net/minecraft/server/EntitySmallFireball.class
net/minecraft/server/EntitySnowball.class
net/minecraft/server/EntityTracker.class
net/minecraft/server/EntityTrackerEntry.class
net/minecraft/server/EntityWolf.class
net/minecraft/server/ItemBlock.class
net/minecraft/server/ItemMonsterEgg.class
net/minecraft/server/ItemStep.class
net/minecraft/server/ItemWorldMap.class
net/minecraft/server/MobEffectList.class
net/minecraft/server/NetServerHandler.class
net/minecraft/server/ServerConfigurationManager.class
net/minecraft/server/SpawnerCreature.class
net/minecraft/server/World.class
org/bukkit/craftbukkit/CraftServer.class
org/bukkit/craftbukkit/CraftWorld.class
org/bukkit/craftbukkit/entity/CraftLivingEntity.class
org/bukkit/craftbukkit/entity/CraftPlayer.class
org/bukkit/craftbukkit/entity/CraftThrownPotion.class
org/bukkit/craftbukkit/event/CraftEventFactory.class
org/bukkit/craftbukkit/inventory/CraftFurnaceRecipe.class
org/bukkit/craftbukkit/inventory/CraftShapedRecipe.class
org/bukkit/craftbukkit/inventory/CraftShapelessRecipe.class
org/bukkit/craftbukkit/potion/CraftPotionBrewer.class
org/bukkit/craftbukkit/potion/CraftPotionEffectType.class

Changed files of MCForge + ModLoaderMP
Spoiler

net/minecraft/server/BaseMod.class
net/minecraft/server/BaseModMp.class
net/minecraft/server/Block.class
net/minecraft/server/BlockButton.class
net/minecraft/server/BlockChest.class
net/minecraft/server/BlockContainer.class
net/minecraft/server/BlockCrops.class
net/minecraft/server/BlockDiode.class
net/minecraft/server/BlockDispenser.class
net/minecraft/server/BlockDoor.class
net/minecraft/server/BlockFire.class
net/minecraft/server/BlockLadder.class
net/minecraft/server/BlockLever.class
net/minecraft/server/BlockLongGrass.class
net/minecraft/server/BlockMinecartTrack.class
net/minecraft/server/BlockNetherWart.class
net/minecraft/server/BlockPiston.class
net/minecraft/server/BlockPressurePlate.class
net/minecraft/server/BlockRedstoneWire.class
net/minecraft/server/BlockSnow.class
net/minecraft/server/BlockTorch.class
net/minecraft/server/BlockTrapdoor.class
net/minecraft/server/Chunk.class
net/minecraft/server/ChunkProviderServer.class
net/minecraft/server/CraftingManager.class
net/minecraft/server/Enchantment.class
net/minecraft/server/EnchantmentManager.class
net/minecraft/server/Entity.class
net/minecraft/server/EntityHuman.class
net/minecraft/server/EntityLiving.class
net/minecraft/server/EntityMinecart.class
net/minecraft/server/EntityPigZombie.class
net/minecraft/server/EntityPlayer.class
net/minecraft/server/EntityTracker.class
net/minecraft/server/EntityTrackerEntry.class
net/minecraft/server/EntityTrackerEntry2.class
net/minecraft/server/Explosion.class
net/minecraft/server/FurnaceRecipes.class
net/minecraft/server/ISpawnable.class
net/minecraft/server/Item.class
net/minecraft/server/ItemBucket.class
net/minecraft/server/ItemDye.class
net/minecraft/server/ItemHoe.class
net/minecraft/server/ItemInWorldManager.class
net/minecraft/server/ItemTool.class
net/minecraft/server/MinecartTrackLogic.class
net/minecraft/server/MinecraftServer.class
net/minecraft/server/MLProp.class
net/minecraft/server/ModLoader.class
net/minecraft/server/ModLoaderMp.class
net/minecraft/server/MovingObjectPosition.class
net/minecraft/server/NetLoginHandler.class
net/minecraft/server/NetServerHandler.class
net/minecraft/server/Packet230ModLoader.class
net/minecraft/server/Packet250CustomPayload.class
net/minecraft/server/Pair.class
net/minecraft/server/ServerConfigurationManager.class
net/minecraft/server/SlotResult.class
net/minecraft/server/SlotResult2.class
net/minecraft/server/TileEntity.class
net/minecraft/server/TileEntityBrewingStand.class
net/minecraft/server/TileEntityFurnace.class
net/minecraft/server/World.class
net/minecraft/server/WorldGenDungeons.class
net/minecraft/server/WorldServer.class
org/bukkit/craftbukkit/CraftServer.class
org/bukkit/craftbukkit/entity/CraftEntity.class
org/bukkit/craftbukkit/event/CraftEventFactory$1.class
org/bukkit/craftbukkit/event/CraftEventFactory.class
org/bukkit/Material.class

The conflicting files (files that both affect) are:

Code: [Select]

net/minecraft/server/Chunk.class
net/minecraft/server/Entity.class
net/minecraft/server/EntityHuman.class
net/minecraft/server/EntityLiving.class
net/minecraft/server/EntityMinecart.class
net/minecraft/server/EntityTracker.class
net/minecraft/server/EntityTrackerEntry.class
net/minecraft/server/NetServerHandler.class
net/minecraft/server/ServerConfigurationManager.class
net/minecraft/server/World.class
org/bukkit/craftbukkit/CraftServer.class
org/bukkit/craftbukkit/event/CraftEventFactory.class

Therefor it might be advised to re-port ModLoaderMP and MCForge and not use it with the current builds (whilist it might work, it might just void some of the recent patches and possibly even cause new bugs)

[Wolvan]

Thank you for that info man... Really nice

[Crazeh]

Normally I'd say wrong section, but this information is very useful and important. Good to have here so everyone can see. Very nice work on comparing them, never thought to do anything like that before.

[GadgetTvMan]

temporarily set as sticky.

[Crazeh]

I can actually confirm this now, while starting my local test server on R4(1938) server log shows the following:

Code: [Select]

14.02 07:26:13 [Server] INFO git-Bukkit-1.1-R3-b1846jnks (MC: 1.1)
I decided to do a little in game testing and run through some bug fixes to see if they held up. Bukkit 1846 had an issue where flaming arrows still hurt players even if PvP is disabled, which was fixed in Build 1938. I tested that with Bukkit 1938 with no mods, and it was fixed. However, once you drop mods in the issue returns.

Again thank you to the poster for this information, it's very helpful. As it is right now, your kind of updating, but not really fixing anything, which is...interesting.

-Crazeh

[Maeyanie]

Updating the APIs (MLMP, Forge, etc.) is always a good idea for new Bukkit versions. It's also pretty easy, since for minor changes like this set git does most of the work itself.

Regular mods should still work fine though.

[Doridian]

git does most of the work itself.

git always does most of the work by itself, you just need to know how to tell it to
[I made the above diff of bukkit using git diff]

[x3ndless]

doridian deffinetly making yourself usefull

[Doridian]

doridian deffinetly making yourself usefull

Thanks, I'll take that as a compliment.
Anyway, my next project is porting MCForge and MLMP to CraftBukkit++ (performance-optimized version of CB, https://github.com/Afforess/CraftBukkitPlusPlus), and, in case I succedd, ask for permission to post my ports obviously.

[Domochevsky]

doridian deffinetly making yourself usefull

Thanks, I'll take that as a compliment.
Anyway, my next project is porting MCForge and MLMP to CraftBukkit++ (performance-optimized version of CB, https://github.com/Afforess/CraftBukkitPlusPlus), and, in case I succedd, ask for permission to post my ports obviously.

Why hello there... that sounds like a great idea. (both the existence of CB++ and you porting this stuff. I presume regular mods (like BC/IC²) won't need to be ported and just work with those?)