A bit of cleanup. Delimeter not figured out yet.

Author: ricklon

app/src/processing/app/debug/Compiler.java

@@ -596,7 +596,7 @@ static private String[] getCommandCompilerS(String avrBasePath,
 		};
 
 		String command = compileFormat.format(  Args );	
-		String[] commandArray = command.split(",");	
+		String[] commandArray = command.split("\|");	
 		return commandArray;
 	}
 
@@ -655,7 +655,7 @@ private String[] getCommandCompilerC(String avrBasePath,
 		};
 
 		String command = compileFormat.format(  Args );	
-		String[] commandArray = command.split(",");	
+		String[] commandArray = command.split("\|");	
 		return commandArray;	
 	}
 /*
@@ -714,7 +714,7 @@ static private String[] getCommandCompilerCPP(String avrBasePath,
 		};
 
 		String command = compileFormat.format(  Args );	
-		String[] commandArray = command.split(",");	
+		String[] commandArray = command.split("\|");	
 
 		/*
 		System.out.println("command:" + command);
@@ -863,7 +863,7 @@ void compileCore (String avrBasePath, String buildPath, String corePath, String
 		System.out.println("compileCore(...) substitute");
 
 			commandString = compileFormat.format(  Args );
-		    String[] commandArray = commandString.split(",");	
+		    String[] commandArray = commandString.split("\|");	
 			execAsynchronously(commandArray);
 
 
@@ -876,13 +876,12 @@ void compileLink(String avrBasePath, String buildPath, String corePath, ArrayLis
 	{	
 		System.out.println("compileLink: start");
 		String baseCommandString = configPreferences.get("recipe.c.combine.pattern");
-		System.out.println("baseCommandstring: " + baseCommandString);
 		String commandString = "";
 		MessageFormat compileFormat = new MessageFormat(baseCommandString);	
 		String objectFileList = "";
 
 		for (File file : objectFiles) {
-			objectFileList = objectFileList + file.getAbsolutePath() + ",";
+			objectFileList = objectFileList + file.getAbsolutePath() + "\|";
 		}
 		System.out.println("objectFileList: " + objectFileList);
 
@@ -901,13 +900,7 @@ void compileLink(String avrBasePath, String buildPath, String corePath, ArrayLis
 				configPreferences.get("ldscript"),	
 			};
 			commandString = compileFormat.format(  Args );
-		    String[] commandArray = commandString.split(",");	
-		 System.out.println("commandString: " + commandString); 
-		for (int ii = 0; ii < commandArray.length; ii++)
-		{
-			System.out.println("'" + commandArray[ii] + "'");
-		} 
-		    System.out.println("4. compileLink:prexec");
+		    String[] commandArray = commandString.split("\|");	
 			execAsynchronously(commandArray);
 	}
 
@@ -929,7 +922,7 @@ void compileEep (String avrBasePath, String buildPath, ArrayList<String> include
 			buildPath + File.separator + primaryClassName
 			};
 		commandString = compileFormat.format(  Args );		
-		String[] commandArray = commandString.split(",");	
+		String[] commandArray = commandString.split("\|");	
 	    execAsynchronously(commandArray);
 	}
 
@@ -951,7 +944,7 @@ void compileHex (String avrBasePath, String buildPath, ArrayList<String> include
 			buildPath + File.separator + primaryClassName
 			};
 		commandString = compileFormat.format(  Args );						
-		String[] commandArray = commandString.split(",");	
+		String[] commandArray = commandString.split("\|");	
 		execAsynchronously(commandArray);	
 	}
 
@@ -1023,7 +1016,7 @@ private static String preparePaths(ArrayList<String> includePaths) {
 		String includes = "";
 		for (int i = 0; i < includePaths.size(); i++) 
 		{
-			includes = includes + (" -I" + (String) includePaths.get(i)) + ",";
+			includes = includes + (" -I" + (String) includePaths.get(i)) + "\|";
 		}
 		//logger.debug("Paths prepared: " + includes);
 		return includes;

hardware/arduino/platforms.txt

@@ -2,38 +2,38 @@
 ##compile c object files
 ##Default.recipe, overide if overide exists, these defauls should remain the same, if you need to change them do it as an overide.
 
-#default.recipe.c.o.pattern={0}{1},{2},{3}{4},-DF_CPU={5},-D{6}={7},{8},{9},-o,{10}
-#default.recipe.cpp.o.pattern={0}{1},{2},{3}{4},-DF_CPU={5},-D{6}={7},{8},{9},-o,{10}
-#default.recipe.ar.pattern={0}{1},{2},{3}{4},{5}
-#default.recipe.c.combine.pattern={0}{1},{2},{3}{4},-o,{5}{6}.elf,{7},{8},-L{9},-lm
-#default.recipe.objcopy.eep.pattern={0}{1},{2},{3}.elf,{4}.eep
-#default.recipe.objcopy.hex.pattern={0}{1},{2},{3}.elf,{4}.hex
+#default.recipe.c.o.pattern={0}{1}|{2}|{3}{4}|-DF_CPU={5}|-D{6}={7}|{8}|{9}|-o|{10}
+#default.recipe.cpp.o.pattern={0}{1}|{2}|{3}{4}|-DF_CPU={5}|-D{6}={7}|{8}|{9}|-o|{10}
+#default.recipe.ar.pattern={0}{1}|{2}|{3}{4}|{5}
+#default.recipe.c.combine.pattern={0}{1}|{2}|{3}{4}|-o|{5}{6}.elf|{7}|{8}|-L{9}|-lm
+#default.recipe.objcopy.eep.pattern={0}{1}|{2}|{3}.elf|{4}.eep
+#default.recipe.objcopy.hex.pattern={0}{1}|{2}|{3}.elf|{4}.hex
 
 ########avr compile pattern ##########
 #avr.recipe.c.o.pattern={0=compiler.path}{1=compiler.c.cmd}{2=compiler.c.flags}{3=compiler.cpudef}{4=build.mcu}-DF_CPU={5=build.f_cpu}-D{7=ARDUINO}={6=Base.REVISION}{7=-I/INCLUDE_PATHS} {8=SOURCE_NAME} -o{9=OBJECT_NAME}
 #object name seems to have build path in it.
-avr.recipe.c.o.pattern={0}{1},{2},{3}{4},-DF_CPU={5},-D{6}={7},{8},{9},-o,{10}
+avr.recipe.c.o.pattern={0}{1}|{2}|{3}{4}|-DF_CPU={5}|-D{6}={7}|{8}|{9}|-o|{10}
 
 
 ##compile cc object files
 #avr.recipe.cc.o.pattern={0=compiler.path}{1=compiler.cc.cmd}{2=compiler.c.flags}{3=compiler.cpudef}{4=build.mcu}-DF_CPU={5=build.f_cpu}-DARDUINO={6=Base.REVISION}{-7=I/INCLUDE_PATHS} {8=SOURCE_NAME} -o{9=BUILD_PATH}{10=OBJECT_NAME}
-avr.recipe.cpp.o.pattern={0}{1},{2},{3}{4},-DF_CPU={5},-D{6}={7},{8},{9},-o,{10}
+avr.recipe.cpp.o.pattern={0}{1}|{2}|{3}{4}|-DF_CPU={5}|-D{6}={7}|{8}|{9}|-o|{10}
 ##create archives
 #avr.recipe.ar.pattern={0=compiler.path}{1=compiler.ar.cmd}{2=compiler.ar.flags}{3=BUILD_PATH}{4=CORE_NAME=core.a}{5=BUILD_PATH}{6=OBJECT_NAME}
-avr.recipe.ar.pattern={0}{1},{2},{3}{4},{5}
+avr.recipe.ar.pattern={0}{1}|{2}|{3}{4}|{5}
 
-##combine gc-sections, archives, and objects
+##combine gc-sections| archives, and objects
 #avr.recipe.c.combine.pattern={0=compiler.path}{1=compiler.c.cmd}{2=compiler.combine.flags}{3=compiler.cpudef}{4=build.mcu} -o {5=BUILD_PATH}{6=SOURCE_NAME}.elf {7=BUILD_PATH}{8=SOURCE_NAME}.o {9=BUILD_PATH}{10=CORE_NAME=core.a} -L{11=BUILD_PATH} -lm
-#avr.recipe.c.combine.pattern={0}{1},{2},{3}{4},-o,{5}{6}.elf,{7}{8},{9},-L{10},-lm
-avr.recipe.c.combine.pattern={0}{1},{2},{3}{4},-o,{5}{6}.elf,{7},{8},-L{9},-lm
+#avr.recipe.c.combine.pattern={0}{1}|{2}|{3}{4}|-o|{5}{6}.elf|{7}{8}|{9}|-L{10}|-lm
+avr.recipe.c.combine.pattern={0}{1}|{2}|{3}{4}|-o|{5}{6}.elf|{7}|{8}|-L{9}|-lm
 
 ##create eeprom
 #avr.recipe.objcopy.eep.pattern={0=compiler.path}{1=compiler.objcopy.cmd}{2=compiler.objcopy.eep.flags} {3=BUILD_PATH}{4=SOURCE_NAME}.elf {5=BUILD_PATH}{6=SOURCE_NAME}.eep
-avr.recipe.objcopy.eep.pattern={0}{1},{2},{3}.elf,{4}.eep
+avr.recipe.objcopy.eep.pattern={0}{1}|{2}|{3}.elf|{4}.eep
 
 ##create hex
 #avr.recipe.objcopy.hex.pattern={0=compiler.path}{1=compiler.objcopy.cmd}{2=compiler.objcopy.elf.flags} {3=BUILD_PATH}{4=SOURCE_NAME}.elf {5=BUILD_PATH}{6=SOURCE_NAME}.hex
-avr.recipe.objcopy.hex.pattern={0}{1},{2},{3}.elf,{4}.hex
+avr.recipe.objcopy.hex.pattern={0}{1}|{2}|{3}.elf|{4}.hex
 
 
 
@@ -42,17 +42,17 @@ avr.name=Arduino
 #avr.compiler.path Official default is correct, only need to change this if you want to overide the initial default
 #avr.compiler.path={0}/hardware/tools/avr/bin/
 avr.compiler.c.cmd=avr-gcc
-avr.compiler.c.flags=,-c,-g,-Os,-w,-ffunction-sections,-fdata-sections
-avr.compiler.c.elf.flags=,-Os,-Wl,--gc-sections
+avr.compiler.c.flags=|-c|-g|-Os|-w|-ffunction-sections|-fdata-sections
+avr.compiler.c.elf.flags=|-Os|-Wl|--gc-sections
 avr.compiler.c.elf.cmd=avr-gcc
-avr.compiler.S.flags=,-c,-g,-assembler-with-cpp
+avr.compiler.S.flags=|-c|-g|-assembler-with-cpp
 avr.compiler.cpp.cmd=avr-g++
-avr.compiler.cpp.flags=,-c,-g,-Os,-w,-fno-exceptions,-ffunction-sections,-fdata-sections
+avr.compiler.cpp.flags=|-c|-g|-Os|-w|-fno-exceptions|-ffunction-sections|-fdata-sections
 avr.compiler.ar.cmd=avr-ar
 avr.compiler.ar.flags=rcs
 avr.compiler.objcopy.cmd=avr-objcopy
-avr.compiler.objcopy.eep.flags=,-O,ihex,-j,.eeprom,--set-section-flags=.eeprom=alloc,load,--no-change-warnings,--change-section-lma,.eeprom=0
-avr.compiler.elf2hex.flags=,-O,ihex,-R,.eeprom
+avr.compiler.objcopy.eep.flags=|-O|ihex|-j|.eeprom|--set-section-flags=.eeprom=alloc|load|--no-change-warnings|--change-section-lma|.eeprom=0
+avr.compiler.elf2hex.flags=|-O|ihex|-R|.eeprom
 avr.compiler.elf2hex.cmd=avr-objcopy
 avr.compiler.ldflags=
 avr.compiler.cpudef=-mmcu=