Skip to content

Commit

Permalink
Merge pull request #1896 from billhollings/ci-reduce-artifact-sizes
Browse files Browse the repository at this point in the history
GitHub CI streamline uploaded artifact sizes.
  • Loading branch information
billhollings authored May 10, 2023
2 parents 5bce402 + 3247bd4 commit dd5faf2
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 5 deletions.
11 changes: 7 additions & 4 deletions .github/workflows/CI.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ jobs:
strategy:
matrix:
xcode: [ "14.3" ]
platform: [ "all" ]
platform: [ "all", "macos", "ios" ]
os: [ "macos-13" ]
upload_artifacts: [ true ]
# additional specific configurations
Expand Down Expand Up @@ -94,14 +94,17 @@ jobs:
- name: Tar Artifacts
if: success() && matrix.upload_artifacts == true
# See: https://github.com/actions/upload-artifact#maintaining-file-permissions-and-case-sensitive-files
run: tar -C Package -s/Release/MoltenVK/ -cvf "MoltenVK.tar" Release/
# To reduce artifact size, don't include any stand-alone shader converter binaries.
run: |
rm -rf Package/Release/MoltenVKShaderConverter
tar -C Package -s/Release/MoltenVK/ -cvf "MoltenVK-${{ matrix.platform }}.tar" Release/
- name: Upload Artifacts
if: success() && matrix.upload_artifacts == true
uses: actions/upload-artifact@v3
with:
name: "MoltenVK"
path: "MoltenVK.tar"
name: "MoltenVK-${{ matrix.platform }}"
path: "MoltenVK-${{ matrix.platform }}.tar"

release:
name: 'Release'
Expand Down
2 changes: 1 addition & 1 deletion Docs/Whats_New.md
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ Released TBD
disable recent fixes to handling LOD for arrayed depth images in shaders,
on Apple Silicon, when those fixes cause regression in rendering behavior.
- For correctness, set `VkPhysicalDeviceLimits::lineWidthGranularity` to `1`.
- Modify GitHub CI to build and create a single universal binary artifact.
- Improve GitHub CI production of binary artifacts on submission and release.



Expand Down

0 comments on commit dd5faf2

Please sign in to comment.