Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bump android container to 7b4ffb683 #6826

Merged
merged 3 commits into from
Sep 20, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 0 additions & 2 deletions .github/workflows/android-app.yml
Original file line number Diff line number Diff line change
Expand Up @@ -206,8 +206,6 @@ jobs:
RUSTFLAGS: --deny warnings
BUILD_TYPE: debug
run: |
# Temporary fix to address maybenot.h build issues.
cargo install --force cbindgen --version "0.26.0"
ARCHITECTURES="${{ matrix.abi }}"
UNSTRIPPED_LIB_PATH="$CARGO_TARGET_DIR/${{ matrix.target }}/$BUILD_TYPE/libmullvad_jni.so"
STRIPPED_LIB_PATH="./android/app/build/extraJni/${{ matrix.abi }}/libmullvad_jni.so"
Expand Down
6 changes: 0 additions & 6 deletions .github/workflows/rust-unused-dependencies.yml
Original file line number Diff line number Diff line change
Expand Up @@ -81,12 +81,6 @@ jobs:
git config --global --add safe.directory '*'
git submodule update --init --recursive --depth=1 wireguard-go-rs

- name: Install cbindgen
run: |
# Temporary fix to address maybenot.h build issues.
# Remove this step when cbindgen has been added to the build container.
cargo install --force cbindgen --version "0.26.0"

- name: Install nightly Rust toolchain
run: |
rustup default $RUST_NIGHTLY_TOOLCHAIN
Expand Down
2 changes: 2 additions & 0 deletions android/BuildInstructions.md
Original file line number Diff line number Diff line change
Expand Up @@ -128,6 +128,8 @@ Linux distro:
#### 5. Install and configure Rust toolchain

- Get the latest **stable** Rust toolchain via [rustup.rs](https://rustup.rs/).
Also install `cbindgen` which is required to build `wireguard-go-rs`:
`cargo install --force cbindgen`

- Configure Android cross-compilation targets and set up linker and archiver. This can be done by setting the following
environment variables:
Expand Down
5 changes: 5 additions & 0 deletions android/docs/BuildInstructions.macos.md
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,11 @@ Finish the install of `rustup`:
rustup-init
```

Install `cbindgen` which is required to build `wireguard-go-rs`:
```bash
cargo install --force cbindgen
```

## 2. Install SDK Tools and Android NDK Toolchain
Open Android Studio -> Tools -> SDK Manager, and install `Android SDK Command-line Tools (latest)`.

Expand Down
8 changes: 0 additions & 8 deletions build-apk.sh
Original file line number Diff line number Diff line change
Expand Up @@ -77,14 +77,6 @@ mkdir -p "app/build/extraAssets"
mkdir -p "app/build/extraJni"
popd

# Temporary fix to address maybenot.h (checked in) sometimes needing to be
# re-generated due to how `make` looks at last-modifications while git neither
# stores nor consistently sets modification metadata on file checkout.
#
# NOTE: The version should match the one used in the checked in
# maybenot.h file.
cargo install --force cbindgen --version "0.26.0"

for ARCHITECTURE in ${ARCHITECTURES:-aarch64 armv7 x86_64 i686}; do
case "$ARCHITECTURE" in
"x86_64")
Expand Down
2 changes: 1 addition & 1 deletion building/android-container-image.txt
Original file line number Diff line number Diff line change
@@ -1 +1 @@
ghcr.io/mullvad/mullvadvpn-app-build-android:63e6d7303
ghcr.io/mullvad/mullvadvpn-app-build-android:7b4ffb683
Loading