Bug 145211 - [GTK] Bump GCC requirements to 4.9.0
Summary: [GTK] Bump GCC requirements to 4.9.0
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: Gtk
Depends on:
Blocks: 98932
  Show dependency treegraph
 
Reported: 2015-05-20 08:57 PDT by Carlos Garcia Campos
Modified: 2015-05-24 23:55 PDT (History)
5 users (show)

See Also:


Attachments
Patch (1.48 KB, patch)
2015-05-20 08:58 PDT, Carlos Garcia Campos
no flags Details | Formatted Diff | Diff
Patch (1.48 KB, patch)
2015-05-20 09:29 PDT, Carlos Garcia Campos
zan: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Carlos Garcia Campos 2015-05-20 08:57:09 PDT
It's required to build with IndexedDB support when using GCC, clang works just fine. All the bots have already been upgraded.
Comment 1 Carlos Garcia Campos 2015-05-20 08:58:43 PDT
Created attachment 253440 [details]
Patch
Comment 2 Zan Dobersek 2015-05-20 09:29:04 PDT
EWS machines apparently haven't been updated.
Comment 3 Carlos Garcia Campos 2015-05-20 09:29:30 PDT
Created attachment 253441 [details]
Patch
Comment 4 Carlos Garcia Campos 2015-05-20 09:32:49 PDT
(In reply to comment #2)
> EWS machines apparently haven't been updated.

They were, see green boxes in bug #98932, but I think CMake stupid cache still had the old compiler version somewhere.
Comment 5 Zan Dobersek 2015-05-20 09:42:29 PDT
Comment on attachment 253441 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=253441&action=review

> Source/cmake/OptionsGTK.cmake:10
> +# IndexedDB support requires GGC 4.9, see https://bugs.webkit.org/show_bug.cgi?id=98932.

GCC*.
Comment 6 Carlos Garcia Campos 2015-05-20 09:45:53 PDT
(In reply to comment #5)
> Comment on attachment 253441 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=253441&action=review
> 
> > Source/cmake/OptionsGTK.cmake:10
> > +# IndexedDB support requires GGC 4.9, see https://bugs.webkit.org/show_bug.cgi?id=98932.
> 
> GCC*.

Oops. Thanks for the review, I'll wait for EWS anyway, and even more important for Martin's approval since he was not convinced yet to bump requirements.
Comment 7 Carlos Garcia Campos 2015-05-21 23:07:17 PDT
Martin?
Comment 8 Martin Robinson 2015-05-22 15:39:42 PDT
(In reply to comment #7)
> Martin?

It seems pretty simple to make the build conditional. If we need to do that we can go back and do it later, so I'm not going to block this. :)
Comment 9 Carlos Garcia Campos 2015-05-24 23:55:21 PDT
Committed r184849: <http://trac.webkit.org/changeset/184849>