qzind / tray

Browser plugin for sending documents and raw commands to a printer or attached device.
https://qz.io
Other
824 stars 266 forks source link

NSIS installer crashes when provisioning file is added #1220

Closed tresf closed 7 months ago

tresf commented 7 months ago

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

The issue has temporarily gone away with this commit: 38e41b1. My current theory is that there's a bug with nsExec::ExecToLog: https://github.com/qzind/tray/blob/3c0ab9495b74ec440cd8faf78b8bdc81ef155ad4/ant/windows/windows-installer.nsi.in#L42

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

tresf commented 7 months ago

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/