-
Notifications
You must be signed in to change notification settings - Fork 95
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
getFileName cannot read property of undefined #54
Comments
What is |
I'm also getting the issue. It looks like
https://github.com/TooTallNate/node-bindings/blob/master/bindings.js#L171 I added some debug logic: console.debug('getFileName|debug|3', {
dummy,
'dummy.stack': dummy.stack,
})
|
same here. I was using node-speaker in a React project with Webpack.
A follow-up question: can node-speaker be used in a front-end project? If I set I tried many ways but had not solve it yet. |
I hit the same issue today. Using for electron. Is there any workaround other than changing target or removing fs, because, i need those two |
For what it's worth, I've also hit this issue when trying to include a local native node module that uses bindings in Electron (no webpack involved here). Haven't figured out a workaround yet. |
exact the same problem here. |
Same error :/ |
Facing same issue, has anyone come up with a solution or workaround? Thanks! |
I'm also encountering this error in a project that uses better-sqlite3 (depends on this). |
Same issue here as @ajmar with electron and better-sqlite3 as well. |
I've encountered this issue using Rollup. It seems to be caused by rollup deleting this line: Line 172 in c8033dc
That If you change the line to |
I've got the same issue with rollup (bundler) and an electron app. A recent PR supposedly made it work with electron, but I don't think it factored in bundling. See: #29 (comment) |
@etylermoss @Stefano1990 @TooTallNate See #66. I proposed a fix for this problem, which does resolve the issues I have. |
Had the same issue with drivelist in an electron app using webpack. Worked fine in dev but not built for production. Finally found some hints on similar issues and found that adding "drivelist" to |
Problem is that if your optimizer is too aggressive, this call will get removed:
Changing the optimizer settings not to optimize such code or exclude drivelist by setting it as external as mentioned above will also work. I fixed it by changing the terser options like this in webpack configuration file;
|
This s a huge issue. Took me hours to figure out a workaround... |
Was no fix found for this?> I'm experiencing the issue with node-midi |
I find a solution for my Electron + Vue app
|
Seeing the below when using https://github.com/jostrander/mouse-forward-back in our project (lbryio/lbry-desktop@cd8436c). We are using webpack 3.
The text was updated successfully, but these errors were encountered: