diff options
| author | satayev <satayev@google.com> | 2021-07-01 18:26:35 +0000 |
|---|---|---|
| committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | 2021-07-01 18:26:35 +0000 |
| commit | 566f41bc419c945459d76af01b635bd468eca0ef (patch) | |
| tree | f7fcaa09e7670641da3ea7adf30eb056654de2d3 | |
| parent | 18c8296320635aa5072755a58057a064489e554c (diff) | |
| parent | 37f499ed98d0c77da181ff40c5164d32556737c8 (diff) | |
| download | platform_packages_modules_SdkExtensions-566f41bc419c945459d76af01b635bd468eca0ef.tar.gz platform_packages_modules_SdkExtensions-566f41bc419c945459d76af01b635bd468eca0ef.tar.bz2 platform_packages_modules_SdkExtensions-566f41bc419c945459d76af01b635bd468eca0ef.zip | |
Add LOG statements to derive_classpath. am: 37f499ed98
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/modules/SdkExtensions/+/15157031
Change-Id: I5dde1bcf02e97c67fdeb3cf5c8b04f01c7174c5f
| -rw-r--r-- | derive_classpath/derive_classpath.cpp | 13 | ||||
| -rw-r--r-- | derive_classpath/main.cpp | 2 |
2 files changed, 12 insertions, 3 deletions
diff --git a/derive_classpath/derive_classpath.cpp b/derive_classpath/derive_classpath.cpp index 5621d72..b22e832 100644 --- a/derive_classpath/derive_classpath.cpp +++ b/derive_classpath/derive_classpath.cpp @@ -96,19 +96,24 @@ bool GlobClasspathFragments(Filepaths* fragments, const std::string& pattern) { // Writes the contents of *CLASSPATH variables to /data in the format expected by `load_exports` // action from init.rc. See platform/system/core/init/README.md. bool WriteClasspathExports(Classpaths classpaths, std::string_view output_path) { - std::stringstream out; + LOG(INFO) << "WriteClasspathExports " << output_path; + std::stringstream out; out << "export BOOTCLASSPATH " << android::base::Join(classpaths[BOOTCLASSPATH], ':') << '\n'; out << "export DEX2OATBOOTCLASSPATH " << android::base::Join(classpaths[DEX2OATBOOTCLASSPATH], ':') << '\n'; out << "export SYSTEMSERVERCLASSPATH " << android::base::Join(classpaths[SYSTEMSERVERCLASSPATH], ':') << '\n'; + const std::string& content = out.str(); + LOG(INFO) << "WriteClasspathExports content\n" << content; + const std::string path_str(output_path); - return android::base::WriteStringToFile(out.str(), path_str, /*follow_symlinks=*/true); + return android::base::WriteStringToFile(content, path_str, /*follow_symlinks=*/true); } bool ReadClasspathFragment(ExportedClasspathsJars* fragment, const std::string& filepath) { + LOG(INFO) << "ReadClasspathFragment " << filepath; std::string contents; if (!android::base::ReadFileToString(filepath, &contents)) { PLOG(ERROR) << "Failed to read " << filepath; @@ -136,10 +141,12 @@ std::string GetAllowedJarPathPrefix(const std::string& fragment_path) { // Finds and parses all classpath fragments on device matching given glob patterns. bool ParseFragments(const std::string& globPatternPrefix, Classpaths& classpaths, bool boot_jars) { - Filepaths fragments; + LOG(INFO) << "ParseFragments for " << (boot_jars ? "bootclasspath" : "systemserverclasspath"); auto glob_patterns = boot_jars ? kBootclasspathFragmentGlobPatterns : kSystemserverclasspathFragmentGlobPatterns; + + Filepaths fragments; for (const auto& pattern : glob_patterns) { if (!GlobClasspathFragments(&fragments, globPatternPrefix + pattern)) { return false; diff --git a/derive_classpath/main.cpp b/derive_classpath/main.cpp index 8cc4e0a..b896eb9 100644 --- a/derive_classpath/main.cpp +++ b/derive_classpath/main.cpp @@ -14,6 +14,7 @@ * limitations under the License. */ +#include <android-base/logging.h> #include <cstdlib> #include <string_view> @@ -26,6 +27,7 @@ int main(int argc, char** argv) { } else if (argc == 2) { output_location = argv[1]; } else { + LOG(ERROR) << "too many arguments " << argc; return EXIT_FAILURE; } if (!android::derive_classpath::GenerateClasspathExports(output_location)) { |
