-
Notifications
You must be signed in to change notification settings - Fork 117
Unhandled exception when trying to convert mmd file #16
Comments
It seems one of the changes in the last commit 1f9a6bc is breaking also the dev version. I'll try to narrow it down |
In my case it's puppetteer's new version (with new chrome) giving me grief. |
In windows, the problem seems to be with my antivirus, for some reason it won't let me execute chrome when I use puppeteer 0.12.0 instead of 0.11.0. I tried with a dockerized node 8.9 and again run into problems, with both mermaid.cli 0.2.1 and 0.3.1 It could be tracked down to puppeteer/puppeteer#290 (comment) |
Maybe @tylerlong can take a look... just in case there are some dependencies missing at the OS level? |
@agusmba All dependencies are installed in my system, and it is werid that it is a dependency when the problem is explicitly about an unhandled promise |
The problem appears to be puppeteer failing to properly launch chrome. There seem to be many different reasons for failing to launch chrome, like an antivirus (my case), missing deps, or lack of kernel support. Does mermaid.cli version 0.2.1 work for you? |
Trying to install/uninstall (as sudo) mermaid.cli globally fails with the following error:
I'm not sure if this error is related or may help to this issue |
For what it's worth, version 0.3.1 is working fine on my windows 7 box (no antivirus issues either), node v6.11.0 @angrykoala it sure looks like there's a problem with your puppeteer / chromium |
0.3.1 also working on my windows 10 (convinced the antivirus that chrome.exe is safe). |
Can confirm this happens in Ubuntu 17.10. |
Manually fettling |
This solves the issue with an UnhandledPromiseRejectionWarning issue mermaidjs#16
Ok ... As part of my testing I reverted the version of puppeteer back to 0.12 to make sure it still did not work ... but it did work. This is confusing as now everything is working fine and I cannot get things to break again. Leaving this here in case it helps! |
I cannot reproduce the issue you reported. If you are having issues, please try the following:
|
@tylerlong
My setup: Let me know if you need more information. |
If you install it locally (in your current folder), you don't need to Just The binary you located might not be the one you just installed. And from the error message, it is a permission issue. You can try |
Tested on ubuntu Linux:
I am not going to test other Linux distributions because there are so many of them. |
When executing
mmdc -i my_file.mmd
with node 6.11.5 in mermaidCLI 0.3.1, the following exception happens:Same issue when using node 8 and 9
The minimal file I tried:
The text was updated successfully, but these errors were encountered: