Closed Bug 1427790 Opened 7 years ago Closed 6 years ago

Bootstrap for android installs latest java (9 now on mac). Should install java 8 to keep android SDK compatibility


(Firefox Build System :: Android Studio and Gradle Integration, defect)

58 Branch
Not set


(firefox59 fixed, firefox60 fixed)

Tracking Status
firefox59 --- fixed
firefox60 --- fixed


(Reporter: p.bertran, Assigned: p.bertran)




(2 files)

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.13; rv:57.0) Gecko/20100101 Firefox/57.0
Build ID: 20171206182557

Steps to reproduce:

Bootstrapping "Firefox for android" on macOS

Actual results:

It installed java 9, which is incompatible with the android SDK

Expected results:

It should have installed Java 8
This can be fixed by adding an 8 there:

I join a patch that does just that. Not sure about the patch format thought as I never contributed myself yet (it's just a plain hg diff).
OS: Unspecified → Android
Hardware: Unspecified → All
Attachment #8939570 - Attachment is patch: true
Attachment #8939570 - Attachment mime type: text/x-patch → text/plain
Comment on attachment 8939570 [details] [diff] [review]

Review of attachment 8939570 [details] [diff] [review]:

Thanks for this!
Attachment #8939570 - Flags: review+
This needs check-in, but might also need a good commit message.  Sheriff, could you try:

Bug 1427790 - Specify Java version 8 in Mac OS X mobile/android bootstrap. r=nalexander
Assignee: nobody → p.bertran
Ever confirmed: true
Keywords: checkin-needed
Pushed by
Ensure that Java8 is bootstrapped on Mac to maintain Android SDK compatibility. r=nalexander
Keywords: checkin-needed
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 59
Looks like we got this wrong -- it's supposed to be

brew cask install caskroom/versions/java8

and we're doing

brew cask install java8

This requires us to partially roll back, since the cask business has changed yet again.
Flags: needinfo?(mozilla)
Resolution: FIXED → ---
Clearing NI; just wanted mkaply to see this ticket.
Flags: needinfo?(mozilla)
See Also: → 1428887
Hey - I just hit this when attempting to run ./mach boostrap for Fennec Artifact builds:

Please choose the version of Firefox you want to build:
1. Firefox for Desktop Artifact Mode
2. Firefox for Desktop
3. Firefox for Android Artifact Mode
4. Firefox for Android
Your choice: 3

Looks like you have Homebrew installed. We will install all required packages via Homebrew.

We are now installing all required packages via Homebrew. You will see a lot of
output as packages are built.

Error: Cask 'java8' is unavailable: No Cask with this name exists.
Error running mach:


The error occurred in code that was called by the mach command. This is either
a bug in the called code itself or in the way that mach is calling it.

You should consider filing a bug for this issue.

If filing a bug, please include the full output of mach, including this error

The details of the failure are as follows:

CalledProcessError: Command '[u'/usr/local/bin/brew', u'cask', u'install', u'java8']' returned non-zero exit status 1

  File "/Users/mikeconley/Projects/mozilla-central/python/mozboot/mozboot/", line 32, in bootstrap
  File "/Users/mikeconley/Projects/mozilla-central/python/mozboot/mozboot/", line 297, in bootstrap
    getattr(self.instance, 'install_%s_packages' % application)()
  File "/Users/mikeconley/Projects/mozilla-central/python/mozboot/mozboot/", line 206, in install_mobile_android_artifact_mode_packages
  File "/Users/mikeconley/Projects/mozilla-central/python/mozboot/mozboot/", line 357, in ensure_homebrew_mobile_android_packages
    installed = self._ensure_homebrew_casks(casks)
  File "/Users/mikeconley/Projects/mozilla-central/python/mozboot/mozboot/", line 315, in _ensure_homebrew_casks
    return self._ensure_homebrew_packages(casks, extra_brew_args=['cask'])
  File "/Users/mikeconley/Projects/mozilla-central/python/mozboot/mozboot/", line 309, in _ensure_homebrew_packages
    subprocess.check_call(cmd + ['install', package])
  File "/usr/local/Cellar/python/2.7.13_1/Frameworks/Python.framework/Versions/2.7/lib/python2.7/", line 186, in check_call
    raise CalledProcessError(retcode, cmd)

I suspect we need to correct the error highlighted in comment 5 of this bug. Would you be able to provide a new patch, Pierre?
Flags: needinfo?(p.bertran)
Note that I was able to work around the issue by installing java8 manually via

brew cask install caskroom/versions/java8

(from comment 5)
(In reply to Mike Conley (:mconley) (:⚙️) - Backlogged on reviews and needinfos from comment #8)
> Would you be able to provide a new patch, Pierre?

Yes I can change the patch to make it look for "caskroom/versions/java8"

But I guess the problem with my patch is the need for this line, which have been removed in the patch mentionned by nalexander in comment 5 :

So I'm not sure why this has been removed. Maybe restore this and let 'java8' alone could be a better approach.

With your advice on this, I'll make a new patch.
Flags: needinfo?(p.bertran) → needinfo?(nalexander)
I hit Alex's Homebrew issue, and I changed it from java8 to java and it seems like it's working for me. Datapoint++
Comment on attachment 8949844 [details]
Bug 1427790 - Bootstrap caskroom/versions/java8 for macOS+brew+mobile/android.

::: python/mozboot/mozboot/
(Diff revision 1)
> +        # Ensure we've tapped the caskroom.  This is idempotent, so no need to
> +        # avoid repeat invocation.

I admire the cleverness of Homebrew for all their beer-related terminology; perhaps we should clarify what "tapped the caskroom" is here for people who aren't intimately familiar with Homebrew terminology?
Attachment #8949844 - Flags: review+
Attachment #8949844 - Flags: review?(core-build-config-reviews) → review+
Pushed by
Bootstrap caskroom/versions/java8 for macOS+brew+mobile/android. r=froydnj
Closed: 7 years ago6 years ago
Resolution: --- → FIXED
Target Milestone: Firefox 59 → Firefox 60
Clearing NI since I believe I this is now addressed.
Flags: needinfo?(nalexander)
Product: Firefox for Android → Firefox Build System
Target Milestone: Firefox 60 → mozilla60
Version: Firefox 58 → 58 Branch
You need to log in before you can comment on or make changes to this bug.