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

Unified Diff: talk/libjingle.gyp

Issue 1412673008: Introduces Android API level linting, fixes all current API lint errors. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc@master
Patch Set: Move AndroidManifest.xml and suppressions.xml to more general location. Created 5 years, 1 month 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: talk/libjingle.gyp
diff --git a/talk/libjingle.gyp b/talk/libjingle.gyp
index cd5385c11fd30067ebde77a3ee7b4aa760364f40..55787a662527572cb0b5644b213d60a3f4eb472e 100755
--- a/talk/libjingle.gyp
+++ b/talk/libjingle.gyp
@@ -227,6 +227,12 @@
'libjingle_peerconnection_so',
],
'variables': {
+ # Designate as Chromium code and point to our lint settings to
+ # enable linting of the WebRTC code (this is the only way to make
+ # lint_action invoke the Android linter).
+ 'android_manifest_path': '<(webrtc_root)/webrtc/build/android/AndroidManifest.xml',
+ 'suppressions_file': '<(webrtc_root)/webrtc/build/android/suppressions.xml',
+ 'chromium_code': 1,
'java_in_dir': 'app/webrtc/java',
'webrtc_base_dir': '<(webrtc_root)/base',
'webrtc_modules_dir': '<(webrtc_root)/modules',

Powered by Google App Engine
This is Rietveld 408576698