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

Side by Side Diff: third_party/libwebm/BUILD.gn

Issue 266893002: Add GN targets for third_party/libwebm (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « BUILD.gn ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
1 # Copyright (c) 2014 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file.
4
5 source_set("libwebm") {
6 sources = [
7 "source/mkvmuxer.cpp",
8 "source/mkvmuxerutil.cpp",
9 "source/mkvwriter.cpp",
10 ]
11
12 # mkvmuxerutil.cpp redefines the _CRT_RAND_S macro. This is the only thing
13 # that blocks this target from using Chromium compiler settings.
14 configs -= [ "//build/config/compiler:chromium_code" ]
15 configs += [ "//build/config/compiler:no_chromium_code" ]
16
17 # TODO: Is the "libwebm_nacl" target used anywhere? If so, implement it in GN
brettw 2014/05/05 17:43:05 I think you can remove this comment. With GN toolc
ckocagil 2014/05/06 04:57:45 I don't really understand. Doesn't nacl have its o
brettw 2014/05/14 16:40:14 In GN you don't need a separate target to compile
18 # after GN gets nacl target capability.
19 }
OLDNEW
« no previous file with comments | « BUILD.gn ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698