buildfile: validate arguments

Author: scivision

buildfile.m

@@ -68,7 +68,7 @@
                          Tag="python", ...
                          SourceFiles=pkg_root, RunOnlyImpactedTests=true,...
                          TestResults="TestResults_python.xml", Strict=true);
- 
+
   addons = matlab.addons.installedAddons;
   if contains(addons.Name, "Matlab Test")
     plan("coverage") = matlab.buildtool.tasks.TestTask(test_root, ...
@@ -189,6 +189,13 @@ function build_exe(context)
 
 
 function [comp, shell] = get_compiler(lang)
+arguments (Input)
+  lang (1,1) string {mustBeMember(lang, ["c", "c++", "fortran"])}
+end
+arguments (Output)
+  comp string {mustBeScalarOrEmpty}
+  shell string {mustBeScalarOrEmpty}
+end
 
 lang = lower(lang);
 
@@ -211,7 +218,6 @@ function build_exe(context)
       if isempty(comp)
         disp("set CC environment variable to the C compiler path, or do 'mex -setup c'")
       end
-    otherwise, error("language not known " + lang)
   end
 else
   comp = co.Details.CompilerExecutable;
@@ -238,10 +244,18 @@ function build_exe(context)
 
 
 function [comp, shell, outFlag] = get_build_cmd(lang)
+arguments (Input)
+  lang (1,1) string {mustBeMember(lang, ["c", "c++", "fortran"])}
+end
+arguments (Output)
+  comp (1,1) string
+  shell (1,1) string
+  outFlag (1,1) string
+end
 
 [comp, shell] = get_compiler(lang);
 
-if contains(shell, "Visual Studio") || contains(comp, "ifx.exe")
+if any(contains(shell, "Visual Studio")) || endsWith(comp, "ifx.exe")
   outFlag = "/Fo" + tempdir + " /link /out:";
 else
   outFlag = "-o";
@@ -251,9 +265,12 @@ function build_exe(context)
 
 
 function srcs = get_mex_sources(build_all)
-arguments
+arguments (Input)
   build_all (1,1) logical = false
 end
+arguments (Output)
+  srcs cell
+end
 
 srcs = {
 "src/remove.cpp", ...
@@ -272,6 +289,10 @@ function build_exe(context)
 
 
 function [compiler_opt, linker_opt] = get_compiler_options()
+arguments (Output)
+  compiler_opt (1,:) string
+  linker_opt (1,1) string
+end
 
 cxx = mex.getCompilerConfigurations('c++');
 flags = cxx.Details.CompilerFlags;
@@ -318,8 +339,9 @@ function build_exe(context)
 
 
 function comp = get_fortran_compiler()
-
-disp("set FC environment variable to the Fortran compiler path, or do 'mex -setup fortran'")
+arguments (Output)
+  comp string {mustBeScalarOrEmpty}
+end
 
 if ismac()
   p = '/opt/homebrew/bin/';
@@ -331,13 +353,14 @@ function build_exe(context)
   if isempty(p)
     p = getenv("MW_MINGW64_LOC");
   end
-  if ~isempty(p) && ~endsWith(p, ["bin", "bin/"])
+  if ~endsWith(p, ["bin", "bin/"])
     p = p + "/bin";
   end
 else
   p = '';
 end
 
+comp = string.empty;
 for fc = ["flang", "gfortran", "ifx"]
   comp = stdlib.which(fc, p);
   if ~isempty(comp)
@@ -347,4 +370,6 @@ function build_exe(context)
   end
 end
 
+disp("to hint Fortran compiler, setenv('FC', <Fortran compiler path>), or do 'mex -setup fortran'")
+
 end

test/is_capable.m

@@ -57,7 +57,7 @@ function is_capable(tc, f)
 
 elseif contains(n, ".native.")
 
-  if endsWith(n, "set_permissions")
+  if endsWith(n, ["is_exe", "is_readable", "is_writable", "set_permissions"])
     tc.assumeFalse(isMATLABReleaseOlderThan('R2025a'))
   end