Used current user's name instead of config option.
authorEd Bartosh <eduard.bartosh@intel.com>
Tue, 28 Aug 2012 11:46:13 +0000 (14:46 +0300)
committerEd Bartosh <eduard.bartosh@intel.com>
Wed, 29 Aug 2012 10:13:02 +0000 (13:13 +0300)
In cmd_build user name from [remotebuild] section of configuration file
was used as a part of temporary directory name. This approach is confusing
(why anything from remotebuild is used in build mode?) and it also caused
gbs crash when the same name is mentioned in [remotebuild] section for
different local users.

Fixes #283

Change-Id: Ie789564811aa0a9271962d7584421592c215ef7d

gitbuildsys/cmd_build.py

index ef5099f1ec39cfd1807deda7a76f596508ea2ff2..c68f91b419e8c70d0967205fb3ab1a1213af56b7 100644 (file)
@@ -27,6 +27,7 @@ import urllib2
 import glob
 import shutil
 import base64
+import pwd
 from urlparse import urlsplit, urlunsplit
 
 from gitbuildsys import msger, utils, runner, errors
@@ -296,7 +297,7 @@ def do(opts, args):
                    (buildarch, ','.join(SUPPORTEDARCHS)))
 
     build_cmd  = configmgr.get('build_cmd', 'build')
-    userid     = configmgr.get('user', 'remotebuild')
+    userid     = pwd.getpwuid(os.getuid())[0]
     tmpdir     = configmgr.get('tmpdir', 'general')
     build_root = os.path.join(tmpdir, userid, 'gbs-buildroot.%s' % buildarch)
     if opts.buildroot: