Opened 11 months ago
Closed 11 months ago
#29179 closed defect (fixed)
Tor Browser nightly fails to build on macOS
Reported by: | boklm | Owned by: | tbb-team |
---|---|---|---|
Priority: | Immediate | Milestone: | |
Component: | Applications/Tor Browser | Version: | |
Severity: | Normal | Keywords: | tbb-rbm, TorBrowserTeam201901R |
Cc: | mcs, brade, pospeselr | Actual Points: | |
Parent ID: | Points: | ||
Reviewer: | Sponsor: |
Description
The latest Tor Browser nightly failed to build on macOS in the firefox part:
17:55.18 /var/tmp/dist/macosx-toolchain/clang/bin/clang++ -target x86_64-apple-darwin11 -B /var/tmp/dist/macosx-toolchain/cctools/bin -isysroot /var/tmp/dist/macosx-toolchain/SDK -Werror=format -Werror=format-security -fstack-protector-strong -D_FORTIFY_SOURCE=2 -std=gnu++14 -o nsDragService.o -c -fvisibility=hidden -fvisibility-inlines-hidden -DNDEBUG=1 -DTRIMMED=1 -DOS_POSIX=1 -DOS_MACOSX=1 -DSTATIC_EXPORTABLE_JS_API -DMOZ_HAS_MOZGLUE -DMOZILLA_INTERNAL_API -DIMPL_LIBXUL -I/var/tmp/build/firefox-394ffba812d6/widget/cocoa -I/var/tmp/build/firefox-394ffba812d6/obj-macos/widget/cocoa -I/var/tmp/build/firefox-394ffba812d6/obj-macos/ipc/ipdl/_ipdlheaders -I/var/tmp/build/firefox-394ffba812d6/ipc/chromium/src -I/var/tmp/build/firefox-394ffba812d6/ipc/glue -I/var/tmp/build/firefox-394ffba812d6/layout/forms -I/var/tmp/build/firefox-394ffba812d6/layout/generic -I/var/tmp/build/firefox-394ffba812d6/layout/style -I/var/tmp/build/firefox-394ffba812d6/layout/xul -I/var/tmp/build/firefox-394ffba812d6/widget -I/var/tmp/build/firefox-394ffba812d6/widget/headless -I/var/tmp/build/firefox-394ffba812d6/obj-macos/dist/include -I/var/tmp/build/firefox-394ffba812d6/obj-macos/dist/include/nspr -I/var/tmp/build/firefox-394ffba812d6/obj-macos/dist/include/nss -fPIC -DMOZILLA_CLIENT -include /var/tmp/build/firefox-394ffba812d6/obj-macos/mozilla-config.h -Qunused-arguments -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=2 -Qunused-arguments -Wall -Wempty-body -Wignored-qualifiers -Woverloaded-virtual -Wpointer-arith -Wsign-compare -Wtype-limits -Wunreachable-code -Wunreachable-code-return -Wwrite-strings -Wno-invalid-offsetof -Wclass-varargs -Wloop-analysis -Wc++1z-compat -Wcomma -Wimplicit-fallthrough -Wstring-conversion -Wno-inline-new-delete -Wno-error=deprecated-declarations -Wno-error=array-bounds -Wformat -Wformat-security -Wno-gnu-zero-variadic-macro-arguments -Wno-unknown-warning-option -Wno-return-type-c-linkage -fno-sized-deallocation -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=2 -fno-exceptions -fno-strict-aliasing -stdlib=libc++ -fno-rtti -ffunction-sections -fdata-sections -fno-exceptions -fno-math-errno -pthread -pipe -g -O3 -fomit-frame-pointer -MD -MP -MF .deps/nsDragService.o.pp -x objective-c++ -fobjc-exceptions /var/tmp/build/firefox-394ffba812d6/widget/cocoa/nsDragService.mm 17:58.32 make[4]: Entering directory '/var/tmp/build/firefox-394ffba812d6/obj-macos/widget/headless' 17:58.32 mkdir -p '.deps/' 17:58.33 /var/tmp/dist/macosx-toolchain/clang/bin/clang++ -target x86_64-apple-darwin11 -B /var/tmp/dist/macosx-toolchain/cctools/bin -isysroot /var/tmp/dist/macosx-toolchain/SDK -Werror=format -Werror=format-security -fstack-protector-strong -D_FORTIFY_SOURCE=2 -std=gnu++14 -o Unified_cpp_widget_headless0.o -c -fvisibility=hidden -fvisibility-inlines-hidden -DNDEBUG=1 -DTRIMMED=1 -DOS_POSIX=1 -DOS_MACOSX=1 -DSTATIC_EXPORTABLE_JS_API -DMOZ_HAS_MOZGLUE -DMOZILLA_INTERNAL_API -DIMPL_LIBXUL -I/var/tmp/build/firefox-394ffba812d6/widget/headless -I/var/tmp/build/firefox-394ffba812d6/obj-macos/widget/headless -I/var/tmp/build/firefox-394ffba812d6/widget -I/var/tmp/build/firefox-394ffba812d6/widget/headless -I/var/tmp/build/firefox-394ffba812d6/widget/cocoa -I/var/tmp/build/firefox-394ffba812d6/obj-macos/ipc/ipdl/_ipdlheaders -I/var/tmp/build/firefox-394ffba812d6/ipc/chromium/src -I/var/tmp/build/firefox-394ffba812d6/ipc/glue -I/var/tmp/build/firefox-394ffba812d6/obj-macos/dist/include -I/var/tmp/build/firefox-394ffba812d6/obj-macos/dist/include/nspr -I/var/tmp/build/firefox-394ffba812d6/obj-macos/dist/include/nss -fPIC -DMOZILLA_CLIENT -include /var/tmp/build/firefox-394ffba812d6/obj-macos/mozilla-config.h -Qunused-arguments -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=2 -Qunused-arguments -Wall -Wempty-body -Wignored-qualifiers -Woverloaded-virtual -Wpointer-arith -Wsign-compare -Wtype-limits -Wunreachable-code -Wunreachable-code-return -Wwrite-strings -Wno-invalid-offsetof -Wclass-varargs -Wloop-analysis -Wc++1z-compat -Wcomma -Wimplicit-fallthrough -Wstring-conversion -Wno-inline-new-delete -Wno-error=deprecated-declarations -Wno-error=array-bounds -Wformat -Wformat-security -Wno-gnu-zero-variadic-macro-arguments -Wno-unknown-warning-option -Wno-return-type-c-linkage -fno-sized-deallocation -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=2 -fno-exceptions -fno-strict-aliasing -stdlib=libc++ -fno-rtti -ffunction-sections -fdata-sections -fno-exceptions -fno-math-errno -pthread -pipe -g -O3 -fomit-frame-pointer -MD -MP -MF .deps/Unified_cpp_widget_headless0.o.pp /var/tmp/build/firefox-394ffba812d6/obj-macos/widget/headless/Unified_cpp_widget_headless0.cpp 18:01.64 /var/tmp/build/firefox-394ffba812d6/widget/cocoa/nsDragService.mm:330:64: error: no member named 'NodePrincipal' in 'nsIDOMNode' 18:01.64 currentTransferable->SetRequestingPrincipal(mSourceNode->NodePrincipal()); 18:01.64 ~~~~~~~~~~~ ^ 18:02.03 1 error generated. 18:02.05 /var/tmp/build/firefox-394ffba812d6/config/rules.mk:1052: recipe for target 'nsDragService.o' failed 18:02.05 make[4]: *** [nsDragService.o] Error 1
Child Tickets
Change History (7)
comment:1 Changed 11 months ago by
Priority: | Medium → Immediate |
---|
comment:2 Changed 11 months ago by
comment:3 Changed 11 months ago by
QIing to nsINode
should fix that. I am currently testing a proposed fix
comment:4 Changed 11 months ago by
I.e.
- currentTransferable->SetRequestingPrincipal(mSourceNode->NodePrincipal()); + nsCOMPtr<nsINode> node = do_QueryInterface(mSourceNode); + currentTransferable->SetRequestingPrincipal(node->NodePrincipal());
comment:5 Changed 11 months ago by
Cc: | mcs brade pospeselr added |
---|---|
Keywords: | TorBrowserTeam201901R added; TorBrowserTeam201901 removed |
Status: | new → needs_review |
Okay, this took longer than anticipated. There was more to actually fix. :( I pushed two fixup commits to tor-browser-60.5.0esr-8.5-1
(https://gitweb.torproject.org/tor-browser.git/commit/?h=tor-browser-60.5.0esr-8.5-1&id=b6487d517d9982332f3bfad1f14b6e5ed9f77c53 and https://gitweb.torproject.org/tor-browser.git/commit/?h=tor-browser-60.5.0esr-8.5-1&id=6dde994c6580634bd3cb9809c2eb0e02499e65b7). Let me know whether they look good to you.
comment:6 Changed 11 months ago by
r=mcs
Both of the fixup commits look good to me. I did not try to build with them, but I know that is happening as part of the 8.5a7 release process.
comment:7 Changed 11 months ago by
Resolution: | → fixed |
---|---|
Status: | needs_review → closed |
Thanks, closing. pospeselr: let me know if you are not happy with the fixups.
That's fallout from #29082, so we should fix that somehow before starting the alpha build.