Fix invalid directory names

manifest/index.go

@@ -26,6 +26,7 @@ type WixManifest struct {
 	Files          WixFiles     `json:"files,omitempty"`
 	Directories    []string     `json:"directories,omitempty"`
 	RelDirs        []string     `json:"-"`
+	DirNames       []string     `json:"-"`
 	Env            WixEnvList   `json:"env,omitempty"`
 	Shortcuts      WixShortcuts `json:"shortcuts,omitempty"`
 	Choco          ChocoSpec    `json:"choco,omitempty"`
@@ -214,6 +215,7 @@ func (wixFile *WixManifest) RewriteFilePaths(out string) error {
 			return err
 		}
 		wixFile.RelDirs = append(wixFile.RelDirs, r)
+		wixFile.DirNames = append(wixFile.DirNames, cleanDirectoryName(r))
 	}
 	for i, s := range wixFile.Shortcuts.Items {
 		if s.Icon != "" {
@@ -230,6 +232,14 @@ func (wixFile *WixManifest) RewriteFilePaths(out string) error {
 	return nil
 }
 
+// cleanDirectoryName replaces invalid characters in directory names.
+func cleanDirectoryName(name string) string {
+	to := "-"
+	repl := strings.NewReplacer(
+		`\`, to, "?", to, "|", to, ">", to, "<", to, ":", to, "/", to, "*", to, `"`, to)
+	return repl.Replace(name)
+}
+
 // Normalize Appropriately fixes some values within the decoded json
 // It applies defaults values on the wix/msi property to
 // to generate the msi package.

templates/product.wxs

@@ -38,8 +38,8 @@
                   {{end}}
                </Component>
                {{end}}
-               {{if gt (.Directories | len) 0}}
-               {{range $i, $e := .Directories}}
+               {{if gt (.DirNames | len) 0}}
+               {{range $i, $e := .DirNames}}
                <Directory Id="APPDIR{{$i}}" Name="{{$e}}" />
                {{end}}
                {{end}}