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

Unified Diff: third_party/libjingle_xmpp/xmllite/BUILD.gn

Issue 2443903004: Add xmllite and xmpp sources to third_party/ (Closed)
Patch Set: Fix GN and sort includes Created 4 years 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 side-by-side diff with in-line comments
Download patch
Index: third_party/libjingle_xmpp/xmllite/BUILD.gn
diff --git a/third_party/libjingle_xmpp/xmllite/BUILD.gn b/third_party/libjingle_xmpp/xmllite/BUILD.gn
new file mode 100644
index 0000000000000000000000000000000000000000..be1bf01309db35381f6ed6828c66ed1270db0217
--- /dev/null
+++ b/third_party/libjingle_xmpp/xmllite/BUILD.gn
@@ -0,0 +1,86 @@
+# Copyright 2016 The Chromium Authors. All rights reserved.
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
+
+import("../../webrtc/build/webrtc.gni")
+
+group("xmllite") {
+ public_deps = [
+ ":rtc_xmllite",
+ ]
+}
+
+rtc_static_library("rtc_xmllite") {
+ sources = [
+ "qname.cc",
+ "qname.h",
+ "xmlbuilder.cc",
+ "xmlbuilder.h",
+ "xmlconstants.cc",
+ "xmlconstants.h",
+ "xmlelement.cc",
+ "xmlelement.h",
+ "xmlnsstack.cc",
+ "xmlnsstack.h",
+ "xmlparser.cc",
+ "xmlparser.h",
+ "xmlprinter.cc",
+ "xmlprinter.h",
+ ]
+
+ deps = [
+ "//third_party/webrtc/base:rtc_base",
+ ]
+
+ if (rtc_build_expat) {
+ deps += [ "//third_party/expat" ]
+ public_deps = [
+ "//third_party/expat",
+ ]
+ }
+
+ if (!build_with_chromium && is_clang) {
+ # Suppress warnings from Chrome's Clang plugins.
+ # See http://code.google.com/p/webrtc/issues/detail?id=163 for details.
+ suppressed_configs += [ "//build/config/clang:find_bad_constructs" ]
+ }
+}
+
+if (rtc_include_tests) {
+ config("xmllite_unittests_config") {
+ # GN orders flags on a target before flags from configs. The default config
+ # adds -Wall, and this flag have to be after -Wall -- so they need to
+ # come from a config and can"t be on the target directly.
+ if (is_clang) {
+ cflags = [
+ "-Wno-missing-braces",
+ "-Wno-sign-compare",
+ "-Wno-unused-const-variable",
+ ]
+ }
+ }
+
+ rtc_test("xmllite_unittests") {
+ configs += [ ":xmplite_unittests_config" ]
+
+ if (!build_with_chromium && is_clang) {
+ # Suppress warnings from the Chromium Clang plugin (bugs.webrtc.org/163).
+ suppressed_configs += [ "//build/config/clang:find_bad_constructs" ]
+ }
+
+ deps = [
+ ":rtc_xmllite",
+ "//testing/gtest",
+ "//third_party/webrtc/base:rtc_base_tests_utils",
pthatcher2 2016/12/22 20:33:29 It's a little unfortunate that this depends on web
kjellander_chromium 2016/12/27 12:49:21 Yeah, let's do that in follow-up. I posted a note
+ ]
+
+ sources = [
+ "qname_unittest.cc",
+ "xmlbuilder_unittest.cc",
+ "xmlelement_unittest.cc",
+ "xmlnsstack_unittest.cc",
+ "xmlparser_unittest.cc",
+ "xmlprinter_unittest.cc",
+ ]
+ }
+}

Powered by Google App Engine
This is Rietveld 408576698