Skip to content

Commit ee138aa

Browse files
committed
fixup! Sign with SignPath
1 parent b10eaa4 commit ee138aa

3 files changed

Lines changed: 54 additions & 18 deletions

File tree

.github/workflows/build.yml

Lines changed: 51 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -69,6 +69,7 @@ jobs:
6969
!Src/Setup/Output/*.skin
7070
!Src/Setup/Output/*.skin7
7171
!Src/Setup/Output/*.zip
72+
retention-days: 1
7273

7374
- name: Sign binaries with SignPath
7475
uses: signpath/github-action-submit-signing-request@v2
@@ -95,6 +96,7 @@ jobs:
9596
name: MSI
9697
path: |
9798
Src/Setup/Temp/*.msi
99+
retention-days: 1
98100

99101
- name: Sign installers with SignPath
100102
uses: signpath/github-action-submit-signing-request@v2
@@ -108,49 +110,82 @@ jobs:
108110
wait-for-completion: true
109111
output-artifact-directory: 'Src/Setup/Temp/'
110112

111-
- name: Build final
113+
- name: Build setup and symbols
112114
shell: cmd
113115
env:
114116
CS_VERSION: ${{ steps.versioning.outputs.NEW_VERSION }}
115117
run: Src\Setup\BuildArchives.bat
116118

117-
- name: Upload final
118-
id: upload-final
119+
- name: Upload symbols
119120
uses: actions/upload-artifact@v7
120121
with:
121-
name: Final
122122
path: |
123-
Src/Setup/Final/
124-
!Src/Setup/Final/OpenShellLoc.zip
123+
Src/Setup/Final/OpenShellSymbols*.7z
124+
archive: false
125125

126-
- name: Sign final artifacts with SignPath
126+
- name: Upload utility
127+
uses: actions/upload-artifact@v7
128+
with:
129+
path: |
130+
Src/Setup/Final/Utility.exe
131+
archive: false
132+
133+
- name: Upload setup
134+
id: upload-setup
135+
uses: actions/upload-artifact@v7
136+
with:
137+
path: |
138+
Src/Setup/Final/OpenShellSetup*.exe
139+
archive: false
140+
141+
- name: Sign setup with SignPath
142+
# if: github.event_name == 'workflow_dispatch' && github.ref == 'refs/heads/master' # Only manual master builds
127143
uses: signpath/github-action-submit-signing-request@v2
128144
with:
129145
api-token: '${{ secrets.SIGNPATH_API_TOKEN }}'
130146
organization-id: 'b34b60e3-e5bf-4a6e-a13c-dcf641b4362c'
131147
project-slug: 'Open-Shell-Menu'
132148
signing-policy-slug: 'test-signing'
133-
artifact-configuration-slug: 'Final'
134-
github-artifact-id: '${{ steps.upload-final.outputs.artifact-id }}'
149+
github-artifact-id: '${{ steps.upload-setup.outputs.artifact-id }}'
135150
wait-for-completion: true
151+
skip-decompress: true
136152
output-artifact-directory: 'Src/Setup/Final/'
137153

138-
- name: Upload final (signed)
154+
# `overwrite: true` doesn't work with `archive: false`, so we have to delete the original first
155+
# https://github.com/actions/upload-artifact/issues/769
156+
# https://github.com/actions/upload-artifact/issues/785
157+
- name: Delete setup
158+
uses: geekyeggo/delete-artifact@v6
159+
with:
160+
name: OpenShellSetup*.exe
161+
162+
- name: Upload setup (signed)
139163
uses: actions/upload-artifact@v7
140164
with:
141-
name: Final-signed
142165
path: |
143-
Src/Setup/Final/
144-
!Src/Setup/Final/OpenShellLoc.zip
166+
Src/Setup/Final/OpenShellSetup*.exe
167+
archive: false
168+
overwrite: true
145169

146170
release:
147-
if: github.event_name == 'workflow_dispatch' && github.ref == 'refs/heads/master' # Only manual master builds
171+
# if: github.event_name == 'workflow_dispatch' && github.ref == 'refs/heads/master' # Only manual master builds
148172
needs: build
149173
runs-on: ubuntu-latest # Cheaper/faster than windows for just uploading
150174
permissions:
151175
contents: write # Elevate permissions ONLY for this job
152176
steps:
153-
- name: Download artifacts
177+
- name: Download setup
178+
uses: actions/download-artifact@v8
179+
with:
180+
pattern: OpenShellSetup*.exe
181+
- name: Download symbols
154182
uses: actions/download-artifact@v8
155183
with:
156-
name: OpenShell
184+
pattern: OpenShellSymbols*.7z
185+
- name: Download utility
186+
uses: actions/download-artifact@v8
187+
with:
188+
pattern: Utility.exe
189+
190+
- name: Display structure of downloaded files
191+
run: ls -R

Src/Setup/BuildArchives.bat

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -160,10 +160,10 @@ cd ..
160160

161161
cd Setup
162162

163-
copy /B ..\..\build\bin\Release\Utility.exe .\Final > nul
163+
copy /B Output\Utility.exe .\Final > nul
164164

165165
if defined APPVEYOR (
166-
appveyor PushArtifact ..\..\build\bin\Release\Utility.exe
166+
appveyor PushArtifact Output\Utility.exe
167167
)
168168

169169
exit /b 0

Src/Setup/BuildBinaries.bat

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,7 @@ copy /B ..\..\build\bin\Setup\StartMenuHelper32.dll Output > nul
5555
copy /B ..\..\build\bin\Release\Update.exe Output > nul
5656
copy /B ..\..\build\bin\Release\DesktopToasts.dll Output > nul
5757
copy /B ..\..\build\bin\Release\SetupHelper.exe Output > nul
58+
copy /B ..\..\build\bin\Release\Utility.exe Output > nul
5859

5960
copy /B ..\..\build\bin\SetupARM64\ClassicExplorer64.dll Output\ARM64 > nul
6061
copy /B ..\..\build\bin\SetupARM64\ClassicIEDLL_64.dll Output\ARM64 > nul

0 commit comments

Comments
 (0)