1379426 - mach bootstrap should maybe not try to install Xcode

Status: RESOLVED FIXED

(Firefox Build System :: Bootstrap Configuration, defect)

Description

[Mike Hommey [:glandium]]

Turns out, Apple is making things hard for people not using the latest version of OSX. I happened to have installed El Capitan (10.11) for reasons, and needed to setup a build environment on it.

mach bootstrap runs xcode-select --switch /Applications/Xcode.app, which opens the App Store for Xcode, which, as of today, shows version 8.3.3 of Xcode... but you can't install that version because it's not compatible with El Capitan. I don't think there's a way to download older versions that doesn't involve logging on https://developer.apple.com.

It also turns out that the build is pretty happy with the command line tools, which I presume one can install with xcode-select --switch /Library/Developer/CommandLineTools (I ended up installing them some other way, I'm just guessing that xcode-select command would work).

Maybe mach bootstrap should install that instead...

Comment 1

[Mike Hommey [:glandium]]

So, the command to run on a fresh OSX install is `xcode-select --install`, which further asks whether to go forward with installing the command line tools or getting Xcode. Which sounds exactly the kind of choice mach bootstrap should give. So I'd vote for xcode-select --install in mach bootstrap.

Comment 2

[Mike Hommey [:glandium]]

Attachment: Bug 1379426 - Simplify Xcode/command line tools setup on macOS.

code hasn't been necessary to build Firefox for a while. Command line
tools are enough, but if Xcode is available, that's also sufficient.

Mach bootstrap, however, for historic reasons, mostly, ends up wanting
to install both, which is a waste of time and disk space.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8b3f022bd588
Simplify Xcode/command line tools setup on macOS. r=firefox-build-system-reviewers,mhentges

Comment 4

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/8b3f022bd588