Comment out android_ndk_repository line in WORKSPACE file

and point to more representative GitHub issue.

Signed-off-by: JP Simard <jp@jpsim.com>

.github/workflows/android_build.yml

@@ -33,6 +33,8 @@ jobs:
       - name: 'Install dependencies'
         if: steps.check-cache.outputs.cache-hit != 'true'
         run: ./ci/mac_ci_setup.sh
+      - name: 'Work around android_sdk_repository bug' # https://github.com/bazelbuild/bazel/issues/14260
+        run: sed -i 's/# android/android/' WORKSPACE
       - name: 'Build envoy.aar distributable'
         if: steps.check-cache.outputs.cache-hit != 'true'
         env:
@@ -72,6 +74,8 @@ jobs:
       # Return to using:
       #   ./bazelw mobile-install --fat_apk_cpu=x86 --start_app //examples/java/hello_world:hello_envoy
       # When https://github.com/envoyproxy/envoy-mobile/issues/853 is fixed.
+      - name: 'Work around android_sdk_repository bug' # https://github.com/bazelbuild/bazel/issues/14260
+        run: sed -i 's/# android/android/' WORKSPACE
       - name: 'Start java app'
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
@@ -115,6 +119,8 @@ jobs:
       # Return to using:
       #   ./bazelw mobile-install --fat_apk_cpu=x86 --start_app //examples/kotlin/hello_world:hello_envoy_kt
       # When https://github.com/envoyproxy/envoy-mobile/issues/853 is fixed.
+      - name: 'Work around android_sdk_repository bug' # https://github.com/bazelbuild/bazel/issues/14260
+        run: sed -i 's/# android/android/' WORKSPACE
       - name: 'Start kotlin app'
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

.github/workflows/android_tests.yml

@@ -118,8 +118,6 @@ jobs:
           java-version: '8'
           java-package: jdk
           architecture: x64
-      - name: 'Work around android_sdk_repository bug' # https://github.com/bazelbuild/bazel/issues/2722
-        run: sed -i '/^android_ndk_repository/ d' WORKSPACE
       - name: 'Run Kotlin library integration tests'
         if: steps.check_context.outputs.run_tests == 'true'
         env:

.github/workflows/asan.yml

@@ -36,8 +36,6 @@ jobs:
           java-version: '8'
           java-package: jdk
           architecture: x64
-      - name: 'Work around android_sdk_repository bug' # https://github.com/bazelbuild/bazel/issues/2722
-        run: sed -i '/^android_ndk_repository/ d' WORKSPACE
       - name: 'Run tests'
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

.github/workflows/cc_tests.yml

@@ -17,8 +17,6 @@ jobs:
       - uses: actions/checkout@v1
         with:
           submodules: true
-      - name: 'Work around android_sdk_repository bug' # https://github.com/bazelbuild/bazel/issues/2722
-        run: sed -i '/^android_ndk_repository/ d' WORKSPACE
       - env:
             GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
         name: 'Run tests'

.github/workflows/coverage.yml

@@ -30,8 +30,6 @@ jobs:
             echo "Skipping coverage."
             echo "::set-output name=run_coverage::false"
           fi
-      - name: 'Work around android_sdk_repository bug' # https://github.com/bazelbuild/bazel/issues/2722
-        run: sed -i '/^android_ndk_repository/ d' WORKSPACE
       - name: 'Run coverage'
         if: steps.check_context.outputs.run_coverage == 'true'
         continue-on-error: true

.github/workflows/perf.yml

@@ -20,8 +20,6 @@ jobs:
       - uses: actions/checkout@v1
         with:
           submodules: true
-      - name: 'Work around android_sdk_repository bug' # https://github.com/bazelbuild/bazel/issues/2722
-        run: sed -i '/^android_ndk_repository/ d' WORKSPACE
       - name: 'Build test binary'
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
@@ -48,8 +46,6 @@ jobs:
       - uses: actions/checkout@v1
         with:
           submodules: true
-      - name: 'Work around android_sdk_repository bug' # https://github.com/bazelbuild/bazel/issues/2722
-        run: sed -i '/^android_ndk_repository/ d' WORKSPACE
       - name: 'Build test binary'
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

.github/workflows/python_tests.yml

@@ -27,8 +27,6 @@ jobs:
             echo "Skipping tests."
             echo "::set-output name=run_tests::false"
           fi
-      - name: 'Work around android_sdk_repository bug' # https://github.com/bazelbuild/bazel/issues/2722
-        run: sed -i '/^android_ndk_repository/ d' WORKSPACE
       - name: 'Run tests'
         if: steps.check_context.outputs.run_tests == 'true'
         env:

.github/workflows/tsan.yml

@@ -36,8 +36,6 @@ jobs:
           java-version: '8'
           java-package: jdk
           architecture: x64
-      - name: 'Work around android_sdk_repository bug' # https://github.com/bazelbuild/bazel/issues/2722
-        run: sed -i '/^android_ndk_repository/ d' WORKSPACE
       - name: 'Run tests'
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

WORKSPACE

@@ -41,4 +41,7 @@ load("//bazel:python.bzl", "declare_python_abi")
 declare_python_abi(name = "python_abi", python_version = "3")
 
 android_sdk_repository(name = "androidsdk", api_level = 30, build_tools_version = "30.0.2")
-android_ndk_repository(name = "androidndk", api_level = 21)
+
+# TODO(jpsim): Uncomment this line once this issue is resolved: https://github.com/bazelbuild/bazel/issues/14260
+# To build any Android targets, uncomment the following line:
+# android_ndk_repository(name = "androidndk", api_level = 21)

docs/root/start/building/building.rst

@@ -52,6 +52,11 @@ For local builds, set ``ANDROID_HOME`` and ``ANDROID_NDK_HOME`` to point to the
 
 See `ci/mac_ci_setup.sh` for the specific NDK version used during builds.
 
+You'll also need to uncomment the ``android_sdk_repository`` line in the ``WORKSPACE`` file to work around
+`this bazel issue
+<https://github.com/bazelbuild/bazel/issues/14260>`_. Take care not to check in those changes to source
+control.
+
 ----------------
 iOS requirements
 ----------------