Skip to content

NSIS installer crashes when provisioning file is added #1220

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

Closed
tresf opened this issue Nov 22, 2023 · 1 comment
Closed

NSIS installer crashes when provisioning file is added #1220

tresf opened this issue Nov 22, 2023 · 1 comment
Assignees

Comments

@tresf
Copy link
Contributor

tresf commented Nov 22, 2023

This bug is reproducible with #1190 (provision branch checked out exactly 80a71ea) calling the following:

ant -Dprovision.file=./test/qz/installer/provision/resources/provision.json nsis
  • Install the produced .exe on Windows, the installer crashes
  • Make NSIS 3.0.8
  • Building on Mac M1
  • Bug goes away if the provisioning file is removed

The issue has temporarily gone away with this commit: 38e41b1. My current theory is that there's a bug with nsExec::ExecToLog:

nsExec::ExecToLog '"$java" ${install.opts} -jar "$OUTDIR\${project.filename}.jar" "${step}" "${option}" "${value}"'

Upstream bug report:
https://sourceforge.net/p/nsis/bugs/1301/

@tresf tresf added the bug label Nov 22, 2023
@tresf tresf self-assigned this Nov 22, 2023
tresf added a commit that referenced this issue Nov 22, 2023
@tresf
Copy link
Contributor Author

tresf commented Nov 22, 2023

Closed via 2f7fe3a. Our envp variable was MUCH MUCH longer than the maximum NSIS string length and it was causing random crashes. The issue surfaced with #1190 because it performs additional shell commands which echo this to the terminal.

Filed a bug upstream with NSIS team: https://sourceforge.net/p/nsis/bugs/1301/

@tresf tresf closed this as completed Nov 22, 2023
@tresf tresf added the upstream label Nov 22, 2023
tresf added a commit that referenced this issue Nov 25, 2023
tresf added a commit that referenced this issue Nov 25, 2023
tresf added a commit that referenced this issue Dec 21, 2023
Add provisioning support through a JSON file.

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

No branches or pull requests

1 participant