RESOLVED FIXED 230480
REGRESSION(r282735): build failing on gtk/gcc 
https://bugs.webkit.org/show_bug.cgi?id=230480
Summary REGRESSION(r282735): build failing on gtk/gcc
Jim Mason
Reported 2021-09-20 07:37:17 PDT
Building with gcc 10, gtk 3. Still breaking as of r282742. Reversing the commit r282735 makes the build complete normally. Build failure is as follows: In file included from /build/rocketman/components/desktop/webkitgtk4-dev/webkit/Source/WebCore/rendering/LegacyInlineFlowBox.cpp:44, from WebCore/DerivedSources/unified-sources/UnifiedSource-043dd90b-4.cpp:7: /build/rocketman/components/desktop/webkitgtk4-dev/webkit/Source/WebCore/rendering/TextBoxPainter.h:50:45: error: 'InlineContent' in namespace 'WebCore::LayoutIntegration' does not name a type 50 | TextBoxPainter(const LayoutIntegration::InlineContent&, const Layout::Run&, PaintInfo&, const LayoutPoint& paintOffset); | ^~~~~~~~~~~~~ /build/rocketman/components/desktop/webkitgtk4-dev/webkit/Source/WebCore/rendering/TextBoxPainter.h:50:67: error: 'Layout' does not name a type; did you mean 'TextLayout'? 50 | TextBoxPainter(const LayoutIntegration::InlineContent&, const Layout::Run&, PaintInfo&, const LayoutPoint& paintOffset); | ^~~~~~ | TextLayout /build/rocketman/components/desktop/webkitgtk4-dev/webkit/Source/WebCore/rendering/TextBoxPainter.h:50:78: error: expected unqualified-id before '&' token 50 | TextBoxPainter(const LayoutIntegration::InlineContent&, const Layout::Run&, PaintInfo&, const LayoutPoint& paintOffset); | ^ /build/rocketman/components/desktop/webkitgtk4-dev/webkit/Source/WebCore/rendering/TextBoxPainter.h:50:78: error: expected ')' before '&' token 50 | TextBoxPainter(const LayoutIntegration::InlineContent&, const Layout::Run&, PaintInfo&, const LayoutPoint& paintOffset); | ~ ^ | ) /build/rocketman/components/desktop/webkitgtk4-dev/webkit/Source/WebCore/rendering/TextBoxPainter.h:50:78: error: constructors may not be ref-qualified /build/rocketman/components/desktop/webkitgtk4-dev/webkit/Source/WebCore/rendering/TextBoxPainter.h:50:81: warning: ISO C++ forbids declaration of 'PaintInfo' with no type [-fpermissive] 50 | TextBoxPainter(const LayoutIntegration::InlineContent&, const Layout::Run&, PaintInfo&, const LayoutPoint& paintOffset); | ^~~~~~~~~ /build/rocketman/components/desktop/webkitgtk4-dev/webkit/Source/WebCore/rendering/TextBoxPainter.h:50:81: error: expected ';' at end of member declaration 50 | TextBoxPainter(const LayoutIntegration::InlineContent&, const Layout::Run&, PaintInfo&, const LayoutPoint& paintOffset); | ^~~~~~~~~ | ; /build/rocketman/components/desktop/webkitgtk4-dev/webkit/Source/WebCore/rendering/TextBoxPainter.h:50:91: error: expected unqualified-id before ',' token 50 | TextBoxPainter(const LayoutIntegration::InlineContent&, const Layout::Run&, PaintInfo&, const LayoutPoint& paintOffset); | ^ /build/rocketman/components/desktop/webkitgtk4-dev/webkit/Source/WebCore/rendering/TextBoxPainter.h:58:58: error: 'PaintInfo' is not a type 58 | TextBoxPainter(LayoutIntegration::TextRunIterator&&, PaintInfo&, const LayoutPoint& paintOffset); | ^~~~~~~~~ /build/rocketman/components/desktop/webkitgtk4-dev/webkit/Source/WebCore/rendering/TextBoxPainter.h:92:5: error: 'PaintInfo' does not name a type 92 | PaintInfo& m_paintInfo; | ^~~~~~~~~ [2758/5099] Building CXX object Source...sources/UnifiedSource-043dd90b-6.cpp.o ninja: build stopped: subcommand failed. gmake: *** [/build/rocketman/make-rules/justmake.mk:65: /build/rocketman/components/desktop/webkitgtk4-dev/build/amd64/.built] Error 1
Attachments
Add attachment proposed patch, testcase, etc.
Jim Mason
Comment 1 2021-09-22 08:45:29 PDT
This has been fixed by r282788 <ross.kirsling@sony.com>: "Unreviewed build fix for WinCairo with ENABLE_EXPERIMENTAL_FEATURES off" Thank you, Ross!
Note You need to log in before you can comment on or make changes to this bug.