astroidmail / homebrew-astroid

Homebrew tap for Astroid
10 stars 5 forks source link

brew install failed, but no clear error message ... #30

Open jimcheetham opened 4 years ago

jimcheetham commented 4 years ago

Trying to install the current astroid on macOS Catalina 10.15.5 19F96 x86_64

'ninja install' failed after one warning & one note, but no obvious errors.

$ brew install astroid
Updating Homebrew...
==> Installing astroid from astroidmail/astroid
==> Downloading https://webkitgtk.org/releases/webkitgtk-2.22.7.tar.xz
Already downloaded: /Users/jim/Library/Caches/Homebrew/downloads/d31bf62d4d59b6dd3e425154edb1d608eca9dd023f5dde5ed3766b2c59d0fdd4--webkitgtk-2.22.7.tar.xz
==> Downloading https://homebrew.bintray.com/bottles/libsigc%2B%2B%402-2.10.3.catalina.bottle.tar.gz
Already downloaded: /Users/jim/Library/Caches/Homebrew/downloads/0fcfbd0fd236bfd3ff596403aa2b37b520a392fb258503bd263a50e2e85492c6--libsigc++@2-2.10.3.catalina.bottle.tar.gz
==> Downloading https://homebrew.bintray.com/bottles/glibmm-2.64.2.catalina.bottle.tar.gz
Already downloaded: /Users/jim/Library/Caches/Homebrew/downloads/827a188de5e769783af6629d0dd102c7b6f2a240b7f2faa436ef3ece6e57bb03--glibmm-2.64.2.catalina.bottle.tar.gz
==> Downloading https://homebrew.bintray.com/bottles/atkmm-2.28.0_2.catalina.bottle.tar.gz
Already downloaded: /Users/jim/Library/Caches/Homebrew/downloads/d49f14261a2f1a50b2e9456d89fca0924e2dfce7fcee871ddde27ec5d39b8c12--atkmm-2.28.0_2.catalina.bottle.tar.gz
==> Downloading https://homebrew.bintray.com/bottles/cairomm-1.12.2_1.catalina.bottle.tar.gz
Already downloaded: /Users/jim/Library/Caches/Homebrew/downloads/6aff9cfb63f9f6dcf95bd21bf8b7155f48d3678886a7d1c350f0ba98e46a2fdd--cairomm-1.12.2_1.catalina.bottle.tar.gz
==> Downloading https://homebrew.bintray.com/bottles/pangomm-2.42.1.catalina.bottle.tar.gz
Already downloaded: /Users/jim/Library/Caches/Homebrew/downloads/79292e8224fe8694d650925d9cf4bc03661413e9d771a7b75ff19439505c93e6--pangomm-2.42.1.catalina.bottle.tar.gz
==> Downloading https://homebrew.bintray.com/bottles/gtkmm3-3.24.2.catalina.bottle.tar.gz
Already downloaded: /Users/jim/Library/Caches/Homebrew/downloads/657db7f75d12cb9e99b4ca87093af32c44ea9b1c1d16aa4d683398ef3218b2bb--gtkmm3-3.24.2.catalina.bottle.tar.gz
==> Downloading https://homebrew.bintray.com/bottles/protobuf-c-1.3.3_1.catalina.bottle.tar.gz
Already downloaded: /Users/jim/Library/Caches/Homebrew/downloads/cfc82d8cbb497872c4f078033748357425ad00105f5318214e1e105e4238d0ee--protobuf-c-1.3.3_1.catalina.bottle.tar.gz
==> Downloading https://github.com/astroidmail/astroid/archive/v0.14.tar.gz
Already downloaded: /Users/jim/Library/Caches/Homebrew/downloads/ce46caf645c92a1c8b98ae6a043f3e71d967208f2e85addc9b479c1c700f3ca2--astroid-0.14.tar.gz
Warning: Cannot verify integrity of ce46caf645c92a1c8b98ae6a043f3e71d967208f2e85addc9b479c1c700f3ca2--astroid-0.14.tar.gz
A checksum was not provided for this resource.
For your reference the SHA-256 is: f2642968919527008d383b4c32d4b038c7b6f785ea084dde1222a65162ebc5ca
==> Installing dependencies for astroidmail/astroid/astroid: webkitgtk, libsigc++@2, glibmm, atkmm, cairomm, pangomm, gtkmm3 and protobuf-c
==> Installing astroidmail/astroid/astroid dependency: webkitgtk
==> Patching
patching file Source/WTF/wtf/RAMSize.cpp
patching file Source/WebCore/page/scrolling/coordinatedgraphics/ScrollingStateNodeCoordinatedGraphics.cpp
patching file Source/WebCore/platform/PlatformWheelEvent.h
patching file Source/WebCore/platform/graphics/OpenGLShims.h
patching file Source/WebCore/platform/gtk/PlatformWheelEventGtk.cpp
patching file Source/WebCore/platform/gtk/ScrollAnimatorGtk.cpp
patching file Source/WTF/wtf/Optional.h
==> cmake -GNinja -DPORT=GTK -DCMAKE_BUILD_TYPE=RelWithDebInfo -DENABLE_INTROSPECTION=OFF -DENABLE_GEOLOCATION=OFF -DENABLE_OPENGL=OFF -DENABL
==> ninja install
Last 15 lines from /Users/jim/Library/Logs/Homebrew/webkitgtk/02.ninja:
In file included from ../Source/JavaScriptCore/runtime/VM.cpp:37:
In file included from ../Source/JavaScriptCore/runtime/CodeCache.h:28:
In file included from ../Source/JavaScriptCore/bytecompiler/BytecodeGenerator.h:35:
In file included from ../Source/JavaScriptCore/interpreter/Interpreter.h:35:
../Source/JavaScriptCore/bytecode/Opcode.h:105:27: warning: adding 'unsigned int' to a string does not append to the string [-Wstring-plus-int]
    return PADDING_STRING + PADDING_STRING_LENGTH - pad;
           ~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~
