-
-
Notifications
You must be signed in to change notification settings - Fork 21
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
rcedit.exe failed - unable to commit changes #590
Comments
On a hunch I rolled back to electron-packager 7.0.0. The build works fine, though it errors out on an lstat, the resulting app.exe is still there with no performance issues. I will try and locate when this issue was initially brought in. |
The issue was introduced in release 7.7.0 - version 7.6.0 does the build fine as previously stated. |
That doesn't make any sense to me. Here are the changes between 7.6.0 and 7.7.0: v7.6.0...v7.7.0 Here's the error message in the I am not a Windows dev, so I would suggest filing a bug in the rcedit issue tracker and seeing if someone responds. |
Agreed, I was just looking through commits between tags 7.6.0 and 7.7.0 and it seems to be minimal changes. BUT I just removed 7.6.0 and re-installed 7.7.0 - same result, albeit different presentation than the latest version.
rcedit is brought in through node-rcedit - is there a difference in the version being pulled down between 7.7.0 and 7.6.0 that I'm not seeing to narrow down the problem in rcedit? |
Where is your project located? In your home directory? Have you tried to disable MS Defender or another antivirus? |
....and the answer is no, both request ^0.5.1. My 7.7.0 install install and 7.6.0 install also have 0.5.1 rcedit in their node_modules, so its not a problem with versioning of rcedit. I've also confirmed that 7.6.0 will build where 7.7.0 does not. I'm at a loss. |
@hlfshell I think I know what the problem is. Could you link a gist of your |
I'm on a company computer, so there is antivirus via Cylance. I can try building the project tonight on a personal computer without the antivirus to see if there's a difference. I'm performing the build in C:\Users\myuser\project-folder\
Here you go!
I HAVE done the electron-rebuild process for serialport, though not every time. Could that be a cause? |
@hlfshell FYI, you should remove Hmmm. So I figured the problem was that in 7.7.0, we started using |
@malept Unfortunately that's a no-go. Re-installed v7.7.0:
|
@hlfshell oh, leave it as |
Sorry, my mistake. Re-ran:
Same result, I'm afraid. |
@hlfshell Do you happen to have access to a different machine? It can have the same antivirus or whatever. |
Yes, I will try to build elsewhere and will report back. Searching for a difference or do you have an idea in mind? |
Just searching for a difference. Also, If you have access to a Mac/Linux machine, I think that would solve your problem. (assuming you install Wine, etc.) |
Afraid the only Linux boxes I have are raspberry pis, so that would prove difficult to build from. Will report back with a different machine in a bit. |
I found this, it might be related: electron/node-rcedit#6 (comment) |
I found that earlier, and created a .ico using an online convereter in case it was as simple as not providing an icon. But let's give it a try with everything else we've done so far. Using v7.7.0:
No-go I'm afraid. |
I was more referring to the comment that it supposedly doesn't work with Node, but does if you invoke rcedit directly. |
I am having a similar problem. I have just started with electron-packager and am a new member here at Github. If this is different, let me know and I will start a new issue. I sometimes get the error above, but also get the more frequent error:
My script is:
My package is:
The only thing I have in the background is Visual Studio where I do my code editing. |
@JAA17 The EPERM error is a different error, please file a different issue. |
Just wanted to comment that I also encountered this error and rolling back also solved the issue for me. I work with @hlfshell and another, third coworker just ran into this problem, so at this point it's safe to say this has something to do with our dev setup. Cylance (network antivirus) seems like a likely culprit to me. |
At this point I would suggest trying to use Electron Packager with the network shut off (assuming you already have prebuilt Electron cached on the machine), given your network antivirus comment. |
We've re-imaged our systems due to a company buyout, and the problem magically went away. I've confirmed now on two other systems that this does not occur in Windows 10. This leads me to believe that something locally in our company caused the issue. Not sure why, since there was no difference in the rcedit executables between our versions. Let's close this issue unless someone else ever runs into it. Thanks for the help everyone! |
I'm having this same issue on macOS 10.12.4. I'm using wine v2.0.1 from homebrew, electron v1.6.6 and electron-packager v8.6.0 from npm.
Building for macOS works fine. I tried the above suggestion of using |
@mikezila has it ever worked for you? Is it possible to try using a real Windows machine, a Windows VM, or an earlier version of Wine? |
Trying it on a Windows 7 vm with a 100% fresh install of node and friends seems to work fine. I was previously using wine 2.0.1, when I used homebrew to swap to an earlier version (2.0) it seemed to work fine, but still failed once I swapped back to 2.0.1. Doing a Thanks for you reply. |
@mikezila , how did you get to finally fix packaging win32 on your mac? I am facing this issue. packaging for macOS works fine. This used to work smoothly on my previous machine before i did a time-machine backup and restored into another mac just last week. I have tried out all tips i have found on the internet.
looks like these dependencies are missing. could this have caused my challenges packaging for windows ? Pls, help. thanks. |
Just restart your machine. It is due to Windows permission issue. I tried to set permission for that folder. But I was not able to. Then restarted machine The issue got resolved. :) |
I'm seeing this come up at least 3 times a day on my CI machines, where rebooting 3 times a day isn't really all that realistic. Can anyone clarify which files or folders windows is supposedly taking a lock on or having permissions problems with? Is there anything I can do to make the whole process a bit more robust so that I can have some dependable builds again? |
@ronnyek did u solve this? same thing here. |
I do experience the same thing, it worked once and after that it doesn't work. Removing the dist folder manually did not work either. |
+1. |
Got the same error:
Hope this helps someone. |
Disabling the Antivirus worked for me! |
Disabling the Antivirus worked for me too! |
I was having the same issue, turns out it was due to the Windows Ransomware protection. Disabling it solved the issue for me. |
rcedit.exe failed with exit code 1. Fatal error: Unable to commit changes 禁用掉防病毒软件windows防火墙再打包就ok了! |
呵呵哒,确实是这样,把所有的电脑管家和防火墙关了就ok了。 |
Thank you so much. This solved my problem! The |
Thank you so much. This solved my problem |
is there any workaround for Cyclance security software? I'm having the same issue on Windows 10, version electron-builder version=22.4.1 os=10.0.17763 |
For the ones that have the issue while using Windows OS and default antivirus (Defender) the steps below will resolve your issue.
|
Disabled Dropbox Sync and it worked. Apparently, external processes reading/locking files causes this problem. |
This issue caused me a lot of pain. What I did today to overcome it, was to clone the rcedit repo, set up Visual Studio (and add all the VS components needed to compile C+ code such as Windows 10 SDK), and build the exe myself. I then replaced \node_modules\rcedit\bin\rcedit.exe with the one I built and since then no issue. |
This didn't resolve my issue. However, what DID resolve my issue was opening Task Manager and ending any process called 'electron.exe'. |
Happened to me when using Synology Drive (or probably any sync tool). Workaround: copy the project locally. |
I'm am using electron-packager 8.5.1 with Electron 1.4.13 (also tried using 1.2.2), attempting to build on Windows 10 for Windows.
I keep getting the following:
While I'm aware that electron-packager comes with rcedit, I've downloaded rcedit from the electron/rcedit repos and placed it into my path to ensure I can execute rcedit and it isn't a pathing issue.
I can't find much through google or the issues of electron-packager or rcedit, so I was hoping someone would have an idea on what I can do from here.
The text was updated successfully, but these errors were encountered: