Browse files

[fix] Preserve HOME env var for version checks

  This is a follow-up to Change Ie3d7f772 , We preserve HOME environment
varialbe in both version check and additional version checking.

  Test plans:
    - Unit tests pass
    - BVT pass on dev_setup
    - BVT pass on dev instance

  Next step: We shouldn't need the additional check, remove this
duplicate.

Change-Id: I3f44a8aa7893ff6bb7a19a7c6855b4d14843bdf5
  • Loading branch information...
1 parent 6e14f86 commit 7d46f3282117bae723600cfb8759f09990f3d048 @d d committed Mar 14, 2012
Showing with 1 addition and 1 deletion.
  1. +1 −1 dea/lib/dea/agent.rb
View
2 dea/lib/dea/agent.rb
@@ -1688,7 +1688,7 @@ def setup_runtimes
expanded_exec.strip!
# java prints to stderr, so munch them both..
- version_check = `env -i #{expanded_exec} #{version_flag} 2>&1`.strip!
+ version_check = `env -i HOME=$HOME #{expanded_exec} #{version_flag} 2>&1`.strip!
unless $? == 0
@logger.info(" #{pname} FAILED, executable '#{runtime['executable']}' not found")
next

0 comments on commit 7d46f32

Please sign in to comment.