567818 - Start building Fennec nightlies with Android SDK 2.2 (android-8)

Status: RESOLVED FIXED

(Release Engineering :: General, defect)

Description

[Matt Brubeck (:mbrubeck)]

This is required to use the new installLocation feature: http://developer.android.com/guide/appendix/install-location.html

Note: Packages built with the android-8 SDK will still be compatible with older versions of Android, if the minVersion in the AndroidManifest.xml file is set appropriately.

Comment 1

[Aki Sasaki (not active)]

I think this is done, but we're seeing

checking for java... :
checking for javac... :
checking for jar... :
configure: error: The programs java, javac and jar were not found.  Set $JAVA_HOME to your java sdk directory or use --with-java-bin-path={java-bin-dir}
*** Fix above errors and then restart with               "make -f client.mk build"
make[1]: *** [configure] Error 1
make[1]: Leaving directory `/builds/slave/android-trunk/build/mozilla-central'
make: *** [/builds/slave/android-trunk/build/mozilla-central/objdir/Makefile] Error 2

Is this a new requirement, or did something regress?

Bear: if this turns out to be the fix, "export JAVA_HOME=" is checked into various other mozconfigs, so that could be the way to fix this.

Comment 2

[Mike Taylor [:bear]]

The factory used to set the PATH to have the jdk6/bin dir present - did
something change?  

exporting JAVA_HOME doesn't work all of the time when
I was testing for a fix to this before.

Comment 3

[Mike Taylor [:bear]]

(In reply to comment #2)
> The factory used to set the PATH to have the jdk6/bin dir present - did
> something change?  
> 
> exporting JAVA_HOME doesn't work all of the time when
> I was testing for a fix to this before.

sorry - I looked more deeply into this and realize this is now happening during the compile/configure step.  For the "make package" step I had to add the environment.

we should try the export JAVA_HOME method to fix this first.

Comment 4

[Mike Taylor [:bear]]

Attachment: add JAVA_HOME to mozconfig

Comment 5

[John Ford [:jhford] CET/CEST Berlin Time]

Comment on attachment 452518 [details] [diff] [review]
add JAVA_HOME to mozconfig 

http://hg.mozilla.org/build/buildbot-configs/rev/4b8167579a18

We should at some point move this to the environment in config.py or make it global for the slave.

Comment 6

[Mike Taylor [:bear]]

Attachment: --disable-ogg is no longer required

according to IRC conversation with vlad it's no longer required

Comment 7

[Aki Sasaki (not active)]

Comment on attachment 452522 [details] [diff] [review]
--disable-ogg is no longer required

http://hg.mozilla.org/build/buildbot-configs/rev/36ef75cd1673

Comment 8

[Mike Taylor [:bear]]

Attachment: ensure platforms/android-8/tools/ files are +x

and yep, this is why chmod's should be done in the spec and not manually in the zip

Comment 9

[Mike Taylor [:bear]]

Attachment: ensure platforms/android-8/tools/ files are +x

Comment 10

[Mike Taylor [:bear]]

Attachment: ensure platforms/android-8/tools/ files are +x

this one actually runs

Comment 11

[Mike Taylor [:bear]]

Attachment: bump version to 0moz3 for rpm repackaging

Comment 12

[[e-mail] (:bhearsum)]

Comment on attachment 452602 [details] [diff] [review]
bump version to 0moz3 for rpm repackaging

changeset:   187:d687e06045f6

Comment 13

[[e-mail] (:bhearsum)]

Comment on attachment 452601 [details] [diff] [review]
ensure platforms/android-8/tools/ files are +x

changeset:   6:c9e8820d90fe

Comment 14

[Mike Taylor [:bear]]

multiple normal builds have built green and the nightly also built green - marking as resolved