1509757 - --with-system-nspr: drop GNU regex extension when using sed

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[Jan Beich]

FreeBSD sed uses BSD regex which is going to desupport escaping ordinary characters. Previously, \S was treated as S but due to * it was elided. Now that sed aborts the build fails because ac_configure_args ends up empty:

In file included from objdir/dom/workers/Unified_cpp_dom_workers0.cpp:11:
dom/workers/ChromeWorkerScope.cpp:56:10: error: use of undeclared identifier 'JS_InitCTypesClass'
    if (!JS_InitCTypesClass(aCx, aGlobal) ||
         ^
dom/workers/ChromeWorkerScope.cpp:61:18: error: unknown type name 'JSCTypesCallbacks'
    static const JSCTypesCallbacks callbacks = {
                 ^

http://package18.nyi.freebsd.org/data/headamd64PR229925-default/2018-08-03_22h32m51s/logs/errors/firefox-esr-60.1.0_4,1.log

Comment 1

[Jan Beich]

Attachment: v1

Note, extra [ ] are for autoconf escaping.

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

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/79b57272e805
Don't rely on GNU regex extension for character classes. r=froydnj

Comment 3

[Cristina Coroiu [:ccoroiu]]

https://hg.mozilla.org/mozilla-central/rev/79b57272e805

Comment 4

[Jan Beich]

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

[Beta/Release Uplift Approval Request]

Feature/Bug causing the regression: Bug 975011

User impact if declined: Broken build on FreeBSD 13.0. Can be worked around via sed -> gsed alias/symlink but this can break other software.

Is this code covered by automated tests?: Yes

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): --with-system-nspr is NPOTB. Otherwise, can only break build.

String changes made/needed: 

[ESR Uplift Approval Request]

If this is not a sec:{high,crit} bug, please state case for ESR consideration: - Facilitate building projects based on ESR e.g., Tor Browser
- Allow building Firefox ESR without downstream patches e.g., for more vanilla experience

User impact if declined: Broken build on FreeBSD 13.0. Can be worked around via sed -> gsed alias/symlink but this can break other software.

Fix Landed on Version: 65

Risk to taking this patch: Low

Why is the change risky/not risky? (and alternatives if risky): --with-system-nspr is NPOTB. Otherwise, can only break build.

String or UUID changes made by this patch:

Comment 5

[Liz Henry (:lizzard)]

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

Support for downstream builds, OK to uplift to beta and ESR.

Comment 6

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/releases/mozilla-beta/rev/460fc055455e

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-esr60/rev/fbba8450659f