build: fixed building configuration for Android

[MeowShe]

Fixed architecture configuration and modified minimum Android SDK version.

In newer versions of the Android NDK, if the Android API version is lower than 24 (Android 7.0), compilation will not work because of API changes (in fact, Android API versions of 24 and below are no longer common), and the Android device architecture now only exists for ‘armv7a’, not ‘armv7’, which can also lead to compilation errors.

Reference:
https://developer.android.com/studio/releases/platforms
https://developer.android.com/ndk/guides/abis

[MeowShe]

This is a necessary change because in newer versions of the Android NDK, older configurations can cause problems during building

[F3n67u]

This is a necessary change because in newer versions of the Android NDK, older configurations can cause problems during building

Thanks for your contribution! Could you add more details about what the problem is?

[MeowShe]

This is a necessary change because in newer versions of the Android NDK, older configurations can cause problems during building

Thanks for your contribution! Could you add more details about what the problem is?

Thanks for your reply. I have updated the details of the problem.

[F3n67u]

CC @nodejs/collaborators anyone with Android knowledge could help to review this pr? Please help. Thanks.

[devnexen]

The toolchain fix looks correct to me.

[F3n67u]

@MeowShe Could you remove the merge commit? Our pr land tools don't allow merge commit and you don't need to merge/rebase nodejs/main to keep your pr updated.

[F3n67u]

The toolchain fix looks correct to me.

@devnexen Could you approve this pr please if it looks good to you?

[MeowShe]

@MeowShe Could you remove the merge commit? Our pr land tools don't allow merge commit and you don't need to merge/rebase nodejs/main to keep your pr updated.

Thanks, that was an accident and I have removed it now.

[nodejs-github-bot]

Commit Queue failed

- Loading data for nodejs/node/pull/44016
✔  Done loading data for nodejs/node/pull/44016
----------------------------------- PR info ------------------------------------
Title      build: fixed building configuration for Android (#44016)
Author     BuShe  (@MeowShe, first-time contributor)
Branch     MeowShe:main -> nodejs:main
Labels     build, author ready, needs-ci, android, review wanted
Commits    2
 - build: optimized and fixed building configuration to Android
 - build: remove unused variables from the configuration file
Committers 1
 - GitHub 
PR-URL: https://github.com/nodejs/node/pull/44016
Reviewed-By: Yash Ladha 
Reviewed-By: David Carlier 
Reviewed-By: Christian Clauss 
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/44016
Reviewed-By: Yash Ladha 
Reviewed-By: David Carlier 
Reviewed-By: Christian Clauss 
--------------------------------------------------------------------------------
   ℹ  This PR was created on Thu, 28 Jul 2022 05:46:26 GMT
   ✔  Approvals: 3
   ✔  - Yash Ladha (@yashLadha): https://github.com/nodejs/node/pull/44016#pullrequestreview-1064280103
   ✔  - David Carlier (@devnexen): https://github.com/nodejs/node/pull/44016#pullrequestreview-1064301207
   ✔  - Christian Clauss (@cclauss): https://github.com/nodejs/node/pull/44016#pullrequestreview-1064309763
   ✔  Last GitHub CI successful
   ℹ  Last Full PR CI on 2022-08-06T15:32:24Z: https://ci.nodejs.org/job/node-test-pull-request/45883/
- Querying data for job/node-test-pull-request/45883/
   ✔  Last Jenkins CI successful
   ⚠  PR author is a new contributor: @MeowShe(bushe.cn@icloud.com)
   ⚠  - commit e9b71a8d955a is authored by 106762153+MeowShe@users.noreply.github.com
--------------------------------------------------------------------------------
   ✔  Aborted `git node land` session in /home/runner/work/node/node/.ncu

https://github.com/nodejs/node/actions/runs/2810656268

[MeowShe]

@MeowShe Commit queue tools complain the commit e9b71a8 is authored by 106762153+MeowShe@users.noreply.github.com but not your github email: bushe.cn@icloud.com. Could you please set your git email to bushe.cn@icloud.com? see Setting your commit email address for how to set your commit email address.

It looks correct now.

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/45895/

[nodejs-github-bot]

Landed in 88d55d2

[F3n67u]

@MeowShe Congrats on your first contribution to node! 🔥