Clean up upgrade properties; support <?ifdef $()?>

1 files changed, 2 insertions, 2 deletions

diff --git a/src/WixToolset.Core/ExtensibilityServices/PreprocessHelper.cs b/src/WixToolset.Core/ExtensibilityServices/PreprocessHelper.cs
index 60726a02..215c7bc4 100644
--- a/src/WixToolset.Core/ExtensibilityServices/PreprocessHelper.cs
+++ b/src/WixToolset.Core/ExtensibilityServices/PreprocessHelper.cs
@@ -154,10 +154,10 @@ namespace WixToolset.Core.ExtensibilityServices
 
         public string GetVariableValue(IPreprocessContext context, string variable, bool allowMissingPrefix)
         {
-            // Strip the "$(" off the front.
+            // Strip the "$(" off the front and the ")" off the back.
             if (variable.StartsWith("$(", StringComparison.Ordinal))
             {
-                variable = variable.Substring(2);
+                variable = variable.Substring(2, variable.Length - 3);
             }
 
             var parts = variable.Split(VariableSplitter, 2);