Skip to content
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

Building on Windows fails with Update.com Error Code #392

Closed
framerate opened this issue May 11, 2016 · 22 comments · May be fixed by qcif/data-curator#563
Closed

Building on Windows fails with Update.com Error Code #392

framerate opened this issue May 11, 2016 · 22 comments · May be fixed by qcif/data-curator#563

Comments

@framerate
Copy link

Running npm run dist on windows exists with the following error:

\node_modules\electron-winstaller-fixed\vendor\Update.com failed with exit code: 4294967295

npm run pack works just fine. I'm trying to create an MSI and work towards windows autoupdating.

Running the latest Windows Technical Preview, in case that matters. Tried both @latest and @3.11.0 suggested in #366

@develar
Copy link
Member

develar commented May 11, 2016

Please attach node_modules/electron-winstaller-fixed/vendor/SquirrelSetup.log.

@framerate
Copy link
Author

framerate commented May 12, 2016

Assuming this is the relevant part:

2016-05-10 19:33:06> Unhandled exception: System.ComponentModel.DataAnnotations.ValidationException: Description is required.
   at NuGet.Manifest.Validate(Manifest manifest)
   at NuGet.Manifest.ReadFrom(Stream stream, IPropertyProvider propertyProvider, Boolean validateSchema)
   at NuGet.LocalPackage.ReadManifest(Stream manifestStream)
   at NuGet.ZipPackage.EnsureManifest()
   at NuGet.ZipPackage..ctor(String filePath, Boolean enableCaching)
   at Squirrel.ReleasePackage.get_SuggestedReleaseFileName()
   at Squirrel.Update.Program.Releasify(String package, String targetDir, String packagesDir, String bootstrapperExe, String backgroundGif, String signingOpts, String baseUrl, String setupIcon, Boolean generateMsi)
   at Squirrel.Update.Program.executeCommandLine(String[] args)
   at Squirrel.Update.Program.main(String[] args)

Apparently a blank package.json description shits the bed. I fixed that and now I get 2016-05-11 19:55:42> Unhandled exception: System.UriFormatException: Invalid URI: The format of the URI could not be determined.

EDIT: Found it, http icons only for windows. Weird

@develar
Copy link
Member

develar commented May 12, 2016

Weird, but it is Windows :( Reopened — must be checked on our side, not on *** Windows *** tool.

@develar develar reopened this May 12, 2016
@framerate
Copy link
Author

Gotcha. Let me know how I can help!
On May 11, 2016 9:51 PM, "Vladimir Krivosheev" [email protected]
wrote:

Weird, but it is Windows :( Reopened — must be checked on our side, not on
*** Windows *** tool.


You are receiving this because you modified the open/close state.
Reply to this email directly or view it on GitHub
#392 (comment)

@marcopiraccini
Copy link

We have the same issue, the strange thing is that all worked until yesterday...
@framerate what does it means "EDIT: Found it, http icons only for windows. Weird"?
I'm using https:// for the icon URL. is this the problem?

@develar
Copy link
Member

develar commented May 13, 2016

@marcopiraccini The same error in the node_modules/electron-winstaller-fixed/vendor/SquirrelSetup.log?

@marcopiraccini
Copy link

Here, on Windows 7 32 bit. Yesterday it worked!

2016-05-13 04:23:04> Program: Starting Squirrel Updater: --releasify C:\app\dist\win\TestApp-0.1.6-full.nupkg --releaseDir C:\app\dist\win --loadingGif C:\app\build\loading-splash.png --no-msi
2016-05-13 04:23:06> Program: Bootstrapper EXE found at:C:\app\node_modules\electron-builder\node_modules\electron-winstaller-fixed\vendor\Setup.exe
2016-05-13 04:23:09> Unhandled exception: System.IO.FileNotFoundException: Could not find file 'C:\app\dist\win\TestApp-0.1.6-full.nupkg'.
File name: 'C:\app\dist\win\TestApp-0.1.6-full.nupkg'
at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath)
at System.IO.FileStream.Init(String path, FileMode mode, FileAccess access, Int32 rights, Boolean useRights, FileShare share, Int32 bufferSize, FileOptions options, SECURITY_ATTRIBUTES secAttrs, String msgPath, Boolean bFromProxy, Boolean useLongPath, Boolean checkHost)
at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share)
at Squirrel.ReleaseEntry.GenerateFromFile(String path, String baseUrl)
at Squirrel.Update.Program.<>c__DisplayClass10_0.b__0(String packageFilename)
at System.Linq.Enumerable.WhereSelectListIterator2.MoveNext() at System.Collections.Generic.List1..ctor(IEnumerable1 collection) at System.Linq.Enumerable.ToList[TSource](IEnumerable1 source)
at Squirrel.Update.Program.ReleasifyElectron(String package, String targetDir, String bootstrapperExe, String backgroundGif, String baseUrl, Boolean generateMsi)
at Squirrel.Update.Program.executeCommandLine(String[] args)
at Squirrel.Update.Program.main(String[] args)

