-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
error in rebuilding native dependency on win10 #8426
Comments
@kerwinxu this looks like 24.13.13 logs for electron-builder I think. If so, can you try |
We faced the same issue and your suggestion actually worked, @mmaietta thanks! Unfortunately, using the v25.0.5 fixes the build on Windows but now is broken on Mac. 😢 Here is the output on Mac OS Sonoma 14.6.1 :
It seems like an issue related to For whatever reason, it tries to find a folder in Version 24.13.3 works fine on Mac. |
@beyondkmp can you please take a look at this when you have a chance? Looks related to the app-builder-bin changes |
The createHash library depends on the md5.js library, and there might be issues with such paths on macOS. I'll try to reproduce and check. https://github.com/browserify/createHash/blob/master/package.json#L35 |
I'm having the same issue on Ubuntu WSL2.
|
@Crease29 Which configuration are you using for the nativeRebuilder? |
I'm not overwriting this config flag, so I assume the default is |
The same command runs perfectly fine with 25.0.0-alpha.7
|
Hmmm, there's not much I can find in the diff (v25.0.0-alpha.7...v25.0.5#diff-7f97d68f3da2a576ab7689e166a9b012ab54ec606fc9833057830240df76c532 other than
|
I've installed electron-builder 25.0.5, applied the changes to that file
|
Just for testing I've added a console log at that spot but it's not even executed.
|
Found the issue! When I remove |
I can confirm that manually removing the
It seems like it is trying to rebuild the With |
I will try it . |
Looks like it needs to run Does this issue happen when building an x64 version of your app? |
@mmaietta thanks for the suggestion, I just tried and here's the result. Running
or
or
Outputs the same error. Let me know if I can help with the troubleshooting in any other way. |
Currently testing version
|
I don't want to bother you but I don't understand why the issue was closed. I'm probably missing something. Currently testing Windows build on macOS Sequoia (MacBook Air M2) version
|
Sorry @TheCrypt0 , looks like that was automation on a separate repo develar/app-builder#132 that was linked to this GH issue. I'll reopen since you're still receiving an error Can you re-execute with |
@mmaietta Thanks for the quick reply! Deleting It is a little strange because I ran Here's the error I'm getting on the windows machine, let me know if this rings a bell: |
This is starting to look like another GH issue reported re: native binaries. Any chance you can provide me with a repo I can clone to test locally on my parallels VM? |
@mmaietta Unfortunately I cannot provide the project I'm working on because it is a work project. I should be able to create one next week, but for now I think what could be useful is that I'm using the |
NVM, I found some time to create a minimal reproducible example. Here's what to expect with a working system: |
@beyondkmp looks like we have an infinite loop/recursion in detecting dependencies via app-builder-bin, I'm seeing this in the logs nonstop. Can you please take a look at the minimal repro project provided?
|
@mmaietta This has been fixed in develar/app-builder#138. |
You're amazing @beyondkmp. I'll kick off the release of electron-builder now then to get this issue resolved. @TheCrypt0, deploying release v25.1.6 now. Please update when you have a chance |
Thank you guys! That was incredibly fast. I'm now getting this output
Solution
Solution Leaving those troubleshooting steps for anyone that might found himself in the same situation in the future. I now got an installer file but is still not working (built on macOS Sequoia (MacBook Air M2), Is there anything else I should try? |
The combination of |
Updates to latest electron-rebuild that allows passing in `platform` for prebuilt node modules (previous versions of electron-rebuild had hardcoded `process.platform`) Fixes: electron-userland#8528 electron-userland#8426
Released in 25.1.7, please feel free to give it a shot! |
Hey @mmaietta , unfortunately it doesn't work.
|
Looks like |
I Sloved this problem , because the thread of vue can not run native , so the main thread can run it 。 |
kindly the below error
but i can Manually build.
The text was updated successfully, but these errors were encountered: