The script you are modifying is the native build script, it's always run by specifying architecture when called by msbuild -- https://github.com/dotnet/machinelearning/blob/0a7a0291fc04e4356dab25c614c43fad524b8e9f/src/Native/Native.proj#L101C60-L101C78

Were you making this change to improve the experience when a user directly tries to build just the native portion?

I'm double-checking that you weren't trying to fix the top-level build script. That one is here -- https://github.com/dotnet/machinelearning/blob/main/build.sh

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 69.02%. Comparing base (3abbb17) to head (711919f).
⚠️ Report is 3 commits behind head on main.

@@           Coverage Diff           @@
##             main    #7549   +/-   ##
=======================================
  Coverage   69.01%   69.02%           
=======================================
  Files        1482     1482           
  Lines      274093   274093           
  Branches    28266    28266           
=======================================
+ Hits       189176   189199   +23     
+ Misses      77531    77513   -18     
+ Partials     7386     7381    -5     

I see.
I got an error on the native part when building from the root. But because the terminal logger doesn't show much detail, I went to build it directly to debug it. Then, the architecture didn't work, so I fixed that and got an error that libmf submodule is missing. After checking it out, I got it built but I never retried clean re-build from the root.
So my root build issue was missing submodule, but it was hidden in the build output because terminal logger eats up the native build output.

But still, this makes it on-par with Windows, so let's keep it.

The errors are: dotnet/dnceng#5015