../Source/JavaScriptCore/bytecode/Opcode.h:105:27: note: use array indexing to silence this warning
    return PADDING_STRING + PADDING_STRING_LENGTH - pad;
                          ^
           &              [                      ]
1 warning generated.
[1309/2516] Building CXX object Source/JavaScriptCore/CMakeFiles/JavaScriptCore.dir/__/__/DerivedSources/JavaScriptCore/unified-sources/UnifiedSource126.cpp.o
[1310/2516] Building CXX object Source/JavaScriptCore/CMakeFiles/JavaScriptCore.dir/__/__/DerivedSources/JavaScriptCore/unified-sources/UnifiedSource125.cpp.o
ninja: build stopped: subcommand failed.

If reporting this issue please do so at (not Homebrew/brew or Homebrew/core):
  https://github.com/astroidmail/homebrew-astroid/issues
$ brew info astroid
astroidmail/astroid/astroid: stable 0.14, HEAD
A graphical threads-with-tags style, lightweight and fast, email client for notmuch, inspired by sup and others
http://astroidmail.github.io/
Not installed
From: https://github.com/astroidmail/homebrew-astroid/blob/master/Formula/astroid.rb
==> Dependencies
Build: cmake ✔
Required: libsass ✔, libpeas ✔, notmuch ✔, boost ✔, vte3 ✔, webkitgtk ✘, gtkmm3 ✘, gnome-icon-theme ✔, protobuf-c ✘
==> Options
--HEAD
    Install HEAD version

I tried brew install astroid --HEAD but apparently my branch was already up to date :-) and I got the same result.

jimcheetham commented 4 years ago

I looked a bit harder at all of /Users/jim/Library/Logs/Homebrew/webkitgtk/02.ninja :-

