985600 - Stop setting JAVA_HOME in buildbot and mozharness to a dir that does not exist

Status: RESOLVED FIXED

(Release Engineering :: Applications: MozharnessCore, defect)

Description

[Justin Wood (:Callek)]

Attachment: [buildbot-configs] v1

So over in Bug 971841 we found that JAVA_HOME is incorrectly set (it's set to where we used to install the jdk on old cent5 machines, pre-mock)

Now that we're using mock and configure is finding java properly, we don't need this. And in fact its mere presence breaks ant as we're trying to use it in Bug 971841.

Comment 1

[Justin Wood (:Callek)]

Comment on attachment 8393653 [details] [diff] [review]
[buildbot-configs] v1

https://hg.mozilla.org/build/buildbot-configs/rev/e8c8603f1ea3

Comment 2

[Justin Wood (:Callek)]

Attachment: [mozharness] v1

Comment 3

[Justin Wood (:Callek)]

Comment on attachment 8393695 [details] [diff] [review]
[mozharness] v1

https://hg.mozilla.org/build/mozharness/rev/f19d877b625b

Comment 4

[Aki Sasaki (not active)]

Comment on attachment 8393695 [details] [diff] [review]
[mozharness] v1

Backed out due to single locale config bustage.
http://hg.mozilla.org/build/mozharness/rev/bc0240453669

Comment 5

[Justin Wood (:Callek)]

Attachment: [mozharness] v2

Not landing this tonight, fwiw.

But this rolls up last patch, and did a full search for JAVA_HOME which no longer returns anything in mozharness

Comment 6

[Aki Sasaki (not active)]

Comment on attachment 8393856 [details] [diff] [review]
[mozharness] v2

I was worried about jarsigner for a bit, then remembered that we use the signing server for android apks now.

Comment 7

[Justin Wood (:Callek)]

Comment on attachment 8393856 [details] [diff] [review]
[mozharness] v2

https://hg.mozilla.org/build/mozharness/rev/4c12054f983d

Comment 8

[Jordan Lund (:jlund)]

in production :)