Skip to content

Commit b3472ff

Browse files
committed
bump to latest windows-sdk and document better
specify timestamp algorithm. Apparently this is required in the latest version of the sdk
1 parent 4f0ee9c commit b3472ff

File tree

2 files changed

+10
-4
lines changed

2 files changed

+10
-4
lines changed

Diff for: .github/workflows/publish-go-nightly-task.yml

+5-2
Original file line numberDiff line numberDiff line change
@@ -172,7 +172,10 @@ jobs:
172172

173173
env:
174174
INSTALLER_CERT_WINDOWS_PFX: "/tmp/cert.pfx"
175-
SIGNTOOL_PATH: "C:/Program Files (x86)/Windows Kits/10/bin/10.0.17763.0/x86/signtool.exe"
175+
# We are hardcoding the path for signtool because is not present on the windows PATH env var by default.
176+
# Keep in mind that this path could change when upgrading to a new runner version
177+
# https://github.com/actions/runner-images/blob/main/images/win/Windows2022-Readme.md#installed-windows-sdks
178+
SIGNTOOL_PATH: "C:/Program Files (x86)/Windows Kits/10/bin/10.0.22621.0/x86/signtool.exe"
176179

177180
steps:
178181
- name: Checkout repository
@@ -206,7 +209,7 @@ jobs:
206209
MSI_FILE: ${{ steps.buildmsi.outputs.msi }} # this comes from .installer/cli.wixproj
207210
CERT_PASSWORD: ${{ secrets.INSTALLER_CERT_WINDOWS_PASSWORD }}
208211
run: |
209-
"${{ env.SIGNTOOL_PATH }}" sign -d "Arduino CLI" -f ${{ env.INSTALLER_CERT_WINDOWS_PFX}} -p ${{ env.CERT_PASSWORD }} -fd sha256 -tr http://timestamp.digicert.com -v "${{ env.MSI_FILE }}"
212+
"${{ env.SIGNTOOL_PATH }}" sign -d "Arduino CLI" -f ${{ env.INSTALLER_CERT_WINDOWS_PFX}} -p ${{ env.CERT_PASSWORD }} -fd sha256 -tr http://timestamp.digicert.com -td SHA256 -v "${{ env.MSI_FILE }}"
210213
211214
- name: Upload artifacts
212215
uses: actions/upload-artifact@v3

Diff for: .github/workflows/release-go-task.yml

+5-2
Original file line numberDiff line numberDiff line change
@@ -179,7 +179,10 @@ jobs:
179179

180180
env:
181181
INSTALLER_CERT_WINDOWS_PFX: "/tmp/cert.pfx"
182-
SIGNTOOL_PATH: "C:/Program Files (x86)/Windows Kits/10/bin/10.0.17763.0/x86/signtool.exe"
182+
# We are hardcoding the path for signtool because is not present on the windows PATH env var by default.
183+
# Keep in mind that this path could change when upgrading to a new runner version
184+
# https://github.com/actions/runner-images/blob/main/images/win/Windows2022-Readme.md#installed-windows-sdks
185+
SIGNTOOL_PATH: "C:/Program Files (x86)/Windows Kits/10/bin/10.0.22621.0/x86/signtool.exe"
183186

184187
steps:
185188
- name: Checkout repository
@@ -214,7 +217,7 @@ jobs:
214217
MSI_FILE: ${{ steps.buildmsi.outputs.msi }} # this comes from .installer/cli.wixproj
215218
CERT_PASSWORD: ${{ secrets.INSTALLER_CERT_WINDOWS_PASSWORD }}
216219
run: |
217-
"${{ env.SIGNTOOL_PATH }}" sign -d "Arduino CLI" -f ${{ env.INSTALLER_CERT_WINDOWS_PFX}} -p ${{ env.CERT_PASSWORD }} -fd sha256 -tr http://timestamp.digicert.com -v "${{ env.MSI_FILE }}"
220+
"${{ env.SIGNTOOL_PATH }}" sign -d "Arduino CLI" -f ${{ env.INSTALLER_CERT_WINDOWS_PFX}} -p ${{ env.CERT_PASSWORD }} -fd sha256 -tr http://timestamp.digicert.com -td SHA256 -v "${{ env.MSI_FILE }}"
218221
219222
- name: Upload artifacts
220223
uses: actions/upload-artifact@v3

0 commit comments

Comments
 (0)