Skip to content

Commit 3c65434

Browse files
committed
Create build without Binaries/Intermediates for the Marketplace.
1 parent cbd9d0c commit 3c65434

File tree

1 file changed

+30
-27
lines changed

1 file changed

+30
-27
lines changed

.github/workflows/build.yml

Lines changed: 30 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -114,21 +114,22 @@ jobs:
114114
with:
115115
name: CesiumForUnreal-52-windows-${{ env.CESIUM_UNREAL_VERSION}}
116116
path: combine
117-
- name: Unreal Marketplace Workaround
118-
run: |
119-
# The UE Marketplace deletes our Intermediates directory and fails to produces new
120-
# intermediates for the Linux platform. The Marketplace team has suggested we copy
121-
# them to the LinuxIntermediate directory instead, as a workaround. Users still have
122-
# to move these files to the correct place manually, though, in order for Linux builds
123-
# to succeed.
124-
mkdir -p combine/CesiumForUnreal/LinuxIntermediate/Build/Linux
125-
cp -r combine/CesiumForUnreal/Intermediate/Build/Linux/* combine/CesiumForUnreal/LinuxIntermediate/Build/Linux
126117
- name: Publish combined package artifact
127118
if: ${{ success() }}
128119
uses: actions/upload-artifact@v4
129120
with:
130121
name: ${{ env.BUILD_CESIUM_UNREAL_PACKAGE_NAME}}
131122
path: combine
123+
- name: Publish combined package artifact for the Unreal Marketplace
124+
if: ${{ success() }}
125+
uses: actions/upload-artifact@v4
126+
with:
127+
name: ${{ env.BUILD_CESIUM_UNREAL_PACKAGE_NAME}}-SourceOnly
128+
path: |
129+
combine
130+
# These are built by Epic, and including them seems to confuse their process.
131+
!combine/Binaries/
132+
!combine/Intermediates/
132133
TestPackage52:
133134
needs: [Combine52]
134135
uses: ./.github/workflows/testPackageOnWindows.yml
@@ -244,21 +245,22 @@ jobs:
244245
with:
245246
name: CesiumForUnreal-53-windows-${{ env.CESIUM_UNREAL_VERSION}}
246247
path: combine
247-
- name: Unreal Marketplace Workaround
248-
run: |
249-
# The UE Marketplace deletes our Intermediates directory and fails to produces new
250-
# intermediates for the Linux platform. The Marketplace team has suggested we copy
251-
# them to the LinuxIntermediate directory instead, as a workaround. Users still have
252-
# to move these files to the correct place manually, though, in order for Linux builds
253-
# to succeed.
254-
mkdir -p combine/CesiumForUnreal/LinuxIntermediate/Build/Linux
255-
cp -r combine/CesiumForUnreal/Intermediate/Build/Linux/* combine/CesiumForUnreal/LinuxIntermediate/Build/Linux
256248
- name: Publish combined package artifact
257249
if: ${{ success() }}
258250
uses: actions/upload-artifact@v4
259251
with:
260252
name: ${{ env.BUILD_CESIUM_UNREAL_PACKAGE_NAME}}
261253
path: combine
254+
- name: Publish combined package artifact for the Unreal Marketplace
255+
if: ${{ success() }}
256+
uses: actions/upload-artifact@v4
257+
with:
258+
name: ${{ env.BUILD_CESIUM_UNREAL_PACKAGE_NAME}}-SourceOnly
259+
path: |
260+
combine
261+
# These are built by Epic, and including them seems to confuse their process.
262+
!combine/Binaries/
263+
!combine/Intermediates/
262264
TestPackage53:
263265
needs: [Combine53]
264266
uses: ./.github/workflows/testPackageOnWindows.yml
@@ -374,21 +376,22 @@ jobs:
374376
with:
375377
name: CesiumForUnreal-54-windows-${{ env.CESIUM_UNREAL_VERSION}}
376378
path: combine
377-
- name: Unreal Marketplace Workaround
378-
run: |
379-
# The UE Marketplace deletes our Intermediates directory and fails to produces new
380-
# intermediates for the Linux platform. The Marketplace team has suggested we copy
381-
# them to the LinuxIntermediate directory instead, as a workaround. Users still have
382-
# to move these files to the correct place manually, though, in order for Linux builds
383-
# to succeed.
384-
mkdir -p combine/CesiumForUnreal/LinuxIntermediate/Build/Linux
385-
cp -r combine/CesiumForUnreal/Intermediate/Build/Linux/* combine/CesiumForUnreal/LinuxIntermediate/Build/Linux
386379
- name: Publish combined package artifact
387380
if: ${{ success() }}
388381
uses: actions/upload-artifact@v4
389382
with:
390383
name: ${{ env.BUILD_CESIUM_UNREAL_PACKAGE_NAME}}
391384
path: combine
385+
- name: Publish combined package artifact for the Unreal Marketplace
386+
if: ${{ success() }}
387+
uses: actions/upload-artifact@v4
388+
with:
389+
name: ${{ env.BUILD_CESIUM_UNREAL_PACKAGE_NAME}}-SourceOnly
390+
path: |
391+
combine
392+
# These are built by Epic, and including them seems to confuse their process.
393+
!combine/Binaries/
394+
!combine/Intermediates/
392395
TestPackage54:
393396
needs: [Combine54]
394397
uses: ./.github/workflows/testPackageOnWindows.yml

0 commit comments

Comments
 (0)