<div dir="ltr">(cc)<div><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><i>Mouaad Aallam</i></div></div></div>
<br><div class="gmail_quote">---------- Forwarded message ----------<br>From: <b class="gmail_sendername">Mouaad Aallam</b> <span dir="ltr"><<a href="mailto:mouaad@aallam.com">mouaad@aallam.com</a>></span><br>Date: Mon, Jul 11, 2016 at 2:52 PM<br>Subject: Re: [Android-tools-devel] google-android-sdk-docs-installer<br>To: Hans-Christoph Steiner <<a href="mailto:hans@at.or.at">hans@at.or.at</a>><br><br><br><div dir="ltr"><span class=""><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">Looks great overall, nice work with the debconf stuff!  A couple of<br>
comments:<br>
<br>
* I found another mirror that looks active:<br>
  <a href="http://android-mirror.bugly.qq.com:8080/android/repository/" rel="noreferrer" target="_blank">http://android-mirror.bugly.qq.com:8080/android/repository/</a></blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><br>
* I'm pretty sure <a href="http://dl-ssl.google.com" rel="noreferrer" target="_blank">dl-ssl.google.com</a> is deprecated and <a href="http://dl.google.com" rel="noreferrer" target="_blank">dl.google.com</a> only<br>
works over HTTPS, we should use <a href="https://dl.google.com" rel="noreferrer" target="_blank">https://dl.google.com</a><br>
* when the host is called 'dl-ssl', its misleading to use an HTTP<br>
connection, i.e. not HTTPS<br>
* the debconf choices should include the URL scheme (https://) so people<br>
can see what kind of connection they are getting<br>
<br>
I think it makes sense to use these as the options:<br>
<br>
* <a href="https://dl.google.com" rel="noreferrer" target="_blank">https://dl.google.com</a><br>
* <a href="http://mirrors.neusoft.edu.cn" rel="noreferrer" target="_blank">http://mirrors.neusoft.edu.cn</a><br>
* <a href="http://android-mirror.bugly.qq.com:8080" rel="noreferrer" target="_blank">http://android-mirror.bugly.qq.com:8080</a><br>
</blockquote></div><br></div></span><div class="gmail_extra">Hello ! :)</div><div class="gmail_extra">I've taken all your comments in consideration and I push a commit[1] please check it, I can apply same changes to other google-android-*-installer packages.</div><div class="gmail_extra"><br></div><div class="gmail_extra">[1]: <a href="https://anonscm.debian.org/cgit/android-tools/google-android-sdk-docs-installer.git/commit/" target="_blank">https://anonscm.debian.org/cgit/android-tools/google-android-sdk-docs-installer.git/commit/</a><br></div></div>
</div><br></div></div>