Bug 1427061 - Only prepare hg store path when at least one checkout is performed. r?dustin draft
authorMike Hommey <mh+mozilla@glandium.org>
Tue, 26 Dec 2017 13:19:45 +0900
changeset 714569 123a0949da2922e2fbc6d55e4e2c7e90124bf306
parent 714514 2e0db1b48499fcf1598f23e04211360f9d39b7af
child 714570 c0aaada03270f973d73c01b37376e6ac6140993e
child 714571 f9aacb4510c2b09d436b57ac016debb08f92678c
push id93951
push userbmo:mh+mozilla@glandium.org
push dateTue, 26 Dec 2017 04:23:13 +0000
reviewersdustin
bugs1427061
milestone59.0a1
Bug 1427061 - Only prepare hg store path when at least one checkout is performed. r?dustin While we're here, add a missing prepare_vcs_checkout for the comm-central checkout.
taskcluster/docker/recipes/run-task
--- a/taskcluster/docker/recipes/run-task
+++ b/taskcluster/docker/recipes/run-task
@@ -542,17 +542,19 @@ def main(args):
             if e.errno != errno.EEXIST:
                 raise
 
         if running_as_root:
             os.chown(store_path, uid, gid)
 
     prepare_checkout_dir(args.vcs_checkout)
     prepare_checkout_dir(args.tools_checkout)
-    prepare_hg_store_path()
+    prepare_checkout_dir(args.comm_checkout)
+    if args.vcs_checkout or args.tools_checkout or args.comm_checkout:
+        prepare_hg_store_path()
 
     if running_as_root:
         # Drop permissions to requested user.
         # This code is modeled after what `sudo` was observed to do in a Docker
         # container. We do not bother calling setrlimit() because containers have
         # their own limits.
         print_line(b'setup', b'running as %s:%s\n' % (args.user, args.group))
         os.setgroups(gids)