Closed Bug 1748425 Opened 3 years ago Closed 3 years ago

"mach bootstrap" for "GeckoView/Firefox for Android Artifact Mode" doesn't install Java even required

Categories

(Firefox Build System :: Bootstrap Configuration, defect)

defect

Tracking

(Not tracked)

RESOLVED WORKSFORME

People

(Reporter: whimboo, Unassigned)

Details

Trying to build a mobile build of Firefox for Android in artifact mode the "mach build" command fails with:

 0:07.25 ERROR: Could not locate Java at /home/henrik/.mozbuild/jdk/jdk8u312-b07/bin, please run ./mach bootstrap --no-system-changes

But even running "mach bootstrap" with option 3 (mobile artifact build) the Java JDK is not getting installed. To fix that "mach bootstrap" needs to be re-run with option 4 (mobile full build).

If the Java JDK is required for artifact builds (I don't know that) then we should also install it when running "mach bootstrap".

After a rebase to mozilla-central this seems to work now. Could this have been a similar issue as I have seen with bug 1748412?

Flags: needinfo?(mh+mozilla)

The Bugbug bot thinks this bug should belong to the 'Firefox Build System::Bootstrap Configuration' component, and is moving the bug to that component. Please revert this change in case you think the bot is wrong.

Component: General → Bootstrap Configuration

I'm guessing that it was related to the changes in JDK bootstrapping.
Since this is happy post-rebase, I'm leaning towards INVALID-ing this ticket unless it pops up again.

Hm, these changes landed in November so I wouldn't expect to see that the JDK isn't getting installed via mach bootstrap by a checkout of central from late December. But I might have the time tomorrow to check if that specific bug fixed it.

Flags: needinfo?(mh+mozilla) → needinfo?(hskupin)

I cannot replicate that mach bootstrap --no-system-changes doesn't install Java under ~/.mozbuild/jdk. As such lets mark this bug as WFM for now.

Status: NEW → RESOLVED
Closed: 3 years ago
Flags: needinfo?(hskupin)
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.