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

Issue 1911483002: Upgrade build and clang. (Closed)

Created:
4 years, 8 months ago by dsinclair
Modified:
4 years, 8 months ago
Reviewers:
Tom Sepez, Lei Zhang, Nico
CC:
pdfium-reviews_googlegroups.com
Base URL:
https://pdfium.googlesource.com/pdfium.git@master
Target Ref:
refs/heads/master
Project:
pdfium
Visibility:
Public.

Description

Upgrade build and clang. The issue with //testing/multiprocess_func_list has been fixed in upstream //build. This Cl upgrades //build to ToT. In order to do so clang must also be updated. The Wno-undefined-var-template is only passed if the pinned clang revision isn't 266460, which is the currently pinned clang revision in chromium. This is there so that the bot that builds with trunk clang gets this flag, but pinned clang doesn't. We accidentally pickup the flag since our pinned clang is different from the Chrome version. So, we must always roll //build and clang together when changing deps or bad things may happen as there is an implicit dependency due to compile flags. BUG=pdfium:106 Committed: https://pdfium.googlesource.com/pdfium/+/6b2c4d7d709dbfca0a59ce6310c0941669d0f53e

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+11 lines, -129 lines) Patch
M DEPS View 1 chunk +2 lines, -2 lines 0 comments Download
A build_overrides/gtest.gni View 1 chunk +9 lines, -0 lines 0 comments Download
D testing/multiprocess_func_list.h View 1 chunk +0 lines, -72 lines 0 comments Download
D testing/multiprocess_func_list.cc View 1 chunk +0 lines, -55 lines 0 comments Download

Messages

Total messages: 11 (4 generated)
dsinclair
PTAL.
4 years, 8 months ago (2016-04-20 16:54:59 UTC) #2
Nico
the cl looks good, but the description is incorrect: Wno-undefined-var-template is only passed if the ...
4 years, 8 months ago (2016-04-20 16:59:25 UTC) #3
Nico
*the cl lgtm, but...
4 years, 8 months ago (2016-04-20 16:59:32 UTC) #4
Tom Sepez
Deferring to Nico.
4 years, 8 months ago (2016-04-20 17:08:32 UTC) #5
dsinclair
On 2016/04/20 16:59:32, Nico wrote: > *the cl lgtm, but... Updated description.
4 years, 8 months ago (2016-04-20 17:47:16 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1911483002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1911483002/1
4 years, 8 months ago (2016-04-20 17:47:32 UTC) #9
commit-bot: I haz the power
4 years, 8 months ago (2016-04-20 17:47:51 UTC) #11
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://pdfium.googlesource.com/pdfium/+/6b2c4d7d709dbfca0a59ce6310c0941669d0...

Powered by Google App Engine
This is Rietveld 408576698