Wireshark-dev: [Wireshark-dev] MAKEDIR NSIS packaging issue with Wireshark 1.10.8

From: Ben Burwell <bburwell@xxxxxxxxxx>
Date: Tue, 24 Jun 2014 09:07:16 -0400
Hello,

When trying to create a package using NSIS, I've run into an issue where $MAKEDIR contains spaces (C:\Program Files (x86)\...), which does not seem to be handled correctly.

I've found this bug report, which is nearly two years old: https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=7870

For example (from the bug report):

#nmake -f Makefile.nmake
......
"C:\Program Files\NSIS\makensis.exe"  /DPROGRAM_NAME=wireshark /DMAKEDIR=C:\Documents and Settings\Administrator\test\wireshark\packaging\nsis
......
Command line defined: "PROGRAM_NAME=wireshark"
Command line defined: "MAKEDIR=C:\Documents"
......
Can't open script "and"
NMAKE : fatal error U1077: '"C:\Program Files\NSIS\makensis.exe"' : return code'0x1'Stop.
NMAKE : fatal error U1077: '"C:\Program Files\Microsoft Visual Studio 10.0\VC\BIN\nmake.exe"' : return code '0x2' Stop.
......

>From what I can see, this is still open and present in releases, though a patch has been submitted in the bug report.

I was unable to find anything related in Gerrit. Does the patch attached in the bug report just need to be submitted on Gerrit?