Closed Bug 1670807 Opened 4 years ago Closed 4 years ago

Bundled autoconf fails to find GNU m4 on BSD systems

Categories

(Firefox Build System :: General: Unsupported Platforms, defect)

Unspecified
FreeBSD
defect

Tracking

(firefox-esr78 unaffected, firefox81 unaffected, firefox82 wontfix, firefox83 wontfix, firefox84 fixed)

RESOLVED FIXED
84 Branch
Tracking Status
firefox-esr78 --- unaffected
firefox81 --- unaffected
firefox82 --- wontfix
firefox83 --- wontfix
firefox84 --- fixed

People

(Reporter: jbeich, Assigned: jbeich, NeedInfo)

References

(Regression)

Details

(Keywords: regression)

Attachments

(2 files, 1 obsolete file)

BSD m4 doesn't support --version unlike GNU m4 from devel/m4 installed as gm4. However, moz.configure fails to find m4 unless export M4=gm4.

$ c++ --version
FreeBSD clang version 11.0.0 (git@github.com:llvm/llvm-project.git llvmorg-11.0.0-rc5-0-g60a25202a7d)
Target: x86_64-unknown-freebsd13.0
Thread model: posix
InstalledDir: /usr/bin

$ ./mach bootstrap
$ ./mach build
[...]
checking for m4... /usr/bin/m4
Refreshing old-configure with build/autoconf/autoconf.sh
Autoconf requires GNU m4 1.1 or later
ERROR: autoconf exited with return code 1
*** Fix above errors and then restart with\
               "./mach build"
gmake: *** [client.mk:115: configure] Error 1
Attached patch fix (obsolete) — Splinter Review
Attached patch fix, v2Splinter Review

Oops, wrong order. Look for gm4 before m4.

Attachment #9181127 - Attachment is obsolete: true

nice, that would allow me to get rid of M4=/path/to/gm4 in CONFIGURE_ENV

Assignee: nobody → sledru
Status: NEW → ASSIGNED
Assignee: sledru → jbeich
Attachment #9182924 - Attachment description: Bug 1670807 - Look for gm4 in addition to m4. → Bug 1670807 - Look for gm4 in addition to m4
Pushed by sledru@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/f38410c45877
Look for gm4 in addition to m4 r=firefox-build-system-reviewers,rstewart
Pushed by sledru@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7a15a7dfb7a2
Look for gm4 in addition to m4 r=firefox-build-system-reviewers,rstewart
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 84 Branch
Has Regression Range: --- → yes
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: