1319405 - --with-system-nss fails to build despite being advised via AC_MSG_ERROR for Tier3 platforms

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[Jan Beich]

$ echo ac_add_options --with-system-nspr >>.mozconfig
$ echo ac_add_options --with-system-nss >>.mozconfig
$ ./mach build
[...]
Creating config.status
Reticulating splines...
Traceback (most recent call last):
  File "configure.py", line 107, in <module>
    sys.exit(main(sys.argv))
  File "configure.py", line 31, in main
    return config_status(config)
  File "configure.py", line 102, in config_status
    return config_status(args=[], **encode(sanitized_config, encoding))
  File "python/mozbuild/mozbuild/config_status.py", line 147, in config_status
    definitions = list(definitions)
  File "python/mozbuild/mozbuild/frontend/emitter.py", line 234, in emit
    objs = list(self._emit_libs_derived(contexts))
  File "python/mozbuild/mozbuild/frontend/emitter.py", line 268, in _emit_libs_derived
    self._link_libraries(context, obj, variable)
  File "python/mozbuild/mozbuild/frontend/emitter.py", line 385, in _link_libraries
    context)
mozbuild.frontend.reader.SandboxValidationError:
==============================
ERROR PROCESSING MOZBUILD FILE
==============================

The error occurred while processing the following file or one of the files it includes:

    security/manager/ssl/moz.build

The error occurred when validating the result of the execution. The reported error is:

    USE_LIBS contains "crmf", which does not match any LIBRARY_NAME in the tree.


*** Fix above errors and then restart with\
               "/usr/local/bin/gmake -f client.mk build"
gmake: *** [client.mk:379: configure] Error 1

Comment 1

[(not currently active) Ted Mielczarek]

Oh, oops, apparently I removed config/external/nss/crmf and didn't replace it with anything:
https://hg.mozilla.org/mozilla-central/rev/ee70776759bf#l3.2

Comment 2

[(not currently active) Ted Mielczarek]

Attachment: bug 1319405 - fix --with-system-nss build.

Review commit: https://reviewboard.mozilla.org/r/95176/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/95176/

Comment 3

[(not currently active) Ted Mielczarek]

Turns out we already put `-lcrmf` in `NSS_LIBS` in the `MOZ_SYSTEM_NSS` case, so the fix for this is really simple!
https://dxr.mozilla.org/mozilla-central/rev/0ddfec7126ec503b54df9c4b7c3b988906f6c882/old-configure.in#2115

Comment 4

[Michael Shal [:mshal]]

Comment on attachment 8813835 [details]
bug 1319405 - fix --with-system-nss build.

https://reviewboard.mozilla.org/r/95176/#review95398

Works for me!

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/10a01f31d432
fix --with-system-nss build. r=mshal

Comment 6

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/10a01f31d432