Important fixes for versions with no server

Author: Lassebq

src/main/java/org/mcphackers/mcp/MCP.java

@@ -36,15 +36,6 @@ public static void main(String[] args) {
         input = new Scanner(System.in);
         logger.log("Operating system: " + System.getProperty("os.name"));
         logger.log("RetroMCP " + VERSION);
-        
-        if(Files.exists(Paths.get(MCPConfig.VERSION)))
-        {
-			try {
-				VersionsParser.setCurrentVersion(new String(Files.readAllBytes(Paths.get(MCPConfig.VERSION))));
-			} catch (Exception e) {
-				logger.info(new Ansi().fgBrightRed().a("Unable to get current version!").fgDefault().toString());
-			}
-        }
 
         boolean startedWithNoParams = false;
         boolean exit = false;
@@ -54,6 +45,11 @@ public static void main(String[] args) {
             logger.println(logo);
         }
         if(Files.exists(Paths.get(MCPConfig.VERSION))) {
+			try {
+				VersionsParser.setCurrentVersion(new String(Files.readAllBytes(Paths.get(MCPConfig.VERSION))));
+			} catch (Exception e) {
+				logger.info(new Ansi().fgBrightRed().a("Unable to get current version!").fgDefault().toString());
+			}
         	logger.info(new Ansi().a("Current version: ").fgBrightCyan().a(VersionsParser.getCurrentVersion()).fgDefault().toString());
         }
         if (args.length <= 0) {

src/main/java/org/mcphackers/mcp/tasks/TaskDecompile.java

@@ -71,24 +71,35 @@ public void doTask() throws Exception {
         for (Path path : new Path[] { Paths.get(tinyOut), Paths.get(excOut), Paths.get(srcZip)}) {
         	Files.deleteIfExists(path);
         }
+    	FileUtil.createDirectories(Paths.get(MCPConfig.TEMP));
 		FileUtil.deleteDirectoryIfExists(ffOut);
 		while(step < STEPS) {
 		    step();
 		    switch (step) {
 			case REMAP:
-		        TinyRemapper remapper = null;
-		
-		        try (OutputConsumerPath outputConsumer = new OutputConsumerPath.Builder(Paths.get(tinyOut)).build()) {
-		            remapper = remap(TinyUtils.createTinyMappingProvider(mappings, "official", "named"), originalJar, outputConsumer, getLibraryPaths(side));
-		            outputConsumer.addNonClassFiles(originalJar, NonClassCopyMode.FIX_META_INF, remapper);
-		        } finally {
-		            if (remapper != null) {
-		                remapper.finish();
-		            }
-		        }
+		    	if (Files.exists(mappings)) {
+			        TinyRemapper remapper = null;
+			
+			        try (OutputConsumerPath outputConsumer = new OutputConsumerPath.Builder(Paths.get(tinyOut)).build()) {
+			            remapper = remap(TinyUtils.createTinyMappingProvider(mappings, "official", "named"), originalJar, outputConsumer, getLibraryPaths(side));
+			            outputConsumer.addNonClassFiles(originalJar, NonClassCopyMode.FIX_META_INF, remapper);
+			        } finally {
+			            if (remapper != null) {
+			                remapper.finish();
+			            }
+			        }
+		    	}
+		    	else {
+		    		Files.copy(originalJar, Paths.get(tinyOut));
+		    	}
 		        break;
 		    case EXCEPTOR:
-			    MCInjector.process(tinyOut, excOut, exc, 0);
+		    	if (Files.exists(Paths.get(exc))) {
+		    		MCInjector.process(tinyOut, excOut, exc, 0);
+				}
+		    	else {
+		    		Files.copy(Paths.get(tinyOut), Paths.get(excOut));
+		    	}
 			    // Copying a fixed jar to libs
 			    if(side == CLIENT) {
 			    	Files.deleteIfExists(Paths.get(MCPConfig.CLIENT_FIXED));

src/main/java/org/mcphackers/mcp/tasks/TaskRecompile.java

@@ -105,6 +105,7 @@ public void recompile(JavaCompiler compiler, DiagnosticCollector<JavaFileObject>
                                   		diagnostic.getSource().getName(),
                                   		diagnostic.getMessage(null)));
         	}
+        mgr.close();
         if (!success) {
             throw new RuntimeException("Compilation error!");
         }