Skip to content

Commit

Permalink
Merge pull request #2406 from LORgames/ssurtees/omnios
Browse files Browse the repository at this point in the history
Add OmniOS CI job
  • Loading branch information
nickclark2016 authored Jan 1, 2025
2 parents eb418ab + d529b96 commit 1420eae
Show file tree
Hide file tree
Showing 2 changed files with 39 additions and 0 deletions.
38 changes: 38 additions & 0 deletions .github/workflows/ci-workflow.yml
Original file line number Diff line number Diff line change
Expand Up @@ -340,6 +340,44 @@ jobs:
with:
name: premake-solaris-${{ matrix.platform }}
path: bin/${{ matrix.config }}/
omnios:
runs-on: ubuntu-latest
strategy:
matrix:
config: [debug, release]
platform: [x64]
cc: [gcc]
timeout-minutes: 30
defaults:
run:
shell: omnios {0}
steps:
- name: Checkout
uses: actions/checkout@v4
- name: Start OmniOS VM
uses: vmactions/omnios-vm@v1
with:
usesh: true
prepare: |
pkg install build-essential web/ca-bundle
- name: Build
run: |
cd $GITHUB_WORKSPACE
CC=${{ matrix.cc }} PLATFORM=${{ matrix.platform }} CONFIG=${{ matrix.config }} PREMAKE_OPTS="--cc=${{ matrix.cc }}" ./Bootstrap.sh
- name: Test
run: |
cd $GITHUB_WORKSPACE
bin/${{ matrix.config }}/premake5 test --test-all
- name: Docs check
run: |
cd $GITHUB_WORKSPACE
bin/${{ matrix.config }}/premake5 docs-check
- name: Upload Artifacts
if: matrix.config == 'release' && matrix.cc == 'gcc'
uses: actions/upload-artifact@v4
with:
name: premake-omnios-${{ matrix.platform }}
path: bin/${{ matrix.config }}/

# This job will be required for PRs to be merged.
# This should depend on (via needs) all jobs that need to be successful for the PR to be merged.
Expand Down
1 change: 1 addition & 0 deletions contrib/curl/premake5.lua
Original file line number Diff line number Diff line change
Expand Up @@ -45,6 +45,7 @@ project "curl-lib"
"/usr/local/share/certs/ca-root-nss.crt",
"/etc/certs/ca-certificates.crt",
"/etc/ssl/cert.pem",
"/etc/ssl/cacert.pem",
"/boot/system/data/ssl/CARootCertificates.pem" } do
if os.isfile(f) then
ca = f
Expand Down

0 comments on commit 1420eae

Please sign in to comment.