Skip to content

Commit 9138062

Browse files
committed
Modify GH ant action to upload installer packages for testing
Signed-off-by: Taylor Smock <[email protected]>
1 parent 8d39d70 commit 9138062

File tree

2 files changed

+49
-10
lines changed

2 files changed

+49
-10
lines changed

.github/workflows/ant-test.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ jobs:
1919
fail-fast: false
2020
matrix:
2121
# test against latest update of each major Java version, as well as specific updates of LTS versions:
22-
java: [8, 11, 17, 21, 22-ea]
22+
java: [8, 11, 17, 21, 22, 23-ea]
2323
os: [ubuntu-latest, macos-latest, windows-latest]
2424
name: Java ${{ matrix.java }} on ${{ matrix.os }}
2525
steps:

.github/workflows/ant.yml

+48-9
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ env:
44
JAVAFX_VERSION: '17.0.7'
55
on:
66
push:
7-
branches: [master]
7+
branches: [master, windows-installer]
88

99
defaults:
1010
run:
@@ -59,6 +59,7 @@ jobs:
5959
JOSM release ${{ env.josm_release_tag }}
6060
draft: false
6161
prerelease: ${{ env.josm_prerelease }}
62+
if: github.ref == 'refs/heads/master'
6263

6364
build:
6465
needs: createrelease
@@ -69,7 +70,7 @@ jobs:
6970
fail-fast: false
7071
matrix:
7172
# test against latest update of each major Java version, as well as specific updates of LTS versions:
72-
java: [8, 11, 17, 21, 22-ea]
73+
java: [8, 11, 17, 21, 22, 23-ea]
7374
os: [ubuntu-latest, macos-latest, windows-latest]
7475
name: Java ${{ matrix.java }} on ${{ matrix.os }}
7576
steps:
@@ -113,18 +114,20 @@ jobs:
113114
SIGN_TSA: ${{ secrets.SIGN_TSA }}
114115
# Calls ant with -Dreleasebuild=true if we're a 'tested' build
115116
run: |
116-
export SIGN_KEYSTORE=certificate.p12
117-
echo "$SIGN_CERT" | base64 --decode > $SIGN_KEYSTORE
117+
if [ ! -z "${SIGN_CERT}" ]; then
118+
export SIGN_KEYSTORE=certificate.p12
119+
echo "$SIGN_CERT" | base64 --decode > $SIGN_KEYSTORE
120+
fi
118121
if [ "${{ needs.createrelease.outputs.josm_prerelease }}" == "true" ]; then
119122
ANT="ant"
120123
else
121124
ANT="ant -Dreleasebuild=true"
122125
fi
123126
$ANT dist
124-
rm $SIGN_KEYSTORE
127+
if [ ! -z "${SIGN_KEYSTORE}" ]; then rm $SIGN_KEYSTORE; fi
125128
126129
- name: Upload jar
127-
if: ${{ always() }}
130+
if: ${{ always() && needs.createrelease.outputs.upload_url }}
128131
env:
129132
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
130133
uses: actions/upload-release-asset@v1
@@ -134,6 +137,15 @@ jobs:
134137
asset_name: JOSM-${{ runner.os}}-java${{ matrix.java }}-${{ needs.createrelease.outputs.josm_revision }}.jar
135138
asset_content_type: application/java-archive
136139

140+
- name: Upload jar
141+
if: ${{ always() && !needs.createrelease.outputs.upload_url }}
142+
env:
143+
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
144+
uses: actions/upload-artifact@v4
145+
with:
146+
path: dist/josm-custom.jar
147+
name: JOSM-${{ runner.os}}-java${{ matrix.java }}-${{ needs.createrelease.outputs.josm_revision }}.jar
148+
137149
- name: Build and package for macOS
138150
if: ${{ runner.os == 'macos' && matrix.java != '8' && matrix.java != '11' && always() }}
139151
env:
@@ -172,7 +184,7 @@ jobs:
172184
./native/windows/win-jpackage.sh ${{ needs.createrelease.outputs.josm_revision }}
173185
174186
- name: Upload macOS app
175-
if: ${{ runner.os == 'macos' && matrix.java != '8' && matrix.java != '11' && always() }}
187+
if: ${{ runner.os == 'macos' && matrix.java != '8' && matrix.java != '11' && always() && needs.createrelease.outputs.upload_url }}
176188
uses: actions/upload-release-asset@v1
177189
env:
178190
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
@@ -182,8 +194,17 @@ jobs:
182194
asset_name: JOSM-${{ runner.os}}-java${{ matrix.java }}-${{ needs.createrelease.outputs.josm_revision }}.zip
183195
asset_content_type: application/zip
184196

197+
- name: Upload macOS app
198+
if: ${{ runner.os == 'macos' && matrix.java != '8' && matrix.java != '11' && always() && !needs.createrelease.outputs.upload_url }}
199+
uses: actions/upload-artifact@v4
200+
env:
201+
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
202+
with:
203+
path: app/JOSM.zip
204+
name: JOSM-${{ runner.os}}-java${{ matrix.java }}-${{ needs.createrelease.outputs.josm_revision }}.zip
205+
185206
- name: Upload Windows Installer executable
186-
if: ${{ runner.os == 'windows' && matrix.java != '8' && matrix.java != '11' && always() }}
207+
if: ${{ runner.os == 'windows' && matrix.java != '8' && matrix.java != '11' && always() && needs.createrelease.outputs.upload_url }}
187208
uses: actions/upload-release-asset@v1
188209
env:
189210
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
@@ -193,8 +214,17 @@ jobs:
193214
asset_name: JOSM-${{ runner.os}}-java${{ matrix.java }}-${{ needs.createrelease.outputs.josm_revision }}.exe
194215
asset_content_type: application/vnd.microsoft.portable-executable
195216

217+
- name: Upload Windows Installer executable
218+
if: ${{ runner.os == 'windows' && matrix.java != '8' && matrix.java != '11' && always() && !needs.createrelease.outputs.upload_url }}
219+
uses: actions/upload-artifact@v4
220+
env:
221+
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
222+
with:
223+
path: app/JOSM.exe
224+
name: JOSM-${{ runner.os}}-java${{ matrix.java }}-${{ needs.createrelease.outputs.josm_revision }}.exe
225+
196226
- name: Upload Windows Installer package
197-
if: ${{ runner.os == 'windows' && matrix.java != '8' && matrix.java != '11' && always() }}
227+
if: ${{ runner.os == 'windows' && matrix.java != '8' && matrix.java != '11' && always() && needs.createrelease.outputs.upload_url }}
198228
uses: actions/upload-release-asset@v1
199229
env:
200230
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
@@ -203,3 +233,12 @@ jobs:
203233
asset_path: app/JOSM.msi
204234
asset_name: JOSM-${{ runner.os}}-java${{ matrix.java }}-${{ needs.createrelease.outputs.josm_revision }}.msi
205235
asset_content_type: application/x-ole-storage
236+
237+
- name: Upload Windows Installer package
238+
if: ${{ runner.os == 'windows' && matrix.java != '8' && matrix.java != '11' && always() && !needs.createrelease.outputs.upload_url }}
239+
uses: actions/upload-artifact@v4
240+
env:
241+
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
242+
with:
243+
path: app/JOSM.msi
244+
name: JOSM-${{ runner.os}}-java${{ matrix.java }}-${{ needs.createrelease.outputs.josm_revision }}.msi

0 commit comments

Comments
 (0)