-
Notifications
You must be signed in to change notification settings - Fork 344
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’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Microsoft Edge not detected (Quarto 1.6.4) #10464
Comments
Does the following path is correct in your machine? |
ls -l /Applications/Microsoft\ Edge.app/Contents/MacOS/
total 920
-rwxrwxr-x 1 USER admin 469248 1 Aug 20:33 Microsoft Edge |
The path set in the PR is correct then, so the issue might be the following part:
|
Replacing the code I mentioned with the code below (similarly to Windows) works: for (let i = 0; i < programs.length; i++) {
path = programs[i];
if (path && existsSync(path)) break;
} or return programs.find(safeExistsSync);
I think the code should also be updated to observe some environment variable, for instance I would personally like to be able to use Brave ( |
I made a PR with the fix: |
I've installed Quarto 1.6.6 but still get the same error… Anything I can check if some old settings might cause the behavior? Here's some more info:
|
The PR was not merged as indicated by GitHub and this issue is still open. Why would you expect things to change in a new release? |
Sorry, I misread
|
That's the initial PR you even mentioned in your report, which you already know is not working as intended. As for every single issues/discussions, we'll update as soon as there is something to update about. |
@Steinthal the fix should be part of the next pre-release build. |
Just installed 1.6.7 and can confirm that the fix is working . Thanks a lot, guys. |
Bug description
I've removed chromium from my MacBook (and Mac Mini). After a restart, Quarto 1.6.4 complains that no chromium was detected and does not use Microsoft Edge which is installed. Reference: #10172
Steps to reproduce
Take the example from the original post (#10162):
Expected behavior
Edge should be used if chromium is not installed
Actual behavior
Error message:
ERROR: Chrome not found
No Chrome or Chromium installation was detected.
Please run 'quarto install chromium' to install Chromium.
Your environment
Quarto check output
The text was updated successfully, but these errors were encountered: