Bug 162776 - REGRESSION(r206505) [GTK] Build broken with GTK+ 3.14
Summary: REGRESSION(r206505) [GTK] Build broken with GTK+ 3.14
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks: 162603
  Show dependency treegraph
 
Reported: 2016-09-30 02:21 PDT by Carlos Alberto Lopez Perez
Modified: 2016-10-26 00:50 PDT (History)
3 users (show)

See Also:


Attachments
Patch (2.64 KB, patch)
2016-10-26 00:43 PDT, Carlos Garcia Campos
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Carlos Alberto Lopez Perez 2016-09-30 02:21:24 PDT
Trying to build current trunk (r206631) of WebKitGTK+ on Debian stable (GTK+ 3.14.5) without the internal jhbuild results in this:



[.... compiler parameters .... ] -o Source/WebKit2/CMakeFiles/WebKit2.dir/UIProcess/gtk/WebContextMenuProxyGtk.cpp.o -c ../../Source/WebKit2/UIProcess/gtk/WebContextMenuProxyGtk.cpp
../../Source/WebKit2/UIProcess/gtk/WebContextMenuProxyGtk.cpp:69:93: error: use of undeclared identifier 'gtk_widget_get_action_group'; did you mean 'gtk_widget_set_margin_top'?
    g_action_map_add_action(((((GActionMap*) g_type_check_instance_cast ((GTypeInstance*) ((gtk_widget_get_action_group(((((GtkWidget*) g_type_check_instance_cast ((GTypeInstance*) ((m_menu)), ((gtk_widget_get_type ())))))), gContextMenuItemGroup))), ((g_action_map_get_type ())))))), action);
                                                                                            ^~~~~~~~~~~~~~~~~~~~~~~~~~~
                                                                                            gtk_widget_set_margin_top
/usr/include/gtk-3.0/gtk/gtkwidget.h:1115:6: note: 'gtk_widget_set_margin_top' declared here
void gtk_widget_set_margin_top (GtkWidget *widget,
     ^
../../Source/WebKit2/UIProcess/gtk/WebContextMenuProxyGtk.cpp:69:226: error: cannot initialize a parameter of type 'gint' (aka 'int') with an lvalue of type 'const char *'
    g_action_map_add_action(((((GActionMap*) g_type_check_instance_cast ((GTypeInstance*) ((gtk_widget_get_action_group(((((GtkWidget*) g_type_check_instance_cast ((GTypeInstance*) ((m_menu)), ((gtk_widget_get_type ())))))), gContextMenuItemGroup))), ((g_action_map_get_type ())))))), action);
                                                                                                                                                                                                                                 ^~~~~~~~~~~~~~~~~~~~~
/usr/include/gtk-3.0/gtk/gtkwidget.h:1116:45: note: passing argument to parameter 'margin' here
                                       gint margin);
                                            ^
2 errors generated.
Comment 1 Carlos Garcia Campos 2016-10-03 04:03:00 PDT
I'm sorry about this, I checked the Since tag of gtk_widget_set_action_group() and assumed it was the same for the getter. I guess we can simply save the action group somewhere to fix this instead of querying the widget.
Comment 2 Carlos Garcia Campos 2016-10-26 00:43:46 PDT
Created attachment 292884 [details]
Patch
Comment 3 Carlos Garcia Campos 2016-10-26 00:48:14 PDT
Committed r207870: <http://trac.webkit.org/changeset/207870>