@marcopiraccini
Copy link

(Indeedm the file 'C:\app\dist\win\TestApp-0.1.6-full.nupkg' does not exists...)

@develar
Copy link
Member

develar commented May 13, 2016

@marcopiraccini As a workaround, set electron-buidler version to 3.20.0. Does it help?

(Indeedm the file 'C:\app\dist\win\TestApp-0.1.6-full.nupkg' does not exists...)

What exists? Please list files.

@marcopiraccini
Copy link

Under "win"? Nothing.
In the xxx-win32-ia32 there is all the working application

@marcopiraccini
Copy link

Is electron-builder nos supporting ia32 anymore? 079989a#diff-6456376d7ed2e6f2d66ffc860e13af6eR55

@develar
Copy link
Member

develar commented May 13, 2016

@marcopiraccini Fully supported, it is just a warning. Try to set version to 3.20.0 until your issue is not fixed.

@marcopiraccini
Copy link

marcopiraccini commented May 13, 2016

The same. Also with 3.17.0 (which used to work 3 days ago!)
It seems something broke on a recursive dependency.

(tried again, forgot the caret in dep, with "electron-builder": "3.17.1", works)

@develar
Copy link
Member

develar commented May 13, 2016

@marcopiraccini Reproduced. Fix will be soon. Our test suite didn't check ia32 and error was not catched during CI run.

@marcopiraccini
Copy link

Tnx very much :)

@develar
Copy link
Member

develar commented May 13, 2016

@marcopiraccini BTW, may be you can just migrate to x64 ;)?

@marcopiraccini
Copy link

I cannot, ia32 it's a requirement :(

@marcopiraccini
Copy link

@develar Tested with v3.22.1, but I have the same problem :(

@marcopiraccini
Copy link

marcopiraccini commented May 13, 2016

Summarizing, Windows 7 ia32, using (main package.json, not the app one):

    "electron-builder": "^3.22.1",
    "electron-prebuilt": "^0.37"

build --platform=win32 --arch=ia32 -> Creates the xxx-win32-ia32 folder. Ok
build --platform=win32 --arch=ia32 -d -> Creates the folders in the image:

builder_1

...with win-ia32 empty, and win-ia32-unpacked with this content:

builder_2

...then the nupkg is not created and the subsequent Squirrel releasify fails.

@marcopiraccini
Copy link

(why this is not labelled as "bug"? )

@develar
Copy link
Member

develar commented May 23, 2016

@marcopiraccini Because this issue about "Apparently a blank package.json description shits the bed. " Please try version 3.26.0 and open new issue in case of error (and please attach Squirrel log file and output).

develar added a commit to develar/electron-builder that referenced this issue May 23, 2016
develar added a commit to develar/electron-builder that referenced this issue May 23, 2016
@L3V147H4N
Copy link

L3V147H4N commented May 24, 2016

Still doesn't work on 3.26.3
Just keep getting Update.com failed with exit code: 4294967295
Only pack works

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants