leerob / youtube-to-mp3

⚡️Electron application to convert and download YouTube videos as MP3s
https://electronjs.org/apps/youtube-to-mp3
MIT License
540 stars 132 forks source link

Got error message on macOS 10.13.1 #3

Closed livingstrong closed 6 years ago

livingstrong commented 6 years ago

I got the following message when I execute "npm start" on macOS 10.13.1 , any suggestion?

Electron could not be found. No hard resets for you! [warn] kq_init: detected broken kqueue; not using.: Undefined error: 0 [warn] kq_init: detected broken kqueue; not using.: Undefined error: 0 [warn] kq_init: detected broken kqueue; not using.: Undefined error: 0 [warn] kq_init: detected broken kqueue; not using.: Undefined error: 0

leerob commented 6 years ago

Is electron properly installed?

npm install electron --save-dev

Maybe try something here? https://github.com/yan-foto/electron-reload/issues/16

IT-MikeS commented 6 years ago

I had issues with electron reload on windows as well, after doing an update on dependencies ( updated to newest electron instead of prebuilt and new electron reload) all went fine.

leerob commented 6 years ago

Would you mind submitting a PR?

On Dec 1, 2017, 1:14 PM -0600, Mike S notifications@github.com, wrote:

I had issues with electron reload on windows as well, after doing an update on dependencies ( updated to newest electron instead of prebuilt and new electron reload) all went fine. — You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.

leerob commented 6 years ago

Disregard - I see you did! Thanks!

On Dec 1, 2017, 1:16 PM -0600, Lee Robinson lrobinson2011@gmail.com, wrote:

Would you mind submitting a PR?

On Dec 1, 2017, 1:14 PM -0600, Mike S notifications@github.com, wrote:

I had issues with electron reload on windows as well, after doing an update on dependencies ( updated to newest electron instead of prebuilt and new electron reload) all went fine. — You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.

IT-MikeS commented 6 years ago

Yeah I did for a few things, feel free to message me if need be about any change I made

leerob commented 6 years ago

@jerry5chang Could you try pulling down this PR https://github.com/leerob/youtube-to-mp3/pull/4 and seeing if it fixes the issues you were having?

IT-MikeS commented 6 years ago

@jerry5chang @leerob this warning is harmless however I believe PR #4 or PR #5 would fix this as electron is upgraded and I think it has to do with chromium in electron. Some people over at this issue thread (https://github.com/spyder-ide/spyder/issues/3730#issue-191337353) have had some luck suppressing it, but some haven't. Rest assured nothing will malfunction in the app because of the warnings 😄

leerob commented 6 years ago

Going to close this based on @IT-MikeS's comment. Hope it's working for you @jerry5chang!