Enable building of extensions in the internal build pipeline using property directly instead of switch

[joperezr]

Description

Official builds are currently broken since arcade updated the build scripts that were edited in this change: #11504 causing the buildExtension switch to not be handled correctly. Instead of trying to pass in a switch, this change just directly passes in the property to the build which will make it so the extension gets built.

FYI: @radical @adamint

Checklist

• Is this feature complete?
  • Yes. Ready to ship.
• Are you including unit tests for the changes and scenario tests if relevant?
  • No
• Did you add public API?
  • No
• Does the change make any security assumptions or guarantees?
  • No
• Does the change require an update in our Aspire docs?
  • No

[github-actions]

🚀 Dogfood this PR with:

⚠️ WARNING: Do not do this without first carefully reviewing the code of this PR to satisfy yourself it is safe.

curl -fsSL https://raw.githubusercontent.com/dotnet/aspire/main/eng/scripts/get-aspire-cli-pr.sh | bash -s -- 11678

Or

• Run remotely in PowerShell:

iex "& { $(irm https://raw.githubusercontent.com/dotnet/aspire/main/eng/scripts/get-aspire-cli-pr.ps1) } 11678"

[Copilot]

Pull Request Overview

This change fixes broken official builds by modifying how extensions are built in the internal build pipeline. Instead of using a switch parameter that isn't being handled correctly after arcade build script updates, the change passes the property directly to the build command.

• Removes the -buildExtension switch from the build command
• Adds /p:BuildExtension=true property to directly enable extension building
• Includes a minor formatting fix in the azure-pipelines.yml file

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
eng/pipelines/templates/BuildAndTest.yml	Replaces -buildExtension switch with /p:BuildExtension=true property
eng/pipelines/azure-pipelines.yml	Minor formatting fix for policheck configuration

[radical]

Should we also go ahead and remove the -buildExtension, and --build-extension params from the sh and ps1 scripts?

[davidfowl]

Why? Don't we want this for PRs?

[joperezr]

bash I think is working fine with the switch (at least that seemed to be the case locally). I do think that this should be cleaned up but perhaps we do want to actually fix it so the switch is supported? I'll leave that up to @adamint but I think we should not block this PR on that as currently the builds from main are broken.