<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><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">Package looks good!  Upon thinking about this, I think its probably<br>better to sort the output of `find`, e.g.<br><br>find ${INSTALL_DIR} | sort >><br>/var/lib/dpkg/info/google-${PKG_NAME}-installer.list<br><br>That goes for all of the google-android-* packages.<br></blockquote><div><br></div><div>Okey, I will patch this. </div><div> </div><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">If you are still interested in working more on these, it would be great<br>to add support for setting a custom download mirror.  For example, here<br>is a complete Chinese mirror:<br><a href="http://mirrors.neusoft.edu.cn/android/repository/" rel="noreferrer" target="_blank">http://mirrors.neusoft.edu.cn/android/repository/</a><br><br>The idea would be for the package to prompt the user which mirror to use<br>for the download when they install the package. For that you use the<br>'debconf' system:<br><a href="http://www.fifi.org/doc/debconf-doc/tutorial.html" rel="noreferrer" target="_blank">http://www.fifi.org/doc/debconf-doc/tutorial.html</a><br><br>You can see a simple example of this in action in this package:<br><a href="https://anonscm.debian.org/cgit/collab-maint/macchanger.git" rel="noreferrer" target="_blank">https://anonscm.debian.org/cgit/collab-maint/macchanger.git</a><br><br></blockquote><div><br></div><div>I Add this too, I will try find more mirrors too if possible.</div></div><div> </div><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">And another idea for improvement would be to make `wget` run as the<br>
'nobody' user, so that would mean something like:<br>
<br>
   install -d -m0700 --owner nobody $(DL_DIR)<br>
<br>
and<br>
<br>
   su nobody -s /bin/sh -c "wget --continue $(PKG_SOURCE_URL)"<br></blockquote><div><br></div><div>Noted, I will added too, I was think about this too. I will do it for all the google-android-*-installer packages.</div><div><br></div><div>One quastion: on the downloaded NDK package, there is multiple tools, each folder have an open source different licence (GPL, Apache ...), so for the ITP report, should i mention them all ?</div><div><br></div><div><i>Cheers,</i></div><div><i>Mouaad</i></div></div></div></div>