Add support to cache deps

Author: shivammathur

.github/workflows/php.yml

@@ -62,9 +62,32 @@ jobs:
         with:
           name: artifacts
 
+      - name: Get Cache Key
+        id: cache-key
+        run: |
+          $pv = '${{inputs.php-version}}'
+          if('${{inputs.php-version}}' -ne 'master') {
+            $pv = '${{inputs.php-version}}'.Split('.')[0..1] -join '.'
+          }
+          $vsVersion = (Get-Content -Raw -Path (Join-Path $(pwd).Path '\php\BuildPhp\config\vs.json') | ConvertFrom-Json).php.$pv
+          $packagesListUrl = "https://downloads.php.net/~windows/php-sdk/deps/series/packages-$pv-$vsVersion-${{ matrix.arch }}-staging.txt"
+          Invoke-WebRequest -OutFile packages.txt -Uri $packagesListUrl
+          Add-Content -Value "cache-key=deps-$pv-${{ matrix.arch }}" -Path $env:GITHUB_OUTPUT
+          Add-Content -Value "cache-dir=C:\deps-$pv-${{ matrix.arch }}" -Path $env:GITHUB_OUTPUT
+
+      - name: Cache Deps
+        id: cache-deps
+        uses: actions/cache@v4
+        with:
+          path: ${{ steps.cache-key.outputs.cache-dir }}
+          key: ${{ steps.cache-key.outputs.cache-key }}-${{ hashFiles('packages.txt') }}
+
       - name: Test PHP
         shell: pwsh
         continue-on-error: true
+        env:
+          DEPS_DIR: ${{ steps.cache-key.outputs.cache-dir }}
+          DEPS_CACHE_HIT: ${{ steps.cache-deps.outputs.cache-hit }}
         run: |
           Import-Module (Join-Path $(pwd).Path '\php\BuildPhp') -Force          
           Invoke-PhpTests -PhpVersion ${{inputs.php-version}} `
@@ -79,6 +102,7 @@ jobs:
         with:
           name: test-results-${{matrix.arch}}-${{matrix.ts}}-${{matrix.opcache}}-${{matrix.test-type}}
           path: test-${{matrix.arch}}-${{matrix.ts}}-${{matrix.opcache}}-${{matrix.test-type}}.xml
+
   upload:
     runs-on: ubuntu-latest
     needs: tests

.github/workflows/test.yml

@@ -20,9 +20,32 @@ jobs:
       - name: Checkout
         uses: actions/checkout@v5
 
+      - name: Get Cache Key
+        id: cache-key
+        run: |
+          $pv = '${{inputs.php-version}}'
+          if('${{inputs.php-version}}' -ne 'master') {
+            $pv = '${{inputs.php-version}}'.Split('.')[0..1] -join '.'
+          }
+          $vsVersion = (Get-Content -Raw -Path (Join-Path $(pwd).Path '\php\BuildPhp\config\vs.json') | ConvertFrom-Json).php.$pv
+          $packagesListUrl = "https://downloads.php.net/~windows/php-sdk/deps/series/packages-$pv-$vsVersion-${{ matrix.arch }}-staging.txt"
+          Invoke-WebRequest -OutFile packages.txt -Uri $packagesListUrl
+          Add-Content -Value "cache-key=deps-$pv-${{ matrix.arch }}" -Path $env:GITHUB_OUTPUT
+          Add-Content -Value "cache-dir=C:\deps-$pv-${{ matrix.arch }}" -Path $env:GITHUB_OUTPUT
+
+      - name: Cache Deps
+        id: cache-deps
+        uses: actions/cache@v4
+        with:
+          path: ${{ steps.cache-key.outputs.cache-dir }}
+          key: ${{ steps.cache-key.outputs.cache-key }}-${{ hashFiles('packages.txt') }}
+
       - name: Run Tests
         shell: pwsh
         continue-on-error: true
+        env:
+          DEPS_DIR: ${{ steps.cache-key.outputs.cache-dir }}
+          DEPS_CACHE_HIT: ${{ steps.cache-deps.outputs.cache-hit }}
         run: |
           Import-Module (Join-Path $(pwd).Path '\php\BuildPhp') -Force
           Invoke-PhpTests -PhpVersion ${{inputs.php-version}} `

php/BuildPhp/private/Add-TestRequirements.ps1

@@ -71,13 +71,22 @@ function Add-TestRequirements {
         }
 
         Get-PhpSdk
-        $env:DEPS_DIR = "$currentDirectory/../deps"
-        New-Item "$env:DEPS_DIR" -ItemType "directory" -Force > $null 2>&1
+        $FetchDeps = $False
+        if($null -eq $env:DEPS_DIR) {
+            $majorMinorVersion = ($PhpVersion -split '\.')[0..1] -join '.'
+            $env:DEPS_DIR = "C:\deps-$majorMinorVersion-$Arch"
+            $FetchDeps = $True
+        }
+        if(-not(Test-Path $env:DEPS_DIR)) {
+            New-Item "$env:DEPS_DIR" -ItemType "directory" -Force > $null 2>&1
+        }
         $branch = if ($PhpVersion -eq 'master') {'master'} else {($PhpVersion -split '\.')[0..1] -join '.'}
         $env:PHP_SDK_VS = $VsVersion
         $env:PHP_SDK_ARCH = $Arch
         $bat_content = @()
-        $bat_content += "cmd /c phpsdk_deps --update --force --no-backup --branch $branch --stability staging --deps $env:DEPS_DIR"
+        if($FetchDeps -eq $True -or $null -eq $Env:DEPS_CACHE_HIT -or $Env:DEPS_CACHE_HIT -ne 'true') {
+            $bat_content += "cmd /c phpsdk_deps --update --force --no-backup --branch $branch --stability staging --deps $env:DEPS_DIR"
+        }
         $bat_content += "editbin /stack:8388608 $binDirectoryPath\php.exe"
         $bat_content += "editbin /stack:8388608 $binDirectoryPath\php-cgi.exe"
         Set-Content -Encoding "ASCII" -Path vs.bat -Value $bat_content