Skip to content

Commit

Permalink
Add valid path check for xcode-select'd Xcode
Browse files Browse the repository at this point in the history
Summary: We should also check that Xcode exists at the path that the user has `xcode-select`'d

Reviewed By: passy

Differential Revision: D34168672

fbshipit-source-id: dc7c36bcb2ad1cec99e6ebb4d531417e55402f01
  • Loading branch information
lawrencelomax authored and facebook-github-bot committed Feb 11, 2022
1 parent ef2b03e commit fbd3212
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 1 deletion.
3 changes: 2 additions & 1 deletion desktop/doctor/package.json
Expand Up @@ -27,6 +27,7 @@
"author": "Facebook, Inc",
"dependencies": {
"envinfo": "^7.8.1",
"fb-watchman": "^2.0.1"
"fb-watchman": "^2.0.1",
"fs-extra": "^10.0.0"
}
}
7 changes: 7 additions & 0 deletions desktop/doctor/src/index.tsx
Expand Up @@ -16,6 +16,7 @@ import * as watchman from 'fb-watchman';
import * as fs from 'fs';
import * as path from 'path';
import type {FlipperDoctor} from 'flipper-common';
import * as fs_extra from 'fs-extra';

export function getHealthchecks(): FlipperDoctor.Healthchecks {
return {
Expand Down Expand Up @@ -188,6 +189,12 @@ export function getHealthchecks(): FlipperDoctor.Healthchecks {
message: `xcode-select has no Xcode selected, You can select it using command "sudo xcode-select -switch <path/to/>Xcode.app".`,
};
}
if ((await fs_extra.pathExists(selectedXcode)) == false) {
return {
hasProblem: true,
message: `xcode-select has path of ${selectedXcode}, however this path does not exist on disk. Run "sudo xcode-select --switch" with a valid Xcode.app path.`,
};
}
return {
hasProblem: false,
message: `xcode-select has path of ${selectedXcode}.`,
Expand Down

0 comments on commit fbd3212

Please sign in to comment.