Skip to content

Commit

Permalink
upgrade uplad-artifact
Browse files Browse the repository at this point in the history
  • Loading branch information
cyjseagull committed Sep 14, 2024
1 parent 3b3a6c3 commit 5692185
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 10 deletions.
15 changes: 8 additions & 7 deletions .github/workflows/cpp_workflow.yml
Original file line number Diff line number Diff line change
Expand Up @@ -129,6 +129,7 @@ jobs:
- name: Build for macos
if: runner.os == 'macOS'
run: |
brew install lcov
bash cpp/tools/install_depends.sh -o macos
mkdir -p cpp/build && cd cpp/build && cmake -DTESTS=ON -DCOVERAGE=ON -DBUILD_SDK=ON -DCMAKE_TOOLCHAIN_FILE=${{ env.VCPKG_ROOT }}/scripts/buildsystems/vcpkg.cmake ../
make -j3
Expand All @@ -137,12 +138,12 @@ jobs:
run: |
cd cpp/build && CTEST_OUTPUT_ON_FAILURE=TRUE make test
make coverage
- uses: actions/upload-artifact@v2
- uses: actions/upload-artifact@v4
if: runner.os == 'macos'
with:
name: libppc-crypto-sdk-jni.dylib
path: ./cpp/wedpr-component-sdk/bindings/java/src/main/resources/META-INF/native/libppc-crypto-sdk-jni.dylib
- uses: actions/upload-artifact@v2
- uses: actions/upload-artifact@v4
if: runner.os == 'Windows'
with:
name: libppc-crypto-sdk-jni.dylib
Expand Down Expand Up @@ -206,19 +207,19 @@ jobs:
run: |
export OMP_NUM_THREADS=1
cd build && CTEST_OUTPUT_ON_FAILURE=TRUE make test
- uses: actions/upload-artifact@v2
- uses: actions/upload-artifact@v4
with:
name: ppc-air-node-centos-x64
path: ./cpp/build/bin/ppc-air-node
- uses: actions/upload-artifact@v2
- uses: actions/upload-artifact@v4
with:
name: ppc-pro-node-centos-x64
path: ./cpp/build/bin/ppc-pro-node
- uses: actions/upload-artifact@v2
- uses: actions/upload-artifact@v4
with:
name: ppc-gateway-service-centos-x64
path: ./cpp/build/bin/ppc-gateway-service
- uses: actions/upload-artifact@v2
- uses: actions/upload-artifact@v4
with:
name: libppc-crypto-sdk-jni.so
path: ./cpp/wedpr-component-sdk/bindings/java/src/main/resources/META-INF/native/libppc-crypto-sdk-jni.so
path: ./cpp/wedpr-component-sdk/bindings/java/src/main/resources/META-INF/native/libppc-crypto-sdk-jni.so
6 changes: 3 additions & 3 deletions cpp/tools/install_depends.sh
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ install_centos_depends()
{
LOG_INFO "install depends for centos ..."
# install the basic package
sudo yum install -y bison flex epel-release centos-release-scl flex bison patch devtoolset-11 rh-perl530-perl cmake3 zlib-devel ccache lcov python-devel python3-devel krb5-devel autoconf
sudo yum install -y lcov bison flex epel-release centos-release-scl flex bison patch devtoolset-11 rh-perl530-perl cmake3 zlib-devel ccache lcov python-devel python3-devel krb5-devel autoconf
LOG_INFO "install basic-package-depends for centos success..."

LOG_INFO "install gsasl ... "
Expand All @@ -85,7 +85,7 @@ install_ubuntu_depends()
{
LOG_INFO "install depends for ubuntu ..."
# install the basic package
sudo apt install -y bison flex g++ libssl-dev openssl cmake git build-essential autoconf texinfo flex patch bison libgmp-dev zlib1g-dev automake pkg-config libtool wget libkrb5-dev autoconf
sudo apt install -y lcov bison flex g++ libssl-dev openssl cmake git build-essential autoconf texinfo flex patch bison libgmp-dev zlib1g-dev automake pkg-config libtool wget libkrb5-dev autoconf
LOG_INFO "install basic-package-depends for ubuntu success..."

LOG_INFO "install gsasl ... "
Expand All @@ -106,7 +106,7 @@ install_ubuntu_depends()
install_macos_depends()
{
LOG_INFO "install depends for macos ..."
brew install clang cmake autoconf gsasl nasm
brew install clang cmake autoconf gsasl nasm lcov
LOG_INFO "install depends for macos success ..."
}

Expand Down
1 change: 1 addition & 0 deletions cpp/wedpr-main/common/NodeStarter.h
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@
#include <execinfo.h>
#include <libhelper/CommandHelper.h>
#include <libhelper/ExitHandler.h>
#include <signal.h>
#include <stdexcept>
#include <thread>

Expand Down

0 comments on commit 5692185

Please sign in to comment.