|Submitter||Diff||Changes||Open Issues||Last Updated|
|Error loading review requests:|
Created attachment 8891455 [details] Build output I recently ran mach bootstrap to get the stuff needed for stylo, and now my build fails in the openvr code. See attachment for build output. This is on El Capitan, OS X 10.11.6.
:O this is an sccache bug. I found the command that was failing (in $OBJDIR/gfx/vr/openvr). Running it without sccache works. Ted, known issue? /Users/kats/.cargo/bin/sccache /usr/bin/clang++ -std=gnu++11 -o pathtools_public.o -c -fvisibility=hidden -fvisibility-inlines-hidden -DNDEBUG=1 -DTRIMMED=1 -DVR_API_PUBLIC -DPOSIX -DOSX -DSTATIC_EXPORTABLE_JS_API -DMOZ_HAS_MOZGLUE -DMOZILLA_INTERNAL_API -DIMPL_LIBXUL -I/Users/kats/zspace/mozilla-wr/gfx/vr/openvr -I/Users/kats/zspace/mozilla-wr/obj-host-opt/gfx/vr/openvr -I/Users/kats/zspace/mozilla-wr/toolkit/components/jsoncpp/include -I/Users/kats/zspace/mozilla-wr/obj-host-opt/dist/include -I/Users/kats/zspace/mozilla-wr/obj-host-opt/dist/include/nspr -I/Users/kats/zspace/mozilla-wr/obj-host-opt/dist/include/nss -fPIC -DMOZILLA_CLIENT -include /Users/kats/zspace/mozilla-wr/obj-host-opt/mozilla-config.h -MD -MP -MF .deps/pathtools_public.o.pp -Qunused-arguments -Qunused-arguments -Wall -Wc++11-compat -Wempty-body -Wignored-qualifiers -Woverloaded-virtual -Wpointer-arith -Wsign-compare -Wtype-limits -Wunreachable-code -Wwrite-strings -Wno-invalid-offsetof -Wclass-varargs -Wloop-analysis -Wc++11-compat-pedantic -Wc++14-compat -Wc++14-compat-pedantic -Wc++1z-compat -Wimplicit-fallthrough -Werror=non-literal-null-conversion -Wstring-conversion -Wno-inline-new-delete -Wno-error=deprecated-declarations -Wno-error=array-bounds -Wformat -Wno-gnu-zero-variadic-macro-arguments -Wformat-security -Wno-unknown-warning-option -Wno-return-type-c-linkage -fno-exceptions -fno-strict-aliasing -stdlib=libc++ -fno-rtti -fno-exceptions -fno-math-errno -pthread -pipe -g -g -fno-omit-frame-pointer -Werror -xobjective-c++ -Wno-error=parentheses -Wno-error=unused-variable /Users/kats/zspace/mozilla-wr/gfx/vr/openvr/src/pathtools_public.cpp
(The exact same thing happens with vrpathregistry_public.cpp in the same folder)
Summary: OS X build failure in gfx/vr/openvr/ code → OS X build failure in gfx/vr/openvr/ code (error: unknown type name 'NSString') caused by sccache
Ted says glandium has a patch to fix this
See Also: → https://github.com/mozilla/sccache/pull/170
Assignee: nobody → mh+mozilla
Marking 55/56 affected since this may be something we can fix during 56 beta (for the sdk update)
status-firefox55: --- → affected
status-firefox56: --- → affected
What's the ETA on this? We have a separate security issue that needs this fix as a prerequisite.
I merged glandium's sccache PR just now. I'll get an updated sccache landed in-tree and land the SDK update. Sorry for the delay!
Comment on attachment 8897954 [details] bug 1385386 - update sccache to revision 210bc791513ed8362b17a492a1f35a621cca64ff. https://reviewboard.mozilla.org/r/169262/#review174686
Attachment #8897954 - Flags: review+
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/da78dd4e29fa update sccache to revision 210bc791513ed8362b17a492a1f35a621cca64ff. r=glandium
Status: NEW → RESOLVED
Last Resolved: 11 months ago
status-firefox57: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
Is this something we can uplift for beta 56?
The fix here was just to update to a newer version of sccache, so it should be fine to uplift that to beta. Is it needed to unblock some other work? It's not super useful in and of itself.
Automation doesn't use sccache on beta, so it wouldn't do anything for automation.
There was a sec issue dependent on this but I'll have to go back and dig around to find it - I forgot to mark this as a blocker.
(In reply to Liz Henry (:lizzard) (needinfo? me) from comment #14) > There was a sec issue dependent on this but I'll have to go back and dig > around to find it - I forgot to mark this as a blocker. Ah yes, there were some bugs that wanted the SDK update, and this was blocking that on nightly. If we're not using sccache for beta builds then this won't actually need uplift.
You need to log in before you can comment on or make changes to this bug.