Skip to content

ci(android): make build_android.sh executable + invoke via bash - #118

Merged
gburd merged 1 commit into
masterfrom
fix/android-script-exec
Jul 31, 2026
Merged

ci(android): make build_android.sh executable + invoke via bash#118
gburd merged 1 commit into
masterfrom
fix/android-script-exec

Conversation

@gburd

@gburd gburd commented Jul 31, 2026

Copy link
Copy Markdown
Collaborator

The Android cross-build job (added in #115) failed with exit 126 (permission denied): the committed dist/android/build_android.sh had mode 100644 and the workflow invoked it directly. Set the executable bit (100755, matching the other dist/ scripts) and invoke via bash as belt-and-suspenders. The job is advisory (continue-on-error).

The Android cross-build job failed with exit 126 (permission denied): the
committed dist/android/build_android.sh had mode 100644 and the workflow ran it
directly. Set the executable bit (100755, matching the other dist/ scripts) and
invoke it via 'bash' as belt-and-suspenders. Advisory (continue-on-error) job.
@gburd
gburd merged commit 401e392 into master Jul 31, 2026
48 of 50 checks passed
@gburd
gburd deleted the fix/android-script-exec branch July 31, 2026 13:28
@github-actions

Copy link
Copy Markdown

Coccinelle convention checks

No new violations. ✅

Resolved since baseline (2) -- update dist/cocci/baseline.txt to lock these in.
rule_mutex_unbalanced|MUTEX_UNBALANCED|src/crypto/mersenne/mt19937db.c|return (ret);
rule_mutex_unbalanced|MUTEX_UNBALANCED|src/mp/mp_register.c|return (ret);

@github-actions

Copy link
Copy Markdown

ABI diff vs v5.3.33 (libabigail — authoritative)

Removed exported symbols (nm -D, _NNNN version suffix normalized)

None.


Advisory: libabigail/nm is the authoritative binary-ABI check; Coccinelle is complementary source-level early warning. See dist/cocci/README.md.

gburd added a commit that referenced this pull request Jul 31, 2026
ci(android): make build_android.sh executable + invoke via bash
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.

1 participant