What steps will reproduce the problem?
1. start the Windows installer
2. choose as installation directory one named "Vim huge"
What is the expected output? What do you see instead?
The expected output would be a normal Vim installation, an error message is
shown instead.
What version of the product are you using? On what operating system?
7.4, Windows 7 64 bit
Please provide any additional information below.
Here is the proposed patch:
diff -r 187798e5e4b5 nsis/gvim.nsi
--- a/nsis/gvim.nsi Tue Sep 23 21:53:41 2014 +0200
+++ b/nsis/gvim.nsi Wed Sep 24 23:50:57 2014 +0200
@@ -1,6 +1,5 @@
# NSIS file to create a self-installing exe for Vim.
# It requires NSIS version 2.0 or later.
-# Last Change: 2010 Jul 30
# WARNING: if you make changes to this script, look out for $0 to be valid,
# because uninstall deletes most files in $0.
@@ -38,7 +37,7 @@
RequestExecutionLevel highest
ComponentText "This will install Vim ${VER_MAJOR}.${VER_MINOR} on your computer."
-DirText "Choose a directory to install Vim (must end in 'vim')"
+DirText "Choose a directory to install Vim"
Icon icons\vim_16c.ico
# NSIS2 uses a different strategy with six different images in a strip...
#EnabledBitmap icons\enabled.bmp
@@ -132,14 +131,10 @@
NoCancelAbort:
FunctionEnd
-# We only accept the directory if it ends in "vim". Using .onVerifyInstDir has
+# Using .onVerifyInstDir has
# the disadvantage that the browse dialog is difficult to use.
Function CheckInstallDir
StrCpy $0 $INSTDIR 3 -3
- StrCmp $0 "vim" PathGood
- MessageBox MB_OK "The path must end in 'vim'."
- Abort
- PathGood:
FunctionEnd
Function .onInstSuccess
Original issue reported on code.google.com by tim.lebe...@gmail.com on 19 Oct 2014 at 6:06
Original issue reported on code.google.com by
tim.lebe...@gmail.com
on 19 Oct 2014 at 6:06