[PATCH] D16196: [test-suite] Add HOME to test env under lit

Matthias Braun via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 14 11:21:17 PST 2016


MatzeB accepted this revision.
MatzeB added a comment.
This revision is now accepted and ready to land.

LGTM. Could be refactored to something like the following though:

  environment_whitelist = ['HOME', 'SSH_AUTH_SOCK']
  for var in environment_whitelist:
    if var in os.environ:
      config.environment[var] = os.environ[var]


http://reviews.llvm.org/D16196





More information about the llvm-commits mailing list