Skip to content

Update Rust base image and optimize Dockerfile - #2

Merged
itouakirai merged 1 commit into
itouakirai:litefrom
daylighttg:lite
Nov 6, 2026
Merged

itouakirai merged 1 commit into
itouakirai:litefrom
daylighttg:lite

Conversation

@daylighttg

Copy link
Copy Markdown

This pull request makes updates to the Docker build process, including a base image update and a change to how the Android NDK is handled.

Build environment updates:

  • Updated the Rust base image from rust:1.85-bookworm to rust:1.90-bookworm in the Dockerfile to use a newer Rust toolchain.

Android NDK installation:

  • Modified the NDK installation step in the Dockerfile to remove the explicit renaming of the extracted NDK directory; now, the directory is left with its default name.Updated the base Rust image version and cleaned up the Dockerfile commands.

Updated the base Rust image version and cleaned up the Dockerfile commands.
Copilot AI balanced review requested due to automatic review settings October 24, 2026 13:29

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

@itouakirai
itouakirai merged commit 20b3f74 into itouakirai:lite Nov 6, 2026
itouakirai pushed a commit that referenced this pull request Nov 6, 2026
GitHub's macOS arm64 runners were too scarce to build the macOS package
(jobs were dropped after 15 minutes in the queue), so the release no longer
waits for one: macOS users run the Docker image.

Docker:
- CI job `docker` builds ghcr.io/<owner>/wrapper from the linux-x86_64
  package (docker/prebuilt.Dockerfile, nothing is compiled again),
  smoke-tests it (agent up, lite starts in the container's sandbox) and
  pushes `nightly` from the default branch and `<version>`, `<major>.<minor>`,
  `latest` from release tags.
- compose.yaml uses that image; every setting comes from .env (token,
  public mode, max accounts, proxy, log level, data dir, bind address and
  port), restart: unless-stopped, platform linux/amd64 for ARM hosts.
- Dockerfile (from source), on top of #2: pass the NDK path to CMake instead
  of relying on the directory name, unpack only the NDK toolchain, curl
  instead of aria2, rust:1-trixie / node:24 / debian:13-slim, a data VOLUME,
  address from WRAPPER_HOST/WRAPPER_PORT.
- wrapper: empty WRAPPER_* variables count as unset (Compose passes
  `${VAR:-}` as an empty string; WRAPPER_MAX_ACCOUNTS= was rejected).
- README: a full Docker guide (start, settings, docker run, update, data and
  backups, public mode, reverse proxy, macOS, ARM, building, troubleshooting).

macOS: removed from the CI matrix, `wrapper update` and the panel's package
download; the update error points to Docker.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants