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

Issue 17408002: remove double including of libpng decoder (Closed)

Created:
7 years, 6 months ago by Zach Reizner
Modified:
7 years, 6 months ago
Reviewers:
bungeman-skia
CC:
skia-review_googlegroups.com
Visibility:
Public.

Description

remove double including of libpng decoder

Patch Set 1 #

Patch Set 2 : add force linking to gm #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -6 lines) Patch
M gm/gmmain.cpp View 1 2 chunks +3 lines, -0 lines 0 comments Download
M gyp/images.gyp View 2 chunks +0 lines, -6 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
Zach Reizner
Turns out I need this patch to build libskia.so, even without building lua.
7 years, 6 months ago (2013-06-18 15:23:21 UTC) #1
bungeman-skia
On 2013/06/18 15:23:21, Zach Reizner wrote: > Turns out I need this patch to build ...
7 years, 6 months ago (2013-06-18 15:43:27 UTC) #2
Zach Reizner
7 years, 6 months ago (2013-06-18 18:14:51 UTC) #3
bungeman-skia
Patch Set 2 looks even better. lgtm
7 years, 6 months ago (2013-06-18 19:54:12 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://skia-tree-status.appspot.com/cq/zachr@google.com/17408002/5001
7 years, 6 months ago (2013-06-18 19:57:44 UTC) #5
commit-bot: I haz the power
7 years, 6 months ago (2013-06-18 20:09:29 UTC) #6
Failed to apply the patch.
Sending        gm/gmmain.cpp
Sending        gyp/images.gyp
Transmitting file data ..svn: Commit failed (details follow):
svn: Commit failed unexpectedly --- please try again later (1)

Powered by Google App Engine
This is Rietveld 408576698