Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(420)

Issue 264963005: Fixed Linux build with glib 2.40 (Ubuntu 14.04). (Closed)

Created:
6 years, 7 months ago by Matt Giuca
Modified:
6 years, 7 months ago
CC:
chromium-reviews, cbentzel+watch_chromium.org, chrome-apps-syd-reviews_chromium.org, haltonhuo, szym, freedom
Visibility:
Public.

Description

Fixed Linux build with glib 2.40 (Ubuntu 14.04). In proxy_config_service_linux, disabled glib deprecation warnings. The function g_settings_list_schemas got deprecated in 2.40, but its replacement is not available in older glib versions (such as the one the build bots currently use, 2.32), so for now, disable the warnings. BUG=351911 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=268417

Patch Set 1 #

Patch Set 2 : Revert SVN 267454 (this is an alternative solution). #

Total comments: 3

Patch Set 3 : Added TODO to remove the suppression when we drop support for Ubuntu 13.10. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+9 lines, -3 lines) Patch
M net/proxy/proxy_config_service_linux.cc View 1 2 2 chunks +9 lines, -3 lines 0 comments Download

Messages

Total messages: 10 (0 generated)
Matt Giuca
This is an alternative to https://codereview.chromium.org/256803006/. I don't think it's appropriate to conditionally use the ...
6 years, 7 months ago (2014-05-02 00:24:45 UTC) #1
eroman
lgtm
6 years, 7 months ago (2014-05-02 00:58:15 UTC) #2
Matt Giuca
phajdan.jr: Ping, did you have any thoughts? (I don't need OWNERS approval from you, just ...
6 years, 7 months ago (2014-05-02 22:57:55 UTC) #3
Matt Giuca
Turns out this issue was already fixed for Clang builds: https://codereview.chromium.org/267623002 But I think that ...
6 years, 7 months ago (2014-05-04 23:39:04 UTC) #4
Paweł Hajdan Jr.
LGTM with a comment, thanks! https://codereview.chromium.org/264963005/diff/20001/net/proxy/proxy_config_service_linux.cc File net/proxy/proxy_config_service_linux.cc (right): https://codereview.chromium.org/264963005/diff/20001/net/proxy/proxy_config_service_linux.cc#newcode11 net/proxy/proxy_config_service_linux.cc:11: #define GLIB_DISABLE_DEPRECATION_WARNINGS Please add ...
6 years, 7 months ago (2014-05-05 11:25:09 UTC) #5
Matt Giuca
Thanks. I just tested on GCC and it is indeed still an issue. I'll commit ...
6 years, 7 months ago (2014-05-05 12:20:26 UTC) #6
Matt Giuca
https://codereview.chromium.org/264963005/diff/20001/net/proxy/proxy_config_service_linux.cc File net/proxy/proxy_config_service_linux.cc (right): https://codereview.chromium.org/264963005/diff/20001/net/proxy/proxy_config_service_linux.cc#newcode11 net/proxy/proxy_config_service_linux.cc:11: #define GLIB_DISABLE_DEPRECATION_WARNINGS On 2014/05/05 12:20:26, Matt Giuca wrote: > ...
6 years, 7 months ago (2014-05-05 23:45:23 UTC) #7
Matt Giuca
The CQ bit was checked by mgiuca@chromium.org
6 years, 7 months ago (2014-05-05 23:45:28 UTC) #8
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/mgiuca@chromium.org/264963005/40001
6 years, 7 months ago (2014-05-05 23:46:51 UTC) #9
commit-bot: I haz the power
6 years, 7 months ago (2014-05-06 04:17:55 UTC) #10
Message was sent while issue was closed.
Change committed as 268417

Powered by Google App Engine
This is Rietveld 408576698