mojo/public/c/system/BUILD.gn - Issue 388333003: Add GN build targets for the rest of mojo/public

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Side by Side Diff: mojo/public/c/system/BUILD.gn

Issue 388333003: Add GN build targets for the rest of mojo/public (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src

Patch Set: add more targets, and handle --exclude-libs=All appropriately Created 6 years, 5 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:

OLD	NEW
(Empty)
	1 # Copyright 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("system") {

	6 sources = [

	7 "buffer.h",

	8 "core.h",

	9 "data_pipe.h",

	10 "functions.h",

	11 "macros.h",

	12 "message_pipe.h",

	13 "system_export.h",

	14 "types.h",

	15 "../../platform/native/system_thunks.cc",

	16 "../../platform/native/system_thunks.h",

	17 ]

	18 defines = [ "MOJO_SYSTEM_IMPLEMENTATION" ]

	19

	20 if (is_android) {

	21 all_dependent_configs -= [ ":no_export_static_lib_symbols" ]
	yzshen1 2014/07/14 21:19:39 I think this should have the path to the config? I think this should have the path to the config? Chris Masone 2014/07/14 21:24:54 Good catch. I think there may also be something el Show quoted text On 2014/07/14 21:19:39, yzshen1 wrote: > I think this should have the path to the config? Good catch. I think there may also be something else wrong, though.
	22 }

	23 }

OLD	NEW