rooty/Arduino
1
0
Fork 0
mirror of https://github.com/arduino/Arduino.git synced 2025-01-17 06:52:18 +01:00
Code Issues Releases Activity

SAM boards stop compiling due to way of handling params with spaces on different OSs. Fixed

Browse Source
This commit is contained in:
Federico Fissore 2015-04-13 12:05:00 +02:00
parent 4c9e5fc564
commit c740f251f4
2 changed files with 39 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
arduino-core/src/processing/app/debug/Compiler.java
View File

@ -54,6 +54,7 @@ import processing.app.preproc.PdePreprocessor;
import processing.app.legacy.PApplet;
import processing.app.packages.LegacyUserLibrary;
import processing.app.packages.UserLibrary;
import processing.app.tools.ArgumentsWithSpaceAwareCommandLine;
public class Compiler implements MessageConsumer {
@ -319,10 +320,10 @@ public class Compiler implements MessageConsumer {
if (maxDataSize > 0) {
System.out
.println(I18n
.format(
_("Global variables use {0} bytes ({2}%%) of dynamic memory, leaving {3} bytes for local variables. Maximum is {1} bytes."),
dataSize, maxDataSize, dataSize * 100 / maxDataSize,
maxDataSize - dataSize));
.format(
_("Global variables use {0} bytes ({2}%%) of dynamic memory, leaving {3} bytes for local variables. Maximum is {1} bytes."),
dataSize, maxDataSize, dataSize * 100 / maxDataSize,
maxDataSize - dataSize));
} else {
System.out.println(I18n
.format(_("Global variables use {0} bytes of dynamic memory."), dataSize));
@ -736,9 +737,9 @@ public class Compiler implements MessageConsumer {
}
});
CommandLine commandLine = new CommandLine(command[0]);
CommandLine commandLine = new ArgumentsWithSpaceAwareCommandLine(command[0]);
for (int i = 1; i < command.length; i++) {
commandLine.addArgument(command[i]);
commandLine.addArgument(command[i], false);
}
int result;

32
arduino-core/src/processing/app/tools/ArgumentsWithSpaceAwareCommandLine.java Normal file
View File

@ -0,0 +1,32 @@
package processing.app.tools;
import org.apache.commons.exec.CommandLine;
import processing.app.BaseNoGui;
import processing.app.Platform;
import processing.app.helpers.OSUtils;
import java.io.File;
public class ArgumentsWithSpaceAwareCommandLine extends CommandLine {
public ArgumentsWithSpaceAwareCommandLine(String executable) {
super(executable);
}
public ArgumentsWithSpaceAwareCommandLine(File executable) {
super(executable);
}
public ArgumentsWithSpaceAwareCommandLine(CommandLine other) {
super(other);
}
@Override
public CommandLine addArgument(String argument, boolean handleQuoting) {
if (argument.contains(" ") && OSUtils.isWindows()) {
argument = argument.replaceAll("\"", "").replaceAll("'", "");
}
return super.addArgument(argument, handleQuoting);
}
}