Added method to determine board FQBN

This simplifies a bit the organization of the arduino-builder calls.

Author: cmaglie

arduino-core/src/cc/arduino/Compiler.java

@@ -172,16 +172,14 @@ public String build(List<CompilerProgressListener> progListeners, boolean export
       throw new RunnerException("Board is not selected");
     }
 
-    TargetPlatform platform = board.getContainerPlatform();
-    TargetPackage aPackage = platform.getContainerPackage();
     String vidpid = VIDPID();
 
-    PreferencesMap prefs = loadPreferences(board, platform, aPackage, vidpid);
+    PreferencesMap prefs = loadPreferences(board, vidpid);
 
     MessageConsumerOutputStream out = new MessageConsumerOutputStream(new ProgressAwareMessageConsumer(new I18NAwareMessageConsumer(System.out, System.err), progListeners), "\n");
     MessageConsumerOutputStream err = new MessageConsumerOutputStream(new I18NAwareMessageConsumer(System.err, Compiler.this), "\n");
 
-    callArduinoBuilder(board, platform, aPackage, vidpid, BuilderAction.COMPILE, out, err);
+    callArduinoBuilder(board, vidpid, BuilderAction.COMPILE, out, err);
 
     out.flush();
     err.flush();
@@ -206,8 +204,6 @@ public String codeComplete(ArrayList<CompilerProgressListener> progListeners, Fi
       throw new RunnerException("Board is not selected");
     }
 
-    TargetPlatform platform = board.getContainerPlatform();
-    TargetPackage aPackage = platform.getContainerPackage();
     String vidpid = VIDPID();
 
     ByteArrayOutputStream completions = new ByteArrayOutputStream();
@@ -217,7 +213,7 @@ public String codeComplete(ArrayList<CompilerProgressListener> progListeners, Fi
     codeCompleteFile = file;
     codeCompleteLine = line;
     codeCompleteCol = col;
-    callArduinoBuilder(board, platform, aPackage, vidpid, BuilderAction.CODE_COMPLETE, out, err);
+    callArduinoBuilder(board, vidpid, BuilderAction.CODE_COMPLETE, out, err);
 
     out.flush();
     err.flush();
@@ -240,12 +236,12 @@ private String VIDPID() {
     return vid.toUpperCase() + "_" + pid.toUpperCase();
   }
 
-  private PreferencesMap loadPreferences(TargetBoard board, TargetPlatform platform, TargetPackage aPackage, String vidpid) throws RunnerException, IOException {
+  private PreferencesMap loadPreferences(TargetBoard board, String vidpid) throws RunnerException, IOException {
     ByteArrayOutputStream stdout = new ByteArrayOutputStream();
     ByteArrayOutputStream stderr = new ByteArrayOutputStream();
     MessageConsumerOutputStream err = new MessageConsumerOutputStream(new I18NAwareMessageConsumer(new PrintStream(stderr), Compiler.this), "\n");
     try {
-      callArduinoBuilder(board, platform, aPackage, vidpid, BuilderAction.DUMP_PREFS, stdout, err);
+      callArduinoBuilder(board, vidpid, BuilderAction.DUMP_PREFS, stdout, err);
     } catch (RunnerException e) {
       System.err.println(new String(stderr.toByteArray()));
       throw e;
@@ -262,7 +258,7 @@ private void addPathFlagIfPathExists(List<String> cmd, String flag, File folder)
     }
   }
 
-  private void callArduinoBuilder(TargetBoard board, TargetPlatform platform, TargetPackage aPackage, String vidpid, BuilderAction action, OutputStream outStream, OutputStream errStream) throws RunnerException {
+  private void callArduinoBuilder(TargetBoard board, String vidpid, BuilderAction action, OutputStream outStream, OutputStream errStream) throws RunnerException {
     List<String> cmd = new ArrayList<>();
     cmd.add(BaseNoGui.getContentFile("arduino-builder").getAbsolutePath());
     cmd.add(action.value);
@@ -284,8 +280,7 @@ private void callArduinoBuilder(TargetBoard board, TargetPlatform platform, Targ
     addPathFlagIfPathExists(cmd, "-built-in-libraries", BaseNoGui.getContentFile("libraries"));
     addPathFlagIfPathExists(cmd, "-libraries", BaseNoGui.getSketchbookLibrariesFolder().folder);
 
-    String fqbn = Stream.of(aPackage.getId(), platform.getId(), board.getId(), boardOptions(board)).filter(s -> !s.isEmpty()).collect(Collectors.joining(":"));
-    cmd.add("-fqbn=" + fqbn);
+    cmd.add("-fqbn=" + getBoardFQBN(board));
 
     if (!"".equals(vidpid)) {
       cmd.add("-vid-pid=" + vidpid);
@@ -514,7 +509,17 @@ protected Thread createPump(InputStream is, OutputStream os, boolean closeWhenEx
     }
   }
 
-  private String boardOptions(TargetBoard board) {
+  public static String getBoardFQBN(TargetBoard board) {
+    TargetPlatform plat = board.getContainerPlatform();
+    TargetPackage pack = plat.getContainerPackage();
+    String fqbn = pack.getId() + ":" + plat.getId() + ":" + board.getId();
+    String opts = boardOptions(board);
+    if (!opts.isEmpty())
+      fqbn += ":" + opts;
+    return fqbn;
+  }
+
+  private static String boardOptions(TargetBoard board) {
     return board.getMenuIds().stream()
       .filter(board::hasMenu)
       .filter(menuId -> {