8000 gh-71052: Add Android build script by mhsmith · Pull Request #115576 · python/cpython · GitHub
[go: up one dir, main page]
Skip to content

gh-71052: Add Android build script #115576

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.

Sign up for GitHub

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

Closed
wants to merge 8 commits into from
Closed

gh-71052: Add Android build script #115576

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
07fab81
Fix Android linker issues
mhsmith Jan 31, 2024
0da8532
Merge branch 'main' into android-link
mhsmith Feb 6, 2024
5f7c39e
More Android linker fixes
mhsmith Feb 8, 2024
d0ad0df
Add Android build script and README
mhsmith Feb 16, 2024
90eec1e
Fix build on Linux
mhsmith Feb 16, 2024
ec59dd1
Add news entry
mhsmith Feb 16, 2024
87f85b9
Merge branch 'main' into android-link
mhsmith Feb 16, 2024
a646e2b
Fix typos
mhsmith Feb 19, 2024
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
Prev Previous commit
8000 Next Next commit
Fix build on Linux
  • Loading branch information
@mhsmith
mhsmith committed Feb 16, 2024
commit 90eec1e8af398f5345f7ecb87980f4b4b6185d17
10 changes: 5 additions & 5 deletions Android/android-env.sh
View file
Open in desktop
CFF3
Original file line number Diff line number Diff line change
Expand Up @@ -61,19 +61,19 @@ done
export CFLAGS=""
export LDFLAGS="-Wl,--build-id=sha1 -Wl,--no-rosegment"

# Many packages get away with omitting this on standard Linux, but Android is stricter.
LDFLAGS+=" -lm"
# Many packages get away with omitting -lm on Linux, but Android is stricter.
LDFLAGS="$LDFLAGS -lm"

# -mstackrealign is included where necessary in the clang launcher scripts which are
# pointed to by $CC, so we don't need to include it here.
if [ $HOST = "arm-linux-androideabi" ]; then
CFLAGS+=" -march=armv7-a -mthumb"
CFLAGS="$CFLAGS -march=armv7-a -mthumb"
fi

if [ -n "${PREFIX:-}" ]; then
abs_prefix=$(realpath $PREFIX)
CFLAGS+=" -I$abs_prefix/include"
LDFLAGS+=" -L$abs_prefix/lib"
CFLAGS="$CFLAGS -I$abs_prefix/include"
LDFLAGS="$LDFLAGS -L$abs_prefix/lib"

export PKG_CONFIG="pkg-config --define-prefix"
export PKG_CONFIG_LIBDIR="$abs_prefix/lib/pkgconfig"
Expand Down
3 changes: 2 additions & 1 deletion Android/android.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
from pathlib import Path

SCRIPT_NAME = Path(__file__).name
CHECKOUT = Path(__file__).parent.parent
CHECKOUT = Path(__file__).resolve().parent.parent
CROSS_BUILD_DIR = CHECKOUT / "cross-build"


Expand Down Expand Up @@ -40,6 +40,7 @@ def run(command, *, host=None, **kwargs):
if host:
env_script = CHECKOUT / "Android/android-env.sh"
env_output = subprocess.run(
f"set -eu; "
f"HOST={host}; "
f"PREFIX={subdir(host)}/prefix; "
f". {env_script}; "
Expand Down
0