lkml.org 
[lkml]   [2020]   [Jan]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [GIT PULL 0/6] perf/urgent fixes
From
Date


On 1/13/20 2:55 PM, Jiri Olsa wrote:
> On Mon, Jan 13, 2020 at 01:58:59PM +0530, Ravi Bangoria wrote:
>
> SNIP
>
>> | ^~~~
>> In file included from /usr/include/glib-2.0/gobject/gobject.h:24,
>> from /usr/include/glib-2.0/gobject/gbinding.h:29,
>> from /usr/include/glib-2.0/glib-object.h:23,
>> from /usr/include/glib-2.0/gio/gioenums.h:28,
>> from /usr/include/glib-2.0/gio/giotypes.h:28,
>> from /usr/include/glib-2.0/gio/gio.h:26,
>> from /usr/include/gtk-2.0/gdk/gdkapplaunchcontext.h:30,
>> from /usr/include/gtk-2.0/gdk/gdk.h:32,
>> from /usr/include/gtk-2.0/gtk/gtk.h:32,
>> from test-gtk2.c:3:
>> /usr/include/glib-2.0/gobject/gtype.h:679:1: note: declared here
>> 679 | {
>> | ^
>> In file included from /usr/include/gtk-2.0/gtk/gtktoolitem.h:31,
>> from /usr/include/gtk-2.0/gtk/gtktoolbutton.h:30,
>> from /usr/include/gtk-2.0/gtk/gtkmenutoolbutton.h:30,
>> from /usr/include/gtk-2.0/gtk/gtk.h:126,
>> from test-gtk2.c:3:
>> /usr/include/gtk-2.0/gtk/gtktooltips.h:73:3: error: ‘GTimeVal’ is deprecated: Use 'GDateTime' instead [-Werror=deprecated-declarations]
>> 73 | GTimeVal last_popdown;
>> | ^~~~~~~~
>> In file included from /usr/include/glib-2.0/glib/galloca.h:32,
>> from /usr/include/glib-2.0/glib.h:30,
>> from /usr/include/glib-2.0/gobject/gbinding.h:28,
>> from /usr/include/glib-2.0/glib-object.h:23,
>> from /usr/include/glib-2.0/gio/gioenums.h:28,
>> from /usr/include/glib-2.0/gio/giotypes.h:28,
>> from /usr/include/glib-2.0/gio/gio.h:26,
>> from /usr/include/gtk-2.0/gdk/gdkapplaunchcontext.h:30,
>> from /usr/include/gtk-2.0/gdk/gdk.h:32,
>> from /usr/include/gtk-2.0/gtk/gtk.h:32,
>> from test-gtk2.c:3:
>> /usr/include/glib-2.0/glib/gtypes.h:551:8: note: declared here
>> 551 | struct _GTimeVal
>> | ^~~~~~~~~
>> cc1: all warnings being treated as errors
>>
>
> patch below fixes that for me.. please let me know
> if it works for you and I'll post full patch
>
> jirka
>

LGTM. You can add:

Tested-by: Ravi Bangoria <ravi.bangoria@linux.ibm.com>

\
 
 \ /
  Last update: 2020-01-13 10:44    [W:0.054 / U:5.540 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site