FAILED: Source/JavaScriptCore/CMakeFiles/JavaScriptCore.dir/__/__/DerivedSources/JavaScriptCore/unified-sources/UnifiedSource123.cpp.o
/usr/local/Homebrew/Library/Homebrew/shims/mac/super/clang++  -DBUILDING_GTK__=1 -DBUILDING_JavaScriptCore -DBUILDING_WITH_CMAKE=1 -DGETTEXT_PACKAGE=\"WebKit2GTK-4.0\" -DHAVE_CONFI
G_H=1 -DJSC_COMPILATION -DJSC_GLIB_API_ENABLED -DJavaScriptCore_EXPORTS -DSTATICALLY_LINKED_WITH_WTF -DWEBKITGTK_API_VERSION_STRING=\"4.0\" -IDerivedSources/ForwardingHeaders -I. -
I../Source/JavaScriptCore -I../Source/JavaScriptCore/API -I../Source/JavaScriptCore/assembler -I../Source/JavaScriptCore/b3 -I../Source/JavaScriptCore/b3/air -I../Source/JavaScript
Core/bindings -I../Source/JavaScriptCore/builtins -I../Source/JavaScriptCore/bytecode -I../Source/JavaScriptCore/bytecompiler -I../Source/JavaScriptCore/dfg -I../Source/JavaScriptC
ore/disassembler -I../Source/JavaScriptCore/disassembler/ARM64 -I../Source/JavaScriptCore/disassembler/udis86 -I../Source/JavaScriptCore/domjit -I../Source/JavaScriptCore/ftl -I../
Source/JavaScriptCore/heap -I../Source/JavaScriptCore/debugger -I../Source/JavaScriptCore/inspector -I../Source/JavaScriptCore/inspector/agents -I../Source/JavaScriptCore/inspector
/augmentable -I../Source/JavaScriptCore/inspector/remote -I../Source/JavaScriptCore/interpreter -I../Source/JavaScriptCore/jit -I../Source/JavaScriptCore/llint -I../Source/JavaScri
ptCore/parser -I../Source/JavaScriptCore/profiler -I../Source/JavaScriptCore/runtime -I../Source/JavaScriptCore/tools -I../Source/JavaScriptCore/wasm -I../Source/JavaScriptCore/was
m/js -I../Source/JavaScriptCore/yarr -IDerivedSources/JavaScriptCore -IDerivedSources/JavaScriptCore/inspector -IDerivedSources/JavaScriptCore/runtime -IDerivedSources/JavaScriptCo
re/yarr -IDerivedSources/ForwardingHeaders/JavaScriptCore/glib -IDerivedSources/JavaScriptCore/javascriptcoregtk/jsc -I../Source/JavaScriptCore/API/glib -IDerivedSources/JavaScript
Core/javascriptcoregtk -I../Source/JavaScriptCore/inspector/remote/glib -I../Source/bmalloc -IDerivedSources -I../Source/ThirdParty -isystem /usr/local/include/glib-2.0 -isystem /u
sr/local/lib/glib-2.0/include -fdiagnostics-color=always -fcolor-diagnostics -Wno-parentheses-equality -Wno-noexcept-type -Qunused-arguments -Wwrite-strings -Wundef -Wpointer-arith
 -Wmissing-format-attribute -Wformat-security -Wcast-align -Wextra -Wall -Wno-unused-lambda-capture -fno-strict-aliasing -fno-exceptions -fno-rtti -std=c++14 -O2 -g -DNDEBUG -isysr
oot /Library/Developer/CommandLineTools/SDKs/MacOSX10.15.sdk -fPIC   -ffp-contract=off -MD -MT Source/JavaScriptCore/CMakeFiles/JavaScriptCore.dir/__/__/DerivedSources/JavaScriptCo
re/unified-sources/UnifiedSource123.cpp.o -MF Source/JavaScriptCore/CMakeFiles/JavaScriptCore.dir/__/__/DerivedSources/JavaScriptCore/unified-sources/UnifiedSource123.cpp.o.d -o So
urce/JavaScriptCore/CMakeFiles/JavaScriptCore.dir/__/__/DerivedSources/JavaScriptCore/unified-sources/UnifiedSource123.cpp.o -c DerivedSources/JavaScriptCore/unified-sources/Unifie
dSource123.cpp
In file included from DerivedSources/JavaScriptCore/unified-sources/UnifiedSource123.cpp:8:
../Source/JavaScriptCore/tools/JSDollarVM.cpp:175:37: error: reference to 'Handle' is ambiguous
    bool isReachableFromOpaqueRoots(Handle<JSC::Unknown> handle, void*, SlotVisitor& visitor) override
                                    ^
/Library/Developer/CommandLineTools/SDKs/MacOSX10.15.sdk/usr/include/MacTypes.h:249:41: note: candidate found by name lookup is 'Handle'
typedef Ptr *                           Handle;
                                        ^
../Source/JavaScriptCore/heap/Handle.h:109:29: note: candidate found by name lookup is 'JSC::Handle'
template <typename T> class Handle : public HandleBase, public HandleConverter<Handle<T>, T> {
                            ^
1 error generated.
mtniehaus commented 3 years ago

Looks like the MacOS "Handle" definition conflicts with the JavaScriptCore "Handle" definition. I see that's been fixed in the Webkit source:

bool isReachableFromOpaqueRoots(JSC::Handle<JSC::Unknown> handle, void*, SlotVisitor& visitor, const char** reason) final

from a later release:

https://trac.webkit.org/browser/webkit/releases/WebKitGTK/webkit-2.30.3/Source/JavaScriptCore/tools/JSDollarVM.cpp

Maybe someone can update this?