Remove duplicate log.cpp compilation in Android CMakeLists

[kirklandsign]

Summary

jni/log.cpp is already included in the base executorch_jni target (line 76). The training and llama conditional blocks were adding it again via target_sources, causing it to be compiled up to 3 times into the same shared library.

Test plan

CI

[pytorch-bot]

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/17634

• 📄 Preview Python docs built from this PR

Note: Links to docs will display an error until the docs builds have been completed.

❌ 3 New Failures, 41 Pending

As of commit dc2c1e1 with merge base cf08087 ():

NEW FAILURES - The following jobs have failed:

• pull / test-mediatek-models-linux / linux-job (gh)
  RuntimeError: Command docker exec -t ebf90b8f4b2940c630ac3b9d5a2023f287f9274e04b9ec2eb1e8f4027554c273 /exec failed with exit code 2
• pull / test-openvino-linux / linux-job (gh)
  RuntimeError: Command docker exec -t 8c8f7525917eadfdc2d2cdaf8a392200e67f3f37440465b35183d87ce8a1b327 /exec failed with exit code 1
• pull / unittest-buck / linux / linux-job (gh)
  RuntimeError: Command docker exec -t 18cafab468c6e762d74fc7708f5268ea4ad5297c71285ce38b9dd88fdf285990 /exec failed with exit code 3

This comment was automatically generated by Dr. CI and updates every 15 minutes.

[github-actions]

This PR needs a release notes: label

If your change should be included in the release notes (i.e. would users of this library care about this change?), please use a label starting with release notes:. This helps us keep track and include your important work in the next release notes.

To add a label, you can comment to pytorchbot, for example
@pytorchbot label "release notes: none"

For more information, see
https://github.com/pytorch/pytorch/wiki/PyTorch-AutoLabel-Bot#why-categorize-for-release-notes-and-how-does-it-work.

[Copilot]

Pull request overview

This PR removes duplicate compilations of jni/log.cpp in the Android CMakeLists.txt file. The log.cpp file is already included in the base executorch_jni target (line 76), but was being added again in the training and llama conditional blocks, potentially causing it to be compiled up to 3 times into the same shared library.

Changes:

• Removed jni/log.cpp from the EXECUTORCH_BUILD_EXTENSION_TRAINING conditional block
• Removed jni/log.cpp from the EXECUTORCH_BUILD_LLAMA_JNI conditional block

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.