Bug 146181 - [EFL][GTK] Define GLIB_VERSION_MIN_REQUIRED and require glib 2.36 for GTK
Summary: [EFL][GTK] Define GLIB_VERSION_MIN_REQUIRED and require glib 2.36 for GTK
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Template Framework (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Linux
: P2 Trivial
Assignee: Michael Catanzaro
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-06-20 08:27 PDT by Michael Catanzaro
Modified: 2015-06-21 02:37 PDT (History)
8 users (show)

See Also:


Attachments
Patch (1.19 KB, patch)
2015-06-20 08:29 PDT, Michael Catanzaro
no flags Details | Formatted Diff | Diff
Patch (2.48 KB, patch)
2015-06-20 08:57 PDT, Michael Catanzaro
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Catanzaro 2015-06-20 08:27:53 PDT
Define GLIB_VERSION_MIN_REQUIRED to silence more deprecation warnings.
Comment 1 Michael Catanzaro 2015-06-20 08:29:26 PDT
Created attachment 255291 [details]
Patch
Comment 2 Michael Catanzaro 2015-06-20 08:46:32 PDT
This fixes the following deprecation warnings that cannot otherwise be fixed without bumping the GTK port's GLib requirement from 2.34 to 2.36 (to gain access to GTask, the replacement for GSimpleAsyncResult):

[4164/4532] Building CXX object Source...oup/WebKitSoupRequestInputStream.cpp.o
../../Source/WebKit2/WebProcess/soup/WebKitSoupRequestInputStream.cpp:57:9: warning: 'g_simple_async_result_set_op_res_gssize' is deprecated [-Wdeprecated-declarations]
        g_simple_async_result_set_op_res_gssize(result, bytesRead);
        ^
/home/mcatanzaro/jhbuild/install/include/glib-2.0/gio/gsimpleasyncresult.h:82:21: note: 'g_simple_async_result_set_op_res_gssize' has been explicitly marked deprecated here
void                g_simple_async_result_set_op_res_gssize   (GSimpleAsyncResult      *simple,
                    ^
../../Source/WebKit2/WebProcess/soup/WebKitSoupRequestInputStream.cpp:60:9: warning: 'g_simple_async_result_take_error' is deprecated [-Wdeprecated-declarations]
        g_simple_async_result_take_error(result, error);
        ^
/home/mcatanzaro/jhbuild/install/include/glib-2.0/gio/gsimpleasyncresult.h:116:21: note: 'g_simple_async_result_take_error' has been explicitly marked deprecated here
void                g_simple_async_result_take_error       (GSimpleAsyncResult      *simple,
                    ^
../../Source/WebKit2/WebProcess/soup/WebKitSoupRequestInputStream.cpp:61:5: warning: 'g_simple_async_result_complete_in_idle' is deprecated [-Wdeprecated-declarations]
    g_simple_async_result_complete_in_idle(result);
    ^
/home/mcatanzaro/jhbuild/install/include/glib-2.0/gio/gsimpleasyncresult.h:106:21: note: 'g_simple_async_result_complete_in_idle' has been explicitly marked deprecated here
void                g_simple_async_result_complete_in_idle (GSimpleAsyncResult      *simple);
                    ^
../../Source/WebKit2/WebProcess/soup/WebKitSoupRequestInputStream.cpp:87:52: warning: 'g_simple_async_result_new' is deprecated [-Wdeprecated-declarations]
    GRefPtr<GSimpleAsyncResult> result = adoptGRef(g_simple_async_result_new(G_OBJECT(stream), callback, userData, reinterpret_cast<void*>(webkitSoupRequestInputStreamReadAsync)));
                                                   ^
/home/mcatanzaro/jhbuild/install/include/glib-2.0/gio/gsimpleasyncresult.h:51:21: note: 'g_simple_async_result_new' has been explicitly marked deprecated here
GSimpleAsyncResult *g_simple_async_result_new              (GObject                 *source_object,
                    ^
../../Source/WebKit2/WebProcess/soup/WebKitSoupRequestInputStream.cpp:92:9: warning: 'g_simple_async_result_set_op_res_gssize' is deprecated [-Wdeprecated-declarations]
        g_simple_async_result_set_op_res_gssize(result.get(), 0);
        ^
/home/mcatanzaro/jhbuild/install/include/glib-2.0/gio/gsimpleasyncresult.h:82:21: note: 'g_simple_async_result_set_op_res_gssize' has been explicitly marked deprecated here
void                g_simple_async_result_set_op_res_gssize   (GSimpleAsyncResult      *simple,
                    ^
../../Source/WebKit2/WebProcess/soup/WebKitSoupRequestInputStream.cpp:93:9: warning: 'g_simple_async_result_complete_in_idle' is deprecated [-Wdeprecated-declarations]
        g_simple_async_result_complete_in_idle(result.get());
        ^
/home/mcatanzaro/jhbuild/install/include/glib-2.0/gio/gsimpleasyncresult.h:106:21: note: 'g_simple_async_result_complete_in_idle' has been explicitly marked deprecated here
void                g_simple_async_result_complete_in_idle (GSimpleAsyncResult      *simple);
                    ^
../../Source/WebKit2/WebProcess/soup/WebKitSoupRequestInputStream.cpp:108:20: warning: 'g_simple_async_result_get_source_tag' is deprecated [-Wdeprecated-declarations]
    g_warn_if_fail(g_simple_async_result_get_source_tag(simpleResult) == webkitSoupRequestInputStreamReadAsync);
                   ^
/home/mcatanzaro/jhbuild/install/include/glib-2.0/glib/gmessages.h:303:18: note: expanded from macro 'g_warn_if_fail'
    if G_LIKELY (expr) ; \
                 ^
/home/mcatanzaro/jhbuild/install/include/glib-2.0/glib/gmacros.h:329:25: note: expanded from macro 'G_LIKELY'
#define G_LIKELY(expr) (expr)
                        ^
/home/mcatanzaro/jhbuild/install/include/glib-2.0/gio/gsimpleasyncresult.h:99:21: note: 'g_simple_async_result_get_source_tag' has been explicitly marked deprecated here
gpointer            g_simple_async_result_get_source_tag   (GSimpleAsyncResult      *simple);
                    ^
../../Source/WebKit2/WebProcess/soup/WebKitSoupRequestInputStream.cpp:110:12: warning: 'g_simple_async_result_get_op_res_gssize' is deprecated [-Wdeprecated-declarations]
    return g_simple_async_result_get_op_res_gssize(simpleResult);
           ^
/home/mcatanzaro/jhbuild/install/include/glib-2.0/gio/gsimpleasyncresult.h:85:21: note: 'g_simple_async_result_get_op_res_gssize' has been explicitly marked deprecated here
gssize              g_simple_async_result_get_op_res_gssize   (GSimpleAsyncResult      *simple);
                    ^
8 warnings generated.
Comment 3 Michael Catanzaro 2015-06-20 08:51:55 PDT
Um, actually, we use GTask all over the place. Let's bump our minimum to 2.36 to reflect reality.

One point to Carloz Lopez!
Comment 4 Michael Catanzaro 2015-06-20 08:57:49 PDT
Created attachment 255292 [details]
Patch
Comment 5 Michael Catanzaro 2015-06-20 09:00:16 PDT
(In reply to comment #3) 
> One point to Carloz Lopez!

(This was just a typo.)
Comment 6 Martin Robinson 2015-06-20 09:48:31 PDT
How long have we depended on something beyond our minimum dependencies?
Comment 7 Michael Catanzaro 2015-06-20 12:34:08 PDT
We've used GTask for 24 months and one day (to be unnecessarily precise). See the bugs tracked by bug #117149, which notes that "now we depend on glib 2.36." I guess at some point (maybe during the build system switch) our version requirement went backwards by accident.
Comment 8 WebKit Commit Bot 2015-06-20 16:01:31 PDT
Comment on attachment 255292 [details]
Patch

Clearing flags on attachment: 255292

Committed r185802: <http://trac.webkit.org/changeset/185802>
Comment 9 WebKit Commit Bot 2015-06-20 16:01:44 PDT
All reviewed patches have been landed.  Closing bug.
Comment 10 Carlos Garcia Campos 2015-06-21 02:37:36 PDT
(In reply to comment #6)
> How long have we depended on something beyond our minimum dependencies?

Never. We agreed on depending on 2.36.0 since WebKitGTK+ 2.0.0

https://lists.webkit.org/pipermail/webkit-gtk/2013-March/001387.html
https://bugs.webkit.org/show_bug.cgi?id=113282
http://trac.webkit.org/changeset/147547