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

Issue 10777011: Remove logic for using system libjpeg from libjpeg_turbo gyp file (Closed)

Created:
8 years, 5 months ago by Steve Block
Modified:
8 years, 5 months ago
CC:
chromium-reviews, Michael Moss
Visibility:
Public.

Description

Remove logic for using system libjpeg from libjpeg_turbo gyp file This is part of simplifying gyp logic for libjpeg. See http://codereview.chromium.org/10696082. Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=147428

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+258 lines, -278 lines) Patch
M third_party/libjpeg_turbo/libjpeg.gyp View 2 chunks +258 lines, -278 lines 1 comment Download

Messages

Total messages: 3 (0 generated)
Steve Block
http://codereview.chromium.org/10777011/diff/1/third_party/libjpeg_turbo/libjpeg.gyp File third_party/libjpeg_turbo/libjpeg.gyp (right): http://codereview.chromium.org/10777011/diff/1/third_party/libjpeg_turbo/libjpeg.gyp#newcode18 third_party/libjpeg_turbo/libjpeg.gyp:18: 'targets': [ This is just an indentation change.
8 years, 5 months ago (2012-07-13 14:37:47 UTC) #1
Steve Block
Removing Evan as he no longer works on Chromium. This change is the result of ...
8 years, 5 months ago (2012-07-17 13:49:21 UTC) #2
Hironori Bono
8 years, 5 months ago (2012-07-19 00:29:05 UTC) #3
LGTM. Thanks for cleaning up this GYP file. :)

Regards,

Hironori Bono

Powered by Google App Engine
This is Rietveld 408576698