Skip to content
This repository has been archived by the owner on Jul 26, 2022. It is now read-only.

Commit

Permalink
#93 Fix mix/matched argument flags.
Browse files Browse the repository at this point in the history
This relates to issue #93 where it was discovered we weren't quoting
the instance name or service name of the parameters passed to
us from the command line. This quotes those values that are passed to
the lib.
  • Loading branch information
drusellers committed May 1, 2012
1 parent e34dbd5 commit 148e00e
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/Topshelf/Hosts/AbstractInstallerHost.cs
Expand Up @@ -106,13 +106,13 @@ Installer CreateInstaller()
string arguments = " "; string arguments = " ";


if (_description.InstanceName.IsNotEmpty()) if (_description.InstanceName.IsNotEmpty())
arguments += " -instance:{0}".FormatWith(_description.InstanceName); arguments += " -instance \"{0}\"".FormatWith(_description.InstanceName);


if (_description.DisplayName.IsNotEmpty()) if (_description.DisplayName.IsNotEmpty())
arguments += " -displayname \"{0}\"".FormatWith(_description.DisplayName); arguments += " -displayname \"{0}\"".FormatWith(_description.DisplayName);


if (_description.Name.IsNotEmpty()) if (_description.Name.IsNotEmpty())
arguments += " -servicename:{0}".FormatWith(_description.Name); arguments += " -servicename \"{0}\"".FormatWith(_description.Name);


var installer = new HostInstaller(_description, arguments, installers); var installer = new HostInstaller(_description, arguments, installers);


Expand Down

0 comments on commit 148e00e

Please sign in to comment.