Added notest option for template gen and allow for default naming

Author: mitchcapper

build/BuildScriptGenerator.csx

@@ -42,11 +42,11 @@ void Main() {
 		While the options below are the common options technically you can specify any option in default_config.ini. When specifying an option you use its name without BLD_CONFIG_ prefix and case does not matter.
 
 		The default value for each arg is in brackets.
-		
+
 		For array args you can specify multiple separated by spaces until the next arg";
 	usage += "\n" + bsgOpts.UsageStr();
 
-	var BUILD_NAME = opts.GetVal("BUILD_NAME");
+	var BUILD_NAME = opts.GetVal("BUILD_NAME",true);
 	if (showUsage || String.IsNullOrWhiteSpace(BUILD_NAME)) {
 		Console.Error.WriteLine(usage);
 		Environment.Exit(1);
@@ -74,9 +74,9 @@ void Main() {
 
 	var context = new TemplateContext();
 	context.PushGlobal(scriptObjs);
-	
-	
-	
+
+
+
 	var full = template.Render(context).Replace("\r","");
 	var blankRegex = new Regex(@"[ \t]*\n([ \t]*\n){2,}", RegexOptions.Singleline);
 	full = blankRegex.Replace(full,"\n\n");
@@ -122,7 +122,7 @@ class BSGOpts {
 		AddOtherOpt(BSG_OPT.GitRepo, "Primary git repository for cloning", "https://github.com/mitchcapper/BUILD_APP_NAME.git");
 		AddOtherOpt(BSG_OPT.HaveOurPatch, "Do we do our normal patch apply for this repo (needs repo patch file in patches dir)", 1);
 		AddOtherOpt(BSG_OPT.NoGnuLibButAutoconf, "Project does not use GNULib but does need an autoconf block, automatically sets  GNU_LIBS_USED=0", 0);
-		
+
 	}
 	public string UsageStr() {
 		StringBuilder ret = new();
@@ -167,23 +167,25 @@ class BSGOpts {
 
 class Opts {
 	private ConfigRead config;
+	public void CheckAllOpts() {
+		GetArrVal("fullcheck"); // we will iterate everything and throw if any are invalid
+	}
 
 	public Opts(ConfigRead config) {
 		this.config = config;
 	}
 	public bool HasVal(String name) => GetArrVal(name).Count != 0;
-	public string GetVal(String name) {
+	public string GetVal(String name,bool allowMissing=false) {
 		var vals = GetArrVal(name);
-		if (vals.Count == 0)
+		if (vals.Count == 0) {
+			if (allowMissing)
+				return null;
 			throw new Exception($"No value found for: {name}");
+		}
 		if (vals.Count > 1)
 			throw new Exception($"Expecting only one value for: {name} but got multiple: {String.Join(", ",vals)}");
 		return vals[0];
 	}
-
-	public void CheckAllOpts() {
-		GetArrVal("fullcheck"); // we will iterate everything and throw if any are invalid
-	}
 	public static List<string> allCLIArgs = new(Environment.GetCommandLineArgs());
 	public List<string> AllPassedConfigOpts=new();
 	private readonly string[] OPTS_ALLOWED_TO_HAVE_UNESCAPED_DOUBLE_DASH_START = new []{"CONFIG_CMD_ADDL","CONFIG_CMD_ADDL_DEBUG","CONFIG_CMD_ADDL_STATIC","CONFIG_CMD_ADDL_SHARED","CONFIG_CMD_DEFAULT","CONFIG_CMD_GNULIB_ADDL", "GNU_LIBS_BOOTSTRAP_EXTRAS_ADD" };
@@ -192,7 +194,7 @@ class Opts {
 		var including = false;
 		var isFullCheck = name == "fullcheck";
 		var RequestedOptRealName = BSGOpts.GetRealOptNameStr(name) ?? config.GetRealName(name);
-		
+
 		if (String.IsNullOrWhiteSpace(RequestedOptRealName) && ! isFullCheck)
 			throw new Exception($"Internal error requesting a manual option of: {name} doesn't seem valid");
 		string curOptName = null;

build/f_TEMPLATE_build.sbn-sh

@@ -66,6 +66,9 @@ fi
 {{else}}
 	cmake_make
 	cmake_install
+{{end}}
+{{if ! ours.enabled CONFIG_NO_TESTS }}
+	make check
 {{end}}
 	finalcommon;
 }