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

Issue 62953015: Add an untrusted NaCl build target for jsoncpp. (Closed)

Created:
7 years, 1 month ago by Peter Lundblad
Modified:
7 years, 1 month ago
CC:
chromium-reviews, David Tseng, dmazzoni, bradnelson, noelallen1
Base URL:
https://chromium.googlesource.com/chromium/src.git@lkgr
Visibility:
Public.

Description

Patch Set 1 #

Patch Set 2 : Add -fstrict-aliasing instead of disabling -Werror #

Patch Set 3 : Remove target from all.gyp #

Unified diffs Side-by-side diffs Delta from patch set Stats (+76 lines, -31 lines) Patch
M third_party/jsoncpp/README.chromium View 1 chunk +1 line, -1 line 0 comments Download
M third_party/jsoncpp/jsoncpp.gyp View 1 chunk +3 lines, -30 lines 0 comments Download
A third_party/jsoncpp/jsoncpp.gypi View 1 chunk +38 lines, -0 lines 0 comments Download
A third_party/jsoncpp/jsoncpp_untrusted.gyp View 1 1 chunk +34 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
Peter Lundblad
Hi, I am adding a thirdparty library that we use in the chromevox extension. It ...
7 years, 1 month ago (2013-11-08 20:51:13 UTC) #1
Peter Lundblad
Adding NaCl people to CC.
7 years, 1 month ago (2013-11-08 20:56:14 UTC) #2
Ronghua Wu (Left Chromium)
lgtm
7 years, 1 month ago (2013-11-08 22:18:39 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/plundblad@chromium.org/62953015/370001
7 years, 1 month ago (2013-11-08 22:23:30 UTC) #4
commit-bot: I haz the power
7 years, 1 month ago (2013-11-09 01:46:30 UTC) #5
Message was sent while issue was closed.
Change committed as 234073

Powered by Google App Engine
This is Rietveld 408576698