From: Daira Hopwood <daira@jacaranda.org>
Date: Fri, 30 Jan 2015 22:05:54 +0000 (+0000)
Subject: zetuptoolz: don't add ';' to an empty path when appending a new item.
X-Git-Url: https://git.rkrishnan.org/components/architecture.txt?a=commitdiff_plain;h=a4612e118387b45a68648e8f84d3a675faa49b42;p=tahoe-lafs%2Ftahoe-lafs.git

zetuptoolz: don't add ';' to an empty path when appending a new item.

Signed-off-by: Daira Hopwood <daira@jacaranda.org>
---

diff --git a/setuptools-0.6c16dev5.egg/setuptools/command/scriptsetup.py b/setuptools-0.6c16dev5.egg/setuptools/command/scriptsetup.py
index e4dd9dd0..b066820c 100644
--- a/setuptools-0.6c16dev5.egg/setuptools/command/scriptsetup.py
+++ b/setuptools-0.6c16dev5.egg/setuptools/command/scriptsetup.py
@@ -159,10 +159,16 @@ def do_scriptsetup(allusers=False, addpath=None):
             (value, type) = user_valueandtype or system_valueandtype or (u'', REG_SZ)
             key_name_path = (HKEY_CURRENT_USER, "HKEY_CURRENT_USER", USER_ENV)
 
+        def path_append(value, addition):
+            if value != "":
+                return value + u';' + addition
+            else:
+                return addition
+
         if addition.lower() in value.lower().split(u';'):
             print "Already done: %s." % (goal,)
         else:
-            changed |= update(key_name_path, varname, value + u';' + addition, type, goal, what)
+            changed |= update(key_name_path, varname, path_append(value, addition), type, goal, what)
 
         if change_allusers:
             # Also change any overriding environment entry for the current user.
@@ -172,7 +178,7 @@ def do_scriptsetup(allusers=False, addpath=None):
             if not (addition.lower() in split_value or u'%'+varname.lower()+u'%' in split_value):
                 now_what = "the overriding user environment variable %s" % (varname,)
                 changed |= update((HKEY_CURRENT_USER, "HKEY_CURRENT_USER", USER_ENV),
-                                  varname, user_value + u';' + addition, user_type,
+                                  varname, path_append(user_value, addition), user_type,
                                   "add %s to %s" % (addition, now_what), now_what)
 
         return changed