From 7e41907f9744fe83a1eb30855116437d77ded5e4 Mon Sep 17 00:00:00 2001
From: Ken Mixter <kmixter@google.com>
Date: Mon, 8 Mar 2010 19:37:52 -0800
Subject: [PATCH] Fix minor problem with autotest permissions

When grabbing buildbot image and test artifacts, properly install the test artifacts as if they were built by the user (with the user as owner).

Review URL: http://codereview.chromium.org/703001
---
 sync_build_test.sh | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/sync_build_test.sh b/sync_build_test.sh
index 96f069c443..8743ae50d3 100755
--- a/sync_build_test.sh
+++ b/sync_build_test.sh
@@ -459,9 +459,12 @@ function grab_buildbot() {
         tar_name="${dl_dir}/autotest.tar.bz2"
       fi
       sudo rm -rf "${dir}/autotest"
-      cd ${dir}
+      # Expand in temp directory as current user, then move it as
+      # root to keep local user ownership
+      run_phase "Unpacking buildbot autotest cross-compiled binaries" \
+        tar ${tar_args} "${tar_name}"
       run_phase "Installing buildbot autotest cross-compiled binaries" \
-        sudo tar ${tar_args} "${tar_name}"
+        sudo mv autotest ${dir}
     fi
   fi
   chdir_relative .