Skip to content

Commit

Permalink
fix: openssl requirements ubuntu (#5)
Browse files Browse the repository at this point in the history
Default features for reqwest on Ubuntu relies on OpenSSL due to the use
of
[`rust-native-tls`](https://github.com/seanmonstar/reqwest/blob/a205128f038431e37bfff25ade4ff04d20e7dd32/Cargo.toml#L27),
in order to avoid relying on OpenSSL `rustls` is used instead.

As of now is set for all platforms but it could be enabled only for
Ubuntu in the future.

<!--
Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.
-->
  • Loading branch information
EstebanBorai committed Jun 6, 2024
1 parent 70036ed commit 11c274e
Show file tree
Hide file tree
Showing 5 changed files with 143 additions and 327 deletions.
46 changes: 34 additions & 12 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,23 +8,45 @@ on:

jobs:
build:
name: Builds on ${{ matrix.os }}
name: Builds on ${{ matrix.name }}
runs-on: ${{ matrix.os }}
strategy:
matrix:
os: [
ubuntu-latest,
windows-latest,
macOS-latest
]
name:
- linux-x64-musl
- macos-x64
- macos-aarch64

include:
- name: linux-x64-musl
os: ubuntu-latest
target: x86_64-unknown-linux-musl
exe: wait-on

- name: macos-x64
os: macos-latest
target: x86_64-apple-darwin
exe: wait-on

- name: macos-aarch64
os: macos-latest
target: aarch64-apple-darwin
exe: wait-on

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

- uses: actions-rs/toolchain@v1
- name: Setup Rust
uses: dtolnay/rust-toolchain@stable
with:
profile: minimal
toolchain: stable
targets: ${{ matrix.target }}

- name: Install ${{ matrix.name }} System Dependencies
if: matrix.name == 'linux-x64-musl'
run: |
sudo apt-get update
sudo apt-get install -y musl-tools
- name: Build
run: cargo build --release
- name: Build for Release
run: cargo build --release --target ${{ matrix.target }}
8 changes: 7 additions & 1 deletion .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,7 @@ jobs:
artifacts:
needs: release
name: Upload Artifacts ${{ matrix.os }}
name: Upload Artifacts ${{ matrix.name }}
runs-on: ${{ matrix.os }}
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
Expand Down Expand Up @@ -121,6 +121,12 @@ jobs:
- name: Build for Release
run: cargo build --release --target ${{ matrix.target }}

- name: Install ${{ matrix.name }} System Dependencies
if: matrix.name == 'linux-x64-musl'
run: |
sudo apt-get update
sudo apt-get install -y musl-tools
- name: Prepare Binary
shell: bash
run: |
Expand Down
Loading

0 comments on commit 11c274e

Please sign in to comment.