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

Missing Squirrel Config information - Icon #10

Closed
mix3d opened this issue Jun 23, 2020 · 9 comments · Fixed by electron/windows-installer#344
Closed

Missing Squirrel Config information - Icon #10

mix3d opened this issue Jun 23, 2020 · 9 comments · Fixed by electron/windows-installer#344
Labels
invalid This doesn't seem right

Comments

@mix3d
Copy link

mix3d commented Jun 23, 2020

https://js.electronforge.io/maker/squirrel/interfaces/makersquirrelconfig#iconurl
This references iconUrl, but not icon.

Because iconUrl expects a remote http/https url string, builds will fail when pointed to a local file.
see: electron-userland/electron-installer-windows#55 (comment)

This should note that only one of [iconUrl, icon] should be used, and icon for local app icon files.

@malept malept added the invalid This doesn't seem right label Jun 23, 2020
@malept
Copy link
Contributor

malept commented Jun 23, 2020

electron-installer-windows isn't used in Electron Forge, electron-winstaller is.

@malept malept closed this as completed Jun 23, 2020
@mix3d
Copy link
Author

mix3d commented Jun 23, 2020

Then why was my app still failing to build when I was using iconUrl with a local path?
iconUrl: "./assets/app.ico"

"Invalid URI: The format of the URI could not be determined."

If I try iconUrl: "file://./assets/app.ico", I get

The 'http://schemas.microsoft.com/packaging/2010/07/nuspec.xsd:iconUrl' element is invalid - The value 'file://./assets/app.ico' is invalid according to its datatype 'http://www.w3.org/2001/XMLSchema:anyURI' - The string 'file://./assets/app.ico' is not a valid Uri value.

@malept
Copy link
Contributor

malept commented Jun 23, 2020

@mix3d
Copy link
Author

mix3d commented Jun 23, 2020

Then I would suggest including the very much important bit of information:

"Note that the icon is fetched at installation time rather than packaging"

thus requiring this to be a fully qualified URL publicly available.

@malept
Copy link
Contributor

malept commented Jun 23, 2020

@mix3d
Copy link
Author

mix3d commented Jun 23, 2020

Nope, I am looking for the icon for the application itself, that is displayed on the EXE and in the installed application menu.
image

Without an iconUrl, I get the default app icon on install:
image

What your Squirrel.Windows link says, is that it is grabbed at installation time, which means (unless I'm reading it wrong) it needs to be a fully qualified URL and accessible via internet. (Which is honestly REALLY annoying for offline installs)

See:
electron/windows-installer#151
electron/windows-installer#56

@malept
Copy link
Contributor

malept commented Jun 24, 2020

(Which is honestly REALLY annoying for offline installs)

Perhaps you want the WiX MSI maker instead.

@mix3d
Copy link
Author

mix3d commented Jun 24, 2020

Either way, I would strongly encourage the docs to reflect this strict publicly hosted URL requirement.

@malept
Copy link
Contributor

malept commented Jun 24, 2020

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
invalid This doesn't seem right
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants