From 17cfbc39470de094f5329f2368b6fe2484daa353 Mon Sep 17 00:00:00 2001 From: Michael Comella Date: Mon, 20 Jul 2020 15:36:21 -0700 Subject: [PATCH] No issue: link to Configure Java guide in README; remove NDK requirement. (#12753) I don't have the NDK installed but I can build Fenix just fine. --- README.md | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/README.md b/README.md index ad6224826..f83172937 100644 --- a/README.md +++ b/README.md @@ -67,7 +67,9 @@ Before you attempt to make a contribution please read the [Community Participati ## Build Instructions -Note: Both Android SDK and NDK are required. +Pre-requisites: +* Android SDK +* To run command line tools, you'll need to configure Java: see [our how-to guide](https://github.com/mozilla-mobile/shared-docs/blob/master/android/configure_java.md). 1. Clone or Download the repository: @@ -131,6 +133,9 @@ recommend you use our provided pre-push hook in `config/pre-push-recommended.sh` Using this hook will guarantee your hook gets updated as the repository changes. This hook tries to run as much as possible without taking too much time. +Before you can run the hook, you'll need to configure Java properly because it relies on command line tools: see +[our how-to guide](https://github.com/mozilla-mobile/shared-docs/blob/master/android/configure_java.md). + To add it on Mac/Linux, run this command from the project root: ```sh ln -s ../../config/pre-push-recommended.sh .git/hooks/pre-push