Skip to content

Commit

Permalink
Add a CI to check if the PR on main is OK and fix bug by putting OPEN…
Browse files Browse the repository at this point in the history
…SSL_DIR to env and PKG CONFIG
  • Loading branch information
Sellig6792 committed Aug 27, 2023
1 parent a391ed0 commit d7935fc
Show file tree
Hide file tree
Showing 2 changed files with 139 additions and 1 deletion.
131 changes: 131 additions & 0 deletions .github/workflows/develop_pr.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,131 @@
name: Develop PR

on:
pull_request:
branches: [ "main" ]

env:
CARGO_TERM_COLOR: always

jobs:
style:
name: Check Style
runs-on: ubuntu-latest
steps:
- name: Checkout
uses: actions/checkout@v1
- name: Install rust
uses: actions-rs/toolchain@v1
with:
toolchain: stable
components: rustfmt
profile: minimal
override: true
- name: cargo fmt -- --check
uses: actions-rs/cargo@v1
with:
command: fmt
args: --all -- --check


test:
name: Test
if: always()
runs-on: ubuntu-latest

strategy:
matrix:
build: [ stable, beta, nightly ]
include:
- build: beta
rust: beta
- build: nightly
rust: nightly
benches: true

steps:
- name: Checkout
uses: actions/checkout@v1

- name: Install rust
uses: actions-rs/toolchain@v1
with:
toolchain: ${{ matrix.rust || 'stable' }}
profile: minimal
override: true

- name: Build debug
uses: actions-rs/cargo@v1
with:
command: build
args: ${{ matrix.features }}

- name: Test
uses: actions-rs/cargo@v1
with:
command: test
args: ${{ matrix.features }}

- name: Test all benches
if: matrix.benches
uses: actions-rs/cargo@v1
with:
command: test
args: --benches ${{ matrix.features }}
deploy_building:
name: Deploy Building
# needs: [ test ]
strategy:
matrix:
os: [ ubuntu-latest ] # Default os for build
target: [ aarch64-unknown-linux-gnu,
i686-pc-windows-gnu, x86_64-pc-windows-gnu,
i686-unknown-linux-gnu, x86_64-unknown-linux-gnu,
x86_64-apple-darwin ]
exclude: # Do not build for macOS on Linux
- target: x86_64-apple-darwin
os: ubuntu-latest
include: # List of all targets to build for and the name for common mortals
- target: aarch64-unknown-linux-gnu
name: linux-arm64
- target: i686-pc-windows-gnu
name: windows-i686
- target: x86_64-pc-windows-gnu
name: windows-x86_64
- target: i686-unknown-linux-gnu
name: linux-i686
- target: x86_64-unknown-linux-gnu
name: linux-x86_64
- target: x86_64-apple-darwin
name: macos-x86_64
os: macos-latest


# Runs on latest ubuntu by default except for windows targets
runs-on: ${{ matrix.os }}

steps:
- name: Checkout
uses: actions/checkout@v1

- name: Install rust
uses: actions-rs/toolchain@v1
with:
toolchain: stable
profile: minimal
override: true
target: ${{ matrix.target }}

- name: PKG Config path (Linux)
if: runner.os == 'Linux'
run: |
echo "PKG_CONFIG_PATH=/usr/lib/x86_64-linux-gnu/pkgconfig/" >> $GITHUB_ENV
- name: Build release
uses: actions-rs/cargo@v1
with:
use-cross: true
command: build
args: --release --target ${{ matrix.target }}
env:
OPENSSL_DIR: /usr/lib/ssl
9 changes: 8 additions & 1 deletion .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -110,6 +110,7 @@ jobs:
steps:
- name: Checkout
uses: actions/checkout@v1

- name: Install rust
uses: actions-rs/toolchain@v1
with:
Expand All @@ -118,12 +119,19 @@ jobs:
override: true
target: ${{ matrix.target }}

- name: PKG Config path (Linux)
if: runner.os == 'Linux'
run: |
echo "PKG_CONFIG_PATH=/usr/lib/x86_64-linux-gnu/pkgconfig/" >> $GITHUB_ENV
- name: Build release
uses: actions-rs/cargo@v1
with:
use-cross: true
command: build
args: --release --target ${{ matrix.target }}
env:
OPENSSL_DIR: /usr/lib/ssl

- name: Rename binary (Linux & macOS)
if: matrix.target != 'x86_64-pc-windows-gnu' && matrix.target != 'i686-pc-windows-gnu'
Expand All @@ -133,7 +141,6 @@ jobs:
if: matrix.target == 'x86_64-pc-windows-gnu' || matrix.target == 'i686-pc-windows-gnu'
run: mv target/${{ matrix.target }}/release/fbf.exe target/${{ matrix.target }}/release/fbf-${{ matrix.name }}.exe


- name: Upload release
uses: softprops/action-gh-release@v1
with:
Expand Down

0 comments on commit d7935fc

Please sign in to comment.