Skip to content

Commit

Permalink
Fixed Linux/MacOS ARM build artifact not being uploaded to our host a…
Browse files Browse the repository at this point in the history
…nd Coverity build
  • Loading branch information
cbnolok committed Aug 7, 2024
1 parent e0e388d commit 7a9a90c
Show file tree
Hide file tree
Showing 9 changed files with 72 additions and 18 deletions.
8 changes: 6 additions & 2 deletions .github/workflows/build_aux_files.yml
Original file line number Diff line number Diff line change
@@ -1,12 +1,16 @@
name: Create/upload auxiliary files to the release.

on:
- push
push:
branches:
- 'master'
- 'main'
- 'dev'

jobs:
upload_github_release:
# Upload artifact linked to GitHub RELEASE we are creating - only if the run is for a pull request, or for selected branches
if: contains(fromJson('["master", "main"]'), github.ref_name)
#if: contains(fromJson('["master", "main"]'), github.ref_name)

runs-on: ubuntu-latest
steps:
Expand Down
14 changes: 11 additions & 3 deletions .github/workflows/build_linux_x86.yml
Original file line number Diff line number Diff line change
@@ -1,8 +1,16 @@
name: Linux x86

on:
- push
- pull_request
push:
branches:
- 'master'
- 'main'
- 'dev'
pull_request:
branches:
- 'master'
- 'main'
- 'dev'

jobs:
linux-x86:
Expand Down Expand Up @@ -135,4 +143,4 @@ jobs:
run-id: ${{ github.run_id }}
- name: Push release
run: |
curl -sST "{SphereSvrX-linux-x86-nightly.zip}" -u ${{secrets.UP_USER}}:${{secrets.UP_PASS}} ${{secrets.UP_WHERE}}
curl -sST "{SphereSvrX-linux-x86-nightly.tar.gz}" -u ${{secrets.UP_USER}}:${{secrets.UP_PASS}} ${{secrets.UP_WHERE}}
14 changes: 11 additions & 3 deletions .github/workflows/build_linux_x86_64.yml
Original file line number Diff line number Diff line change
@@ -1,8 +1,16 @@
name: Linux x86_64

on:
- push
- pull_request
push:
branches:
- 'master'
- 'main'
- 'dev'
pull_request:
branches:
- 'master'
- 'main'
- 'dev'

jobs:
linux-x86_64:
Expand Down Expand Up @@ -87,4 +95,4 @@ jobs:
run-id: ${{ github.run_id }}
- name: Push release
run: |
curl -sST "{SphereSvrX-linux-x86_64-nightly.zip}" -u ${{secrets.UP_USER}}:${{secrets.UP_PASS}} ${{secrets.UP_WHERE}}
curl -sST "{SphereSvrX-linux-x86_64-nightly.tar.gz}" -u ${{secrets.UP_USER}}:${{secrets.UP_PASS}} ${{secrets.UP_WHERE}}
14 changes: 11 additions & 3 deletions .github/workflows/build_osx_arm.yml
Original file line number Diff line number Diff line change
@@ -1,8 +1,16 @@
name: MacOS ARM

on:
- push
- pull_request
push:
branches:
- 'master'
- 'main'
- 'dev'
pull_request:
branches:
- 'master'
- 'main'
- 'dev'

jobs:
macos-arm64:
Expand Down Expand Up @@ -89,7 +97,7 @@ jobs:
- name: Download builds
uses: actions/download-artifact@v4
with:
name: Build-arm64
name: Build-osx-arm64
merge-multiple: true
run-id: ${{ github.run_id }}
- name: Push release
Expand Down
12 changes: 10 additions & 2 deletions .github/workflows/build_osx_x86_64.yml
Original file line number Diff line number Diff line change
@@ -1,8 +1,16 @@
name: MacOS x86_64

on:
- push
- pull_request
push:
branches:
- 'master'
- 'main'
- 'dev'
pull_request:
branches:
- 'master'
- 'main'
- 'dev'

jobs:
macos-x86_64:
Expand Down
12 changes: 10 additions & 2 deletions .github/workflows/build_win_x86.yml
Original file line number Diff line number Diff line change
@@ -1,8 +1,16 @@
name: Windows x86

on:
- push
- pull_request
push:
branches:
- 'master'
- 'main'
- 'dev'
pull_request:
branches:
- 'master'
- 'main'
- 'dev'

jobs:
windows-x86:
Expand Down
12 changes: 10 additions & 2 deletions .github/workflows/build_win_x86_64.yml
Original file line number Diff line number Diff line change
@@ -1,8 +1,16 @@
name: Windows x86_64

on:
- push
- pull_request
push:
branches:
- 'master'
- 'main'
- 'dev'
pull_request:
branches:
- 'master'
- 'main'
- 'dev'

jobs:
windows-x86_64:
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/coverity-scan.yml
Original file line number Diff line number Diff line change
Expand Up @@ -70,6 +70,7 @@ jobs:
- name: Submit the result to Coverity Scan
run: |
pwd
cd build
tar czvf build-data.tgz cov-int
curl \
--form project=SphereServer/Source-X \
Expand Down
3 changes: 2 additions & 1 deletion src/common/sphere_library/CSObjCont.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,8 @@ CSObjCont::CSObjCont() :

CSObjCont::~CSObjCont()
{
ClearContainer(true);
// Do not virtually call this method, since i'm in the destructor of the base-most class.
CSObjCont::ClearContainer(true);
}

// CSObjCont:: Modifiers.
Expand Down

0 comments on commit 7a9a90c

Please sign in to comment.