[vcpkg ci] Update formatting CI

scripts/azure-pipelines/azure-pipelines.yml

@@ -5,7 +5,36 @@ variables:
   windows-pool: 'PrWin-2020-06-30'
   linux-pool: 'PrLin-2020-06-30'
 
-jobs:
+stages:
+- stage: check_cxx_formatting
+  displayName: 'Check the formatting of the C++'
+  pool: $(windows-pool)
+  jobs:
+  - job:
+    steps:
+    - task: Powershell@2
+      displayName: 'Check C++ Formatting'
+      inputs:
+        filePath: 'scripts/azure-pipelines/windows/Check-CxxFormatting.ps1'
+        arguments: '-Toolsrc ./toolsrc'
+- stage: check_manifest_formatting
+  displayName: Check the formatting of port manifests
+  pool: $(windows-pool)
+  dependsOn: []
+  jobs:
+  - job:
+    steps:
+    - task: Powershell@2
+      displayName: 'Check port manifest Formatting'
+      inputs:
+        filePath: 'scripts/azure-pipelines/windows/Check-ManifestFormatting.ps1'
+        arguments: '-PortsTree ./ports'
+- stage: run_port_ci
+  displayName: 'Run the Port CI'
+  dependsOn:
+  - check_formatting
+  - check_manifest_formatting
+  jobs:
   - template: windows/azure-pipelines.yml
     parameters:
       triplet: x86-windows

scripts/azure-pipelines/windows/check-formatting.ps1 → scripts/azure-pipelines/windows/Check-CxxFormatting.ps1

@@ -1,3 +1,5 @@
+#Requires -Version 3.0
+
 [CmdletBinding()]
 Param(
     [Parameter(Mandatory=$True)]
@@ -26,10 +28,7 @@ try
 
     & $clangFormat -style=file -i @fileNames
 
-    $changedFiles = git status --porcelain $Toolsrc | ForEach-Object {
-        (-split $_)[1]
-    }
-
+    $changedFiles = & "$PSScriptRoot/Get-ChangedFiles.ps1" -Directory $Toolsrc
     if (-not $IgnoreErrors -and $null -ne $changedFiles)
     {
         $msg = @(

scripts/azure-pipelines/windows/Check-ManifestFormatting.ps1

@@ -0,0 +1,51 @@
+#Requires -Version 3.0
+
+[CmdletBinding()]
+Param(
+    [Parameter(Mandatory=$True)]
+    [string]$PortsTree,
+    [Parameter()]
+    [switch]$IgnoreErrors # allows one to just format
+)
+
+$vcpkgRoot = $PSScriptRoot ` # .../vcpkg/scripts/azure-pipelines/windows
+    | Split-Path ` # .../vcpkg/scripts/azure-pipelines
+    | Split-Path ` # .../vcpkg/scripts
+    | Split-Path # .../vcpkg
+
+$PortsTree = Get-Item $PortsTree
+
+if (-not (Test-Path "$vcpkgRoot/.vcpkg-root"))
+{
+    Write-Error "The vcpkg root was not at $vcpkgRoot; did the script get moved?"
+    throw
+}
+
+if (-not (Test-Path "$vcpkgRoot/vcpkg.exe"))
+{
+    & "$vcpkgRoot/bootstrap-vcpkg.bat"
+    if (-not $?)
+    {
+        Write-Error "Bootstrapping vcpkg failed"
+        throw
+    }
+}
+
+& "$vcpkgRoot/vcpkg.exe" 'x-format-manifest' '--all'
+$changedFiles = & "$PSScriptRoot/Get-ChangedFiles.ps1" -Directory $PortsTree
+if (-not $IgnoreErrors -and $null -ne $changedFiles)
+{
+    $msg = @(
+        "",
+        "The formatting of the manifest files didn't match our expectation.",
+        "If your build fails here, you need to run:"
+    )
+    $msg += "    vcpkg x-format-manifest --all"
+    $msg += ""
+
+    $msg += "vcpkg should produce the following diff:"
+    $msg += git diff $Toolsrc
+
+    Write-Error ($msg -join "`n")
+    throw
+}

scripts/azure-pipelines/windows/Get-ChangedFiles.ps1

@@ -0,0 +1,9 @@
+[CmdletBinding()]
+Param(
+    [Parameter(Mandatory=$True)]
+    [string]$Directory
+)
+
+git status --porcelain $Directory | ForEach-Object {
+    (-split $_)[1]
+}

scripts/azure-pipelines/windows/azure-pipelines.yml

@@ -20,13 +20,7 @@ jobs:
     condition: always()
     inputs:
       filePath: 'scripts/azure-pipelines/windows/disk-space.ps1'
-  - task: Powershell@2
-    displayName: 'Check C++ Formatting'
-    condition: eq('${{ parameters.triplet }}', 'x86-windows')
-    inputs:
-      filePath: 'scripts/azure-pipelines/windows/check-formatting.ps1'
-      arguments: '-Toolsrc ./toolsrc'
-    # Note: D: is the Azure machines' temporary disk.
+  # Note: D: is the Azure machines' temporary disk.
   - task: CmdLine@2
     displayName: 'Build vcpkg'
     inputs: