-
Notifications
You must be signed in to change notification settings - Fork 39
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
Report more information when a prebuild is not found #54
Comments
We already do: Lines 47 to 60 in 0078d78
|
This is not happening when people run |
I would guess because of https://github.com/zeromq/zeromq.js/blob/dd72eed5835e321e5c728959744e08f093557beb/package.json#L73 (the additional steps before |
I added those just right now, and it is not released yet. Those reports are older. |
OK, may need |
Looking at the code, the code path is not clear to me. A lot of old-style JS dynamism and prototype assignment is used. Using TypeScript could make it easier to find the reason. Where is the call site of the Lines 18 to 25 in 0078d78
I also see some ignored error variables. Line 9 in 0078d78
|
Currently, it is quite hard to understand why a prebuild is not picked up, especially when the log comes from the end user. I think adding a
console.log
that includes some information like the host's platform, architecture, libc, etc. will help debug the issues.The text was updated successfully, but these errors were encountered: