Windows icon path unquoted, leads to error

• Version: electron-builder@20.40.2

• Target: Windows

In my windows configuration, I use the following:

    "win": {
      "target": "zip",
      "icon" : "logos/favicon/Favicon.ico"
    }  

And the release command:

"release": "npm test && npm run generate-license && webpack --config=build/webpack.app.config.js --env=production && electron-builder",

When I run “npm run release”, I get an error because the windows file path is not quoted, so drive “C” is the attempted target for finding an icon:

  • electron-builder version=20.40.2
  • loaded configuration file=package.json ("build" field)
  • no native production dependencies
  • packaging       platform=win32 arch=x64 electron=2.0.2 appOutDir=dist\win-unpacked
Error: Exit code: 1. Command failed: C:\Users\Joe\AppData\Local\electron-builder\Cache\winCodeSign\winCodeSign-2.4.0\rcedit-ia32.exe C:\Users\Joe\Documents\Organizations\Satyrn\satyrn\dist\win-unpacked\Satyrn.exe --set-version-string FileDescription Satyrn --set-version-string ProductName Satyrn --set-version-string LegalCopyright Copyright © 2019 Joe Andrieu --set-file-version 0.0.0 --set-product-version 0.0.0.0 --set-version-string InternalName Satyrn --set-version-string OriginalFilename  --set-version-string CompanyName Joe Andrieu --set-icon C:\Users\Joe\Documents\Organizations\Satyrn\satyrn\logos\favicon\Favicon.ico
Reserved header is not 0 or image type is not icon for 'C'
Fatal error: Unable to set icon

Reserved header is not 0 or image type is not icon for 'C'
Fatal error: Unable to set icon

    at C:\Users\Joe\Documents\Organizations\Satyrn\satyrn\node_modules\builder-util\src\util.ts:126:16

[I snipped the rest of the trace]

Note: the constructed file path for the icon is correct. If it were just quoted, I believe this would work.

I looked through the source, but my initial inspection did not find the spot where the command is constructed.

Any pointers would be appreciated.

[lvsouza]

I had this same problem. In my case the problem was in the format of the single image. I used a different converter for the ico format and it worked without problems.

I hope it helps you!

[cyberdev101]

I had this same problem. In my case the problem was in the format of the single image. I used a different converter for the ico format and it worked without problems.

I hope it helps you!

Thank you so much, this was perfect! This bug should be closed.

Detailed Explanation: When compiling with Electron Packager/Builder, I got this: WARNING: Could not find icon "C:\Users\AM\electron-quick-start\icon.ico", not updating app icon This told me that it was automatically searching for a .ico file. So I decided to rename the file from icon.png to icon.ico. This might of changed the file type, but the binaries of PNG wouldnt be converted to ICO. You need to use an PNG to ICO converter for this. Thank you to @lvsouza for sparking this answer!