Doctor/windows

packages/cli/src/commands/doctor/healthchecks/androidSDK.js

@@ -2,15 +2,60 @@ import chalk from 'chalk';
 import {logManualInstallation} from './common';
 import versionRanges from '../versionRanges';
 import {doesSoftwareNeedToBeFixed} from '../checkInstallation';
+import {exec} from 'child_process';
+
+const installMessage = `Read more about how to update Android SDK at ${chalk.dim(
+  'https://developer.android.com/studio',
+)}`;
 
 export default {
   label: 'Android SDK',
-  getDiagnosticsAsync: async ({SDKs}) => ({
-    needsToBeFixed: doesSoftwareNeedToBeFixed({
-      version: SDKs['Android SDK']['Build Tools'][0],
-      versionRange: versionRanges.ANDROID_NDK,
-    }),
-  }),
+  getDiagnosticsAsync: async ({SDKs}) => {
+    let sdks = SDKs['Android SDK'];
+
+    // This is a workaround for envinfo's Android SDK check not working on
+    // Windows. This can be removed when envinfo fixes it.
+    // See the PR: https://github.com/tabrindle/envinfo/pull/119
+    if (sdks === 'Not Found' && process.platform !== 'darwin') {
+      try {
+        sdks = await new Promise((resolve, reject) => {
+          exec(
+            process.env.ANDROID_HOME
+              ? `${process.env.ANDROID_HOME}/tools/bin/sdkmanager --list`
+              : 'sdkmanager --list',
+            (err, stdout) => {
+              if (err) {
+                reject();
+              } else {
+                const matches = [];
+                const regex = /build-tools;([\d|.]+)[\S\s]/g;
+                let match = null;
+                while ((match = regex.exec(stdout)) !== null) {
+                  matches.push(match[1]);
+                }
+                matches.length > 0
+                  ? resolve({
+                      'Build Tools': matches,
+                    })
+                  : reject();
+              }
+            },
+          );
+        });
+      } catch {
+        sdks = 'Not Found';
+      }
+    }
+
+    return {
+      needsToBeFixed:
+        (sdks === 'Not Found' && installMessage) ||
+        doesSoftwareNeedToBeFixed({
+          version: sdks['Build Tools'][0],
+          versionRange: versionRanges.ANDROID_NDK,
+        }),
+    };
+  },
   runAutomaticFix: async ({loader, environmentInfo}) => {
     const version = environmentInfo.SDKs['Android SDK'][0];
     const isNDKInstalled = version !== 'Not Found';
@@ -19,15 +64,13 @@ export default {
 
     if (isNDKInstalled) {
       return logManualInstallation({
-        message: `Read more about how to update Android SDK at ${chalk.dim(
-          'https://developer.android.com/studio',
-        )}`,
+        message: installMessage,
       });
     }
 
     return logManualInstallation({
       healthcheck: 'Android SDK',
-      url: 'https://developer.android.com/studio',
+      url: 'https://facebook.github.io/react-native/docs/getting-started',
     });
   },
 };

packages/cli/src/commands/doctor/healthchecks/index.js

@@ -16,7 +16,12 @@ export const HEALTHCHECK_TYPES = {
 export const getHealthchecks = ({contributor}) => ({
   common: {
     label: 'Common',
-    healthchecks: [nodeJS, yarn, npm, watchman],
+    healthchecks: [
+      nodeJS,
+      yarn,
+      npm,
+      ...(process.platform === 'darwin' ? [watchman] : []),
+    ],
   },
   android: {
     label: 'Android',