Skip to content

Squirrel.Windows Maker requires version to be specified, whereas other makers do not #3047

Open

Description

When I don't specify version in package.json (which I know is not recommended) I get:

An unhandled error has occurred inside Forge:
An error occured while making for target: squirrel
Cannot read properties of undefined (reading 'split')
TypeError: Cannot read properties of undefined (reading 'split')
    at convertVersion (D:\a\taxonium\taxonium\taxonium_electron\node_modules\electron-winstaller\lib\index.js:56:25)

whereas other platforms don't give this error. This inconsistency seems undesirable.

I believe this might be fixed by something like: https://github.com/electron/windows-installer/compare/master...theosanderson:windows-installer:patch-1?expand=1 [ed: there is also at least some downstream usage]

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

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions