1496733 - ./mach bootstrap is missing a dependency on FreeBSD

Status: RESOLVED FIXED

(Firefox Build System :: Bootstrap Configuration, defect)

Description

[Jan Beich]

One of the other dependencies probably stopped depending on libXt, so we need to list it explicitly. Tested inside pristine FreeBSD 12.0 amd64 jail.

$ ./mach bootstrap
$ ./mach build
[...]
 0:05.18 configure: error: Can't find X headers (install libxt-dev (Debian/Ubuntu), libXt-devel (Fedora), or xorg-x11-libXt-devel (SuSE)).
 0:05.19 DEBUG: <truncated - see config.log for full output>
 0:05.19 DEBUG: configure: failed program was:
 0:05.19 DEBUG: #line 14886 "configure"
 0:05.19 DEBUG: #include "confdefs.h"
 0:05.19 DEBUG:
 0:05.19 DEBUG:         #include <stdio.h>
 0:05.19 DEBUG:         #include <stdlib.h>
 0:05.19 DEBUG:         #include <X11/Xlib.h>
 0:05.19 DEBUG:         #include <X11/Intrinsic.h>
 0:05.19 DEBUG:         #include <X11/extensions/XShm.h>
 0:05.19 DEBUG:
 0:05.20 DEBUG: int main() {
 0:05.20 DEBUG:
 0:05.20 DEBUG:         Display *dpy = 0;
 0:05.20 DEBUG:         if ((dpy = XOpenDisplay(NULL)) == NULL) {
 0:05.20 DEBUG:             fprintf(stderr, ": can't open %s\n", XDisplayName(NULL));
 0:05.20 DEBUG:             exit(1);
 0:05.20 DEBUG:         }
 0:05.20 DEBUG:
 0:05.20 DEBUG: ; return 0; }
 0:05.20 DEBUG: configure: error: Can't find X headers (install libxt-dev (Debian/Ubuntu), libXt-devel (Fedora), or xorg-x11-libXt-devel (SuSE)).
 0:05.20 ERROR: old-configure failed
 0:05.22 *** Fix above errors and then restart with\
 0:05.22                "./mach build"
 0:05.22 gmake: *** [client.mk:128: configure] Error 1

Comment 1

[Jan Beich]

Attachment: v1

Comment 2

[Jan Beich]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=be6c29d1e08e301ddf4bda7fd2f73a071ec536b1

Comment 3

[Jan Beich]

Comment on attachment 9014796 [details] [diff] [review]
v1

[Beta/Release Uplift Approval Request]

Feature/Bug causing the regression: None

User impact if declined: Broken build on FreeBSD after ./mach bootstrap.

Is this code covered by automated tests?: Unknown

Has the fix been verified in Nightly?: Yes

Needs manual test from QE?: No

If yes, steps to reproduce: 

List of other uplifts needed: None

Risk to taking this patch: Low

Why is the change risky/not risky? (and alternatives if risky): ./mach bootstrap isn't used by the automation. If I'm wrong then assume this can only break build in case of syntax errors (e.g., missing quote).

String changes made/needed: 

[ESR Uplift Approval Request]

If this is not a sec:{high,crit} bug, please state case for ESR consideration: Help get involved in porting Firefox to FreeBSD and build Firefox and, especially, projects based on ESR (e.g., Tor Browser, Thunderbird) outside of downstream build system in order check for regressions/differences.

User impact if declined: Broken build on FreeBSD after ./mach bootstrap.

Fix Landed on Version: 64

Risk to taking this patch: Low

Why is the change risky/not risky? (and alternatives if risky): ./mach bootstrap isn't used by the automation. If I'm wrong then assume this can only break build in case of syntax errors (e.g., missing quote).

String or UUID changes made by this patch:

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/410533a1241e
Explicitly install libXt on FreeBSD. r=Build r=froydnj

Comment 5

[Noemi Erli[:noemi_erli]]

https://hg.mozilla.org/mozilla-central/rev/410533a1241e

Comment 6

[Pascal Chevrel:pascalc (PTO until April 26)]

Comment on attachment 9014796 [details] [diff] [review]
v1

Sorry, this is too late in the Beta cycle (we build the last 63 beta on Thursday before entering RC) to uplift unplanned tier-3 patches as we need to minimize any potential regression for the final release.

Comment 7

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 9014796 [details] [diff] [review]
v1

Per IRC discussion, this patch is NPOTB and doesn't need approval for uplift. Sorry for the confusion :(

Comment 8

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-beta/rev/a3246a951d2e

Comment 9

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-esr60/rev/7bb6996ff660