Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Flipper Doctor incorrectly reports "Android Studio is not installed." when using Jetbrains Toolbox on OSX #5436

Open
kenyee opened this issue Jan 25, 2024 · 4 comments

Comments

@kenyee
Copy link
Contributor

kenyee commented Jan 25, 2024

馃悰 Bug Report

The Flipper Setup Doctor incorrectly reports "Android Studio is not installed." when it's installed via Jetbrains Toolbox on OSX.
Ditto w/ IDB when installed via Homebrew because it looks in /usr/local/bin for it instead of /opt/homebrew/bin

To Reproduce

Remove all installations of Android Studio from your Applications in OSX.
Install Jetbrains Toolbox.
Use JB Toolbox to install AndroidStudio.

Environment

Flipper version 0.239.0
AS Installed via JB toolbox only (I have latest stable, beta, canary all installed)

@ihgvan
Copy link

ihgvan commented Jan 25, 2024

Same "Android Studio is not installed" symptom for me, but I haven't noticed any adverse effects yet.

@kenyee
Copy link
Contributor Author

kenyee commented Jan 25, 2024

Oops...should have mentioned there are no adverse affects. But one of our devs was wondering why the doctor said it was broken so it's confusing 馃樃

@VictorIreri
Copy link

VictorIreri commented Feb 1, 2024

Looks like it's the code below that defines the Android Studio location so I guess that could be updated to include the default location for Android Studio when installing via JetBrains Toolbox, which seems to be the Applications folder in the user directory.

const hasProblem = !fs.existsSync(
'/Applications/Android Studio.app',

Furthermore, the check should consider that the filename might not be exactly "Android Studio.app" especially if you have a Beta/Preview app installed e.g. Android Studio Iguana 2023.2.1 Beta 2.app or Android Studio Preview.app.

Also, it would be nice if that location could be configured by the user.

@LaGregance
Copy link

Looks like it's the following line that defines the Android Studio location so I guess that could be updated to include the default location for Android Studio when installing via JetBrains Toolbox, which seems to be the Applications folder in the user directory.

const hasProblem = !fs.existsSync(
'/Applications/Android Studio.app',

Furthermore, the check should consider that the filename might not be exactly "Android Studio.app" especially if you have a Beta/Preview app installed e.g. Android Studio Iguana 2023.2.1 Beta 2.app or Android Studio Preview.app.

Also, it would be nice if that location could be configured by the user.

Same problem, I recently fresh install my Mac and install Android Studio using JetBrains Toolbox, so it's installed in ~/Applications (not /Applications)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants