Repository navigation
heronin: build the Android app - #38
Merged
Merged
Conversation
Signed-off-by: fizzy pop-os-codex-2 <slot+pop-os-codex-2@fizzy.invalid>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
heronin: build the Android app
Part of fizzyedit/work#1191
In plain words
Heronin can now be packaged as an Android app through fizzy's Android support.
The person building it can supply the server address without publishing that address.
What changes
What you will notice
The Android package is labelled Heronin. Phone startup remains unverified.
How we know it works
Formatting and all six requested build gates passed, including 129 tests.
The native build with
zig build -Dserver-url=https://example.invalidalso passed.Android acceptance evidence follows in the technical summary.
How this serves fizzy
This extends fizzy's platform support
by packaging Heronin for Android through the shared build.
For agents: technical summary
Goal
Build Heronin's APK and provide the native server address at build time.
What changed and why
Pin fizzy to d9d816db9758beed7bfc01dbaa29fbc894db7370, the main commit adding
android to the package paths. Forward SDK and keystore options, set the package
identity and label, add APK steps, and stop before the host-only build graph.
Add heronin_build options and select environment, build option, then local fallback.
Source counts from
git diff --numstat HEAD: ADDED non-test lines 31;added+deleted 35. No new source files. Delivery artifacts are excluded per R5.
Both counts meet the revised limits of 45 added and 50 total.
How it was verified
zig fmt --check build.zig tools tests/check_public model app server: passed.zig build: 67/67 steps succeeded.zig build server: 3/3 steps succeeded.zig build test: 67/67 steps succeeded; 129/129 tests passed.zig build web: 55/55 steps succeeded.zig build check-public: 3/3 steps succeeded.zig build -Dserver-url=https://example.invalid: 67/67 steps succeeded.The exact acceptance build initially failed because the default debug keystore
was absent. With a temporary debug keystore supplied through the requested option:
GitHub fetching was blocked by network access. The pin was obtained from the
locally available fizzy main history using git archive and zig fetch; no upstream
source was changed. The package cache archive needed its duplicate root flattened
before Zig could read the package with its declared hash.
Not done / follow-ups
UNVERIFIED: APK startup on the lab phone; Ops A7 smoke follows K10b.
Phone sign-in, native import, Drive, app icon, release key, Play listing,
install script, CI and server changes remain outside this job.
Part of fizzyedit/work#1191