#439 this was sort of a racing condition

What happened what that the boards.txt is processed line by line. In the
esp file the upload.tool is specified 3 times for the tested board.
Once as upload.tool and 2 times for the upload protocol.
Apparently the upload.tool was processed after the menu.
So one could say that this is a bug in the boards.txt file.
However I catered for this situation.

Author: jantje

it.baeyens.arduino.core/src/it/baeyens/arduino/tools/Helpers.java

@@ -730,7 +730,11 @@ private static void setTheEnvironmentVariablesAddtheBoardsTxt(IContributedEnviro
 		String keyString = MakeKeyString(currentPair.getKey());
 		String valueString = MakeEnvironmentString(currentPair.getValue(), Const.ENV_KEY_BOARD_START);
 		contribEnv.addVariable(new EnvironmentVariable(keyString, valueString), confDesc);
-	    } else {
+	    }
+	}
+	for (Entry<String, String> currentPair : boardSectionMap.entrySet()) {
+	    // if it is not a menu item add it
+	    if (currentPair.getKey().startsWith(Messages.Helpers_menu)) {
 
 		String[] keySplit = currentPair.getKey().split("\\."); //$NON-NLS-1$
 		String menuID = keySplit[1];