#927 #984 pre/post link and pre build

added support for pre_link post_link and pre_build though pre_build is
not documented :-(

Author: jantje

io.sloeber.core/plugin.xml

@@ -342,7 +342,7 @@
            <tool
                  announcement="%tool.Arduino.combiner.announcement"
                  command="${A.RECIPE.C.COMBINE.PATTERN}"
-                 commandLinePattern="${A.RECIPE.C.COMBINE.PATTERN.1} ${FLAGS} ${INPUTS} ${A.RECIPE.C.COMBINE.PATTERN.2} ${A.ARCHIVE_FILE_PATH} ${A.RECIPE.C.COMBINE.PATTERN.3} ${JANTJE.EXTRA.ALL} ${JANTJE.EXTRA.LINK}"
+                 commandLinePattern="${A.JANTJE.PRE.LINK}${A.RECIPE.C.COMBINE.PATTERN.1} ${FLAGS} ${INPUTS} ${A.RECIPE.C.COMBINE.PATTERN.2} ${A.ARCHIVE_FILE_PATH} ${A.RECIPE.C.COMBINE.PATTERN.3} ${JANTJE.EXTRA.ALL} ${JANTJE.EXTRA.LINK}${A.JANTJE.POST.LINK}"
                  errorParsers="org.eclipse.cdt.core.GLDErrorParser"
                  id="io.sloeber.tool.combine"
                  name="%tool.combiner.name"

io.sloeber.core/src/io/sloeber/core/toolchain/ArduinoGnuMakefileGenerator.java

@@ -1132,6 +1132,18 @@ private StringBuffer addTargets(List<String> outputVarsAdditionsList, boolean re
 		// IConfiguration config = info.getDefaultConfiguration();
 		// Assemble the information needed to generate the targets
 		String prebuildStep = config.getPrebuildStep();
+		//JABA issue927 adding recipe.hooks.sketch.prebuild.NUMBER.pattern as cdt prebuild command if needed
+		ICConfigurationDescription confDesc = ManagedBuildManager.getDescriptionForConfiguration(config);
+        String sketchPrebuild= io.sloeber.core.common.Common.getBuildEnvironmentVariable(confDesc, "A.JANTJE.PREBUILD", new String(), false);
+        if (!sketchPrebuild.isEmpty()) {
+            String separator = new String();
+            if (!prebuildStep.isEmpty()) {
+                prebuildStep = prebuildStep + "\n\t" + sketchPrebuild;
+            } else {
+                prebuildStep = sketchPrebuild;
+            }
+		}
+		//end off JABA issue927 
 		try {
 			// try to resolve the build macros in the prebuild step
 			prebuildStep = ManagedBuildManager.getBuildMacroProvider().resolveValueToMakefileFormat(prebuildStep, EMPTY_STRING, WHITESPACE, IBuildMacroProvider.CONTEXT_CONFIGURATION, config);

io.sloeber.core/src/io/sloeber/core/tools/Helpers.java

@@ -934,11 +934,39 @@ private static void setTheEnvironmentVariablesPostProcessing(IContributedEnviron
 		Collections.sort(objcopyCommand);
 		setBuildEnvironmentVariable(contribEnv, confDesc, "JANTJE.OBJCOPY", StringUtil.join(objcopyCommand, "\n\t"));
 
+		//handle the hooks
+		setHookBuildEnvironmentVariable(contribEnv, confDesc, "A.JANTJE.PRE.LINK","A.RECIPE.HOOKS.LINKING.PRELINK.XX.PATTERN",false);
+		setHookBuildEnvironmentVariable(contribEnv, confDesc, "A.JANTJE.POST.LINK","A.RECIPE.HOOKS.LINKING.POSTLINK.XX.PATTERN",true);
+		setHookBuildEnvironmentVariable(contribEnv, confDesc, "A.JANTJE.PREBUILD","A.RECIPE.HOOKS.PREBUILD.XX.PATTERN",false);
+
 	}
 
 
 
-	/**
+    private static void setHookBuildEnvironmentVariable(IContributedEnvironment contribEnv,
+            ICConfigurationDescription confDesc, String varName, String hookName, boolean post) {
+        String envVarString = new String();
+        String postSeparator = "}\n\t";
+        String preSeparator = "${";
+        if (post) {
+            postSeparator = "${";
+            preSeparator = "}\n\t";
+        }
+        for (int numDigits = 1; numDigits <= 2; numDigits++) {
+            int counter = 1;
+            String hookVarName = hookName.replace("XX",
+                    String.format("%0" + Integer.toString(numDigits) + "d", new Integer(counter)));
+            while (!getBuildEnvironmentVariable(confDesc, hookVarName, "", true).isEmpty()) {
+                envVarString = envVarString + preSeparator + hookVarName + postSeparator;
+                hookVarName = hookName.replace("XX",
+                        String.format("%0" + Integer.toString(numDigits) + "d", new Integer(++counter)));
+            }
+            if (!envVarString.isEmpty()) {
+                setBuildEnvironmentVariable(contribEnv, confDesc, varName, envVarString);
+            }
+        }
+    }
+    /**
 	 * When parsing boards.txt and platform.txt some processing needs to be done to
 	 * get "acceptable environment variable values" This method does the parsing
 	 * {xx} is replaced with ${XX} if to uppercase is true {xx} is replaced with