Open
Bug 1423235
Opened 8 years ago
Updated 3 years ago
[macOS] ./mach bootstrap should not require xcode for artifact builds
Categories
(Firefox Build System :: Bootstrap Configuration, enhancement)
Firefox Build System
Bootstrap Configuration
Tracking
(firefox59 affected)
NEW
Tracking | Status | |
---|---|---|
firefox59 | --- | affected |
People
(Reporter: Gijs, Unassigned)
References
Details
For artifact builds, xcode shouldn't really be necessary. The only thing we should really need that's not already there would be `make`, and we could install that (and/or anything else we might need that I've forgotten about) via homebrew instead. That would save people downloading the several-gb gigantic gargoyle that is xcode + commandline tools.
Xref bug 1379426, which AIUI would maybe install less stuff but still end up installing xcode. This isn't a strict dupe, though it's possible that just fixing bug 1379426 may be a better way to resolve this if "install stuff via homebrew" turns out to be a wild goose chase.
Comment 1•8 years ago
|
||
I ran into this during the Build&Go onboarding session today :/
Off the top of my head, `make` is what I know we need from the Xcode Commandline tools. There may be other lingering tools though. Nuking the dependency on Xcode completely would be *really* nice if we can get replacement binaries from Homebrew/MacPorts. We already require one of these package managers. So no big deal leaning on it more.
Comment 2•8 years ago
|
||
Haven't we had problems with homebrew or macports clang in the past?
Reporter | ||
Comment 3•8 years ago
|
||
(In reply to Mike Hommey [:glandium] from comment #2)
> Haven't we had problems with homebrew or macports clang in the past?
Well, this report was originally specifically about artifact builds, for which we should not need clang, I think? Do we use clang even for artifact builds? That would surprise me, though I guess I've been here long enough that I ought not to be surprised anymore...
Updated•7 years ago
|
Product: Core → Firefox Build System
Updated•7 years ago
|
Component: General → Bootstrap Configuration
Updated•3 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•