Merge pull request #1609 from lumien231/1.8

Fixes #1603: Moving the start of the update thread to the pre init of the forge mod container

src/main/java/net/minecraftforge/common/ForgeModContainer.java

@@ -280,6 +280,11 @@ public class ForgeModContainer extends DummyModContainer implements WorldAccessC
         MinecraftForge.EVENT_BUS.register(MinecraftForge.INTERNAL_HANDLER);
         ForgeChunkManager.captureConfig(evt.getModConfigurationDirectory());
         FMLCommonHandler.instance().bus().register(this);
+        
+        if (!ForgeModContainer.disableVersionCheck)
+        {
+            ForgeVersion.startVersionCheck();
+        }
     }
 
     @Subscribe

src/main/java/net/minecraftforge/common/MinecraftForge.java

@@ -46,11 +46,6 @@ public class MinecraftForge
 
        OreDictionary.getOreName(0);
 
-       if (!ForgeModContainer.disableVersionCheck)
-       {
-           ForgeVersion.startVersionCheck();
-       }
-
        //Force these classes to be defined, Should prevent derp error hiding.
        new CrashReport("ThisIsFake", new Exception("Not real"));