From 35fb39ee052155e901ed56449b0e5af6f905cd52 Mon Sep 17 00:00:00 2001 From: Raphael Date: Mon, 14 Sep 2009 15:24:47 -0700 Subject: [PATCH] SDK Updater: fix wrong error when unable to get repository.xml BUG 2095530 Change-Id: I19622378b8da049f1c3416b8e0951ecc1f7475f0 --- sdkmanager/app/.classpath | 1 + .../sdklib/src/com/android/sdklib/internal/repository/RepoSource.java | 4 +++- .../com/android/sdkuilib/internal/repository/RepoSourcesAdapter.java | 2 +- 3 files changed, 5 insertions(+), 2 deletions(-) diff --git a/sdkmanager/app/.classpath b/sdkmanager/app/.classpath index 4f2e142dd..3380ba1a7 100644 --- a/sdkmanager/app/.classpath +++ b/sdkmanager/app/.classpath @@ -7,5 +7,6 @@ + diff --git a/sdkmanager/libs/sdklib/src/com/android/sdklib/internal/repository/RepoSource.java b/sdkmanager/libs/sdklib/src/com/android/sdklib/internal/repository/RepoSource.java index d86859dce..b0bc50cf4 100755 --- a/sdkmanager/libs/sdklib/src/com/android/sdklib/internal/repository/RepoSource.java +++ b/sdkmanager/libs/sdklib/src/com/android/sdklib/internal/repository/RepoSource.java @@ -192,8 +192,10 @@ public class RepoSource implements IDescription { if (exception[0] != null) { if (exception[0] instanceof FileNotFoundException) { reason = "File not found"; + mFetchError += ": " + reason; } else if (exception[0] instanceof SSLKeyException) { - reason = "SSL error. You might want to force download through http in the settings."; + reason = "HTTPS SSL error. You might want to force download through HTTP in the settings."; + mFetchError += ": HTTPS SSL error"; } else if (exception[0].getMessage() != null) { reason = exception[0].getMessage(); } diff --git a/sdkmanager/libs/sdkuilib/src/com/android/sdkuilib/internal/repository/RepoSourcesAdapter.java b/sdkmanager/libs/sdkuilib/src/com/android/sdkuilib/internal/repository/RepoSourcesAdapter.java index de12666e2..9c6a80af2 100755 --- a/sdkmanager/libs/sdkuilib/src/com/android/sdkuilib/internal/repository/RepoSourcesAdapter.java +++ b/sdkmanager/libs/sdkuilib/src/com/android/sdkuilib/internal/repository/RepoSourcesAdapter.java @@ -208,7 +208,7 @@ public class RepoSourcesAdapter { packages = null; } - if (packages != null && source.getFetchError() != null) { + if (packages == null && source.getFetchError() != null) { // Return a dummy entry to display the fetch error return new Object[] { new RepoSourceError(source) }; } -- 2.11.0