From 75e7d1d73e74cdc85a5fb731fa13159ab073d939 Mon Sep 17 00:00:00 2001 From: Aurimas Liutikas Date: Fri, 17 Feb 2017 12:06:41 -0800 Subject: [PATCH] Removes an already defined warning in aapt. Apps that expect to use the current SDK don't specify anything in their manifest and the build system rightfully injects the current SDK. Any apps that specify their own or versionCode are telling the build system that they are purposely using some other minSdkVersion, etc (support library demos). Test: None Change-Id: I0aba3a71e15703c7bcafcf4d35608a1851dd8567 Fixes: 34901699 --- tools/aapt/Resource.cpp | 9 --------- 1 file changed, 9 deletions(-) diff --git a/tools/aapt/Resource.cpp b/tools/aapt/Resource.cpp index cf5badc821179..881ac87480831 100644 --- a/tools/aapt/Resource.cpp +++ b/tools/aapt/Resource.cpp @@ -721,11 +721,6 @@ bool addTagAttribute(const sp& node, const char* ns8, XMLNode::attribute_entry* existingEntry = node->editAttribute(ns, attr); if (existingEntry != NULL) { if (replaceExisting) { - if (kIsDebug) { - printf("Info: AndroidManifest.xml already defines %s (in %s);" - " overwriting existing value from manifest.\n", - String8(attr).string(), String8(ns).string()); - } existingEntry->string = String16(value); return true; } @@ -737,10 +732,6 @@ bool addTagAttribute(const sp& node, const char* ns8, return false; } - fprintf(stderr, "Warning: AndroidManifest.xml already defines %s (in %s);" - " using existing value in manifest.\n", - String8(attr).string(), String8(ns).string()); - // don't stop the build. return